Skip to content

Lock default image version#353

Closed
gberche-orange wants to merge 1 commit intooauth2-proxy:mainfrom
orange-cloudfoundry:lock-default-image-version
Closed

Lock default image version#353
gberche-orange wants to merge 1 commit intooauth2-proxy:mainfrom
orange-cloudfoundry:lock-default-image-version

Conversation

@gberche-orange
Copy link
Copy Markdown

@gberche-orange gberche-orange commented Sep 17, 2025

Expecting renovate helm-values manager to regularly propose bumps the container image version.

See https://docs.renovatebot.com/modules/manager/helm-values/

In this PR the N-1 version (v7.11.0) is used to demonstrate bump by renovate, with a a sample PR at orange-cloudfoundry#2 that includes release notes from oauth2-proxy

Fixes #354

Expecting renovate helm-values manager to regularly propose bumps the container image version.

See https://docs.renovatebot.com/modules/manager/helm-values/
@tuunit
Copy link
Copy Markdown
Member

tuunit commented Sep 17, 2025

This doesn't follow helm chart best practices.

Helm docs state:

appVersion: The version of the app that this contains (optional). Needn't be SemVer. Quotes recommended.

https://helm.sh/docs/topics/charts/

and helm charts per default when created include a substitution that checks if a tag has been set by an end user of the Chart or if empty use the appVersion specified in the Chart.yaml

@tuunit tuunit closed this Sep 17, 2025
@tuunit tuunit mentioned this pull request Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lock default OCI images

2 participants