Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace invalid "+" with valid character #1221

Merged
merged 1 commit into from
Oct 11, 2024
Merged

replace invalid "+" with valid character #1221

merged 1 commit into from
Oct 11, 2024

Conversation

ritzk
Copy link
Contributor

@ritzk ritzk commented Oct 11, 2024

What does this PR do?

replace '+' with '_' in chart version.

flux will version chart with + when using revision reconciliation strategy. This break deployment.

  • Fixes #

deployment via flux

 Helm upgrade failed for release jenkins/jenkins with chart [email protected]+087b605ce3dd.11: 
   cannot patch "jenkins-jenkins-config-credentials" with kind ConfigMap: 
     ConfigMap "jenkins-jenkins-config-credentials" is invalid: 
       metadata.labels: Invalid value: "jenkins-5.6.1+087b605ce3dd.11":

If you modified files in the ./charts/jenkins/ directory, please also include the following:

Submitter checklist

Special notes for your reviewer

@ritzk ritzk force-pushed the main branch 2 times, most recently from 1e0b9f4 to 07e82f6 Compare October 11, 2024 13:11
@ritzk ritzk marked this pull request as ready for review October 11, 2024 13:13
@ritzk ritzk requested a review from a team as a code owner October 11, 2024 13:13
Copy link
Member

@timja timja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does it have that version we don't have that? Can you point to some documentation from helm or flux please.

@ritzk
Copy link
Contributor Author

ritzk commented Oct 11, 2024

Why does it have that version we don't have that? Can you point to some documentation from helm or flux please.

jenkins chart is checked into local git repo (mirrored from upstream).

flux/source-controller will append git hash to chart version when using gitrepository with revision reconciliation
https://github.com/fluxcd/source-controller/blob/main/CHANGELOG.md#0191

@timja timja merged commit c573cb0 into jenkinsci:main Oct 11, 2024
6 checks passed
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.

2 participants