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

.github: Fail if print-chart-version.sh fails or does not exist #26086

Merged
merged 1 commit into from Jun 9, 2023

Conversation

chancez
Copy link
Contributor

@chancez chancez commented Jun 9, 2023

If print-chart-version.sh doesn't exist (ie: if it isn't back ported to a stable branch) the workflow does not fail, so make it fail so we don't push invalid charts.

@chancez chancez added area/CI Continuous Integration testing issue or flake kind/cleanup This includes no functional changes. labels Jun 9, 2023
@chancez chancez requested a review from a team as a code owner June 9, 2023 17:06
@chancez chancez self-assigned this Jun 9, 2023
@chancez chancez requested a review from a team as a code owner June 9, 2023 17:06
@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 Jun 9, 2023
Copy link
Contributor

@michi-covalent michi-covalent left a comment

Choose a reason for hiding this comment

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

magical pipe

@michi-covalent michi-covalent added the release-note/ci This PR makes changes to the CI. label Jun 9, 2023
@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 Jun 9, 2023
Signed-off-by: Chance Zibolski <chance.zibolski@gmail.com>
@chancez chancez force-pushed the pr/chancez/chart_push_check_errors branch from 00c8640 to 863ac47 Compare June 9, 2023 18:07
@michi-covalent
Copy link
Contributor

chance tested this in a forked repo. no need to run the full CI since it only modifies .github/workflows/push-chart-ci.yaml. let's wait for the basic CI to finish, and i'll merge it.

@michi-covalent michi-covalent merged commit dbc2502 into main Jun 9, 2023
49 checks passed
@michi-covalent michi-covalent deleted the pr/chancez/chart_push_check_errors branch June 9, 2023 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake kind/cleanup This includes no functional changes. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants