Kong / Kong/kubernetes-testing-framework

When kong chart is already deployed and in a broken state error is silently ignored because of name re-use

Open Beginner friendly
#289 0 comments 1 reaction 0 assignees View on GitHub
bug priority/low
Dominant language
Go
Stars
44
Forks
11
PR merge metrics
No merged PRs in 30d

Description

When user already has kong addon deployed in an existing cluster and it's in a broken state then no error will be displayed causing confusing test failures etc.

https://github.com/Kong/kubernetes-testing-framework/blob/b3ce22e2996d048ec4452d0369b36e1db094bca1/pkg/clusters/addons/kong/addon.go#L275-L277

There could be for instance more logic introduced around kong's helm release deployment or simply a warning being printed out whenever the problem described in this issue occurs.

### Acceptance criteria

- [ ] Log an error when chart is already deployed and there's an error deploying

Contributor guide

Open the contributing guide

Research direction

Start in pkg/clusters/addons/kong/addon.go at the deployment logic around lines 275-277, and trace how an existing Kong Helm release error is handled. Make the described deployment failure visible through an error log, then verify the acceptance criterion against the broken, already-deployed chart scenario.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, helm, kubernetes
Domain
devops, testing-qa
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.