Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

Core: Distributor doesn't get the version #6652

Closed
4 tasks
thisthat opened this issue Jan 21, 2022 · 0 comments · Fixed by #6658
Closed
4 tasks

Core: Distributor doesn't get the version #6652

thisthat opened this issue Jan 21, 2022 · 0 comments · Fixed by #6658
Assignees
@bacherfl
Labels
area:core type:bug Something is not working as intended/documented
Milestone
0.13.0

Comments

@thisthat
Copy link
Member

Action Item

The distributor always reports 0.9.0 as version https://github.com/keptn/keptn/blob/master/distributor/pkg/config/envconfig.go#L32
Most likely is due to this helm configuration that is used in all distributor configuration: https://github.com/keptn/keptn/blob/master/installer/manifests/keptn/charts/control-plane/templates/quality-gates.yaml#L95

Acceptance Criteria

  • The distributor shows the correct version
  • Our helm charts use a correct configuration

DoD

  • Unit tests
  • Manual testing
@thisthat thisthat added area:core ready-for-refinement Issue is relevant for the next backlog refinment labels Jan 21, 2022
@thisthat thisthat added this to the 0.14.0 milestone Jan 21, 2022
@thisthat thisthat added type:bug Something is not working as intended/documented and removed ready-for-refinement Issue is relevant for the next backlog refinment labels Jan 24, 2022
@thisthat thisthat modified the milestones: 0.14.0, 0.13.0 Jan 24, 2022
@bacherfl bacherfl self-assigned this Jan 24, 2022
bacherfl added a commit that referenced this issue Jan 24, 2022
@bacherfl
fix(installer): Set distributor version in helm chart (#6652)
78ff986 
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@bacherfl bacherfl mentioned this issue Jan 24, 2022
Merged
bacherfl added a commit that referenced this issue Jan 24, 2022
@bacherfl
fix(installer): Set distributor version in helm chart (#6652) (#6658)
8c2d8de 
* fix(installer): Set distributor version in helm chart (#6652)

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed helm chart

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bacherfl bacherfl

Labels
area:core type:bug Something is not working as intended/documented
Projects
None yet
Milestone
0.13.0
Development

Successfully merging a pull request may close this issue.

fix(installer): Set distributor version in helm chart (#6652) keptn/keptn
2 participants
@thisthat @bacherfl