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

Add configurable suffix for operator image repo in helm #14952

Merged
merged 2 commits into from
Feb 14, 2021

Conversation

nebril
Copy link
Member

@nebril nebril commented Feb 12, 2021

Because operator has cloud provider custom suffixes we are not able to add
custom suffixes (like -dev or -ci) for the operator image seamlesly.

This change also enables this option to be used in test runs.

@nebril nebril requested review from a team as code owners February 12, 2021 11:20
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Feb 12, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Feb 12, 2021
@nebril
Copy link
Member Author

nebril commented Feb 12, 2021

test-net-next

@nebril nebril added the release-note/misc This PR makes changes that have no direct user impact. label Feb 12, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Feb 12, 2021
@nebril
Copy link
Member Author

nebril commented Feb 12, 2021

test-net-next

@qmonnet qmonnet removed their assignment Feb 12, 2021
Because operator has cloud provider custom suffixes we are not able to add custom
suffixes (like `-dev` or `-ci`) for the operator image seamlesly.

New operator-specific option allows to do this.

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
This option makes it possible to use new operator.image.suffix Helm
value in test runs.

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
@nebril
Copy link
Member Author

nebril commented Feb 12, 2021

test-4.9

@joestringer joestringer moved this from Done to In progress in 1.10.0 Feb 12, 2021
@aanm aanm merged commit 1141308 into master Feb 14, 2021
1.10.0 automation moved this from In progress to Done Feb 14, 2021
@aanm aanm deleted the pr/helm-image-suffix branch February 14, 2021 12:38
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.9.5 Mar 3, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.9 in 1.9.5 Mar 5, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.9 to Backport done to v1.9 in 1.9.5 Mar 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.9.5
Backport done to v1.9
Development

Successfully merging this pull request may close these issues.

None yet

9 participants