-
Notifications
You must be signed in to change notification settings - Fork 17
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
Backport to v1.6.x #796
Backport to v1.6.x #796
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## v1.6.x #796 +/- ##
=========================================
Coverage ? 57.74%
=========================================
Files ? 42
Lines ? 5180
Branches ? 0
=========================================
Hits ? 2991
Misses ? 1868
Partials ? 321 ☔ View full report in Codecov by Sentry. |
The default channel should be added in this PR too. That's the reason for keeping it as a draft |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
9195f38
to
0b133f7
Compare
This commit adds a development channel reference to builds done in OBS (under or branched from isv:Rancher:Elemental) Signed-off-by: David Cassany <[email protected]> (cherry picked from commit 6ee9211)
Signed-off-by: David Cassany <[email protected]> (cherry picked from commit 0f59b14)
* Update channels * Adding a downgrade/upgrade charts e2e test * Only fetch logs for running containers * Do not fail fast in e2e matrix Signed-off-by: David Cassany <[email protected]> (cherry picked from commit d7a712d)
Signed-off-by: Loic Devulder <[email protected]> (cherry picked from commit b866419)
Signed-off-by: David Cassany <[email protected]>
0b133f7
to
e9f1012
Compare
{{ if hasPrefix "registry.opensuse.org" .Values.registryUrl }} | ||
# Unstable channel for isv:Rancher:Elemental OBS projects | ||
# Unstable channel for testing isv:Rancher:Elemental OBS projects | ||
# it is only rendered if the registryUrl value includes a known OBS project reference |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be present also on the final released chart, right?
Not 100% happy that we bring this in the final chart. I see it shouldn't harm, but production ready version should be free of the test references IMHO...
Btw, we can drop it in the marketplace later if needed (maybe even the whole channel-dev 🤔 ... for a later discussion 😄 ).
Backporting changes related to channels