Skip to content

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
@chancez chancez requested a review from viktor-kurchenko 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
@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.

4 participants