-
Notifications
You must be signed in to change notification settings - Fork 10
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
introduce tests to ensure chart image repository/tags exist #187
Comments
Hey @tobybellwood could you provide a link to an example test run with this issue? |
ha! Most of mine from last week until I figured it out! Here's one from today - https://github.com/uselagoon/lagoon-charts/runs/1586136229?check_suite_focus=true The issue is at https://github.com/uselagoon/lagoon-charts/runs/1586136229?check_suite_focus=true#step:8:3505 |
I'm confused, isn't that error what you want to see? |
yeah - sorry - it took a lot of hunting to work out what it was - the initial error was that it "timed out". This isn't a must-have, just a nice to have -I'll think about it a bit! |
I hunted around in the github action s documentation, couldn't find anything about highlighting logs based on regex. |
Hey I'm just grooming issues - is this still a problem? |
Given the complex inter-relations between the chart versions and lagoon versions, we should introduce a test that ensures that all images referenced in the charts are actually pullable. Currently, there is no output to GitHub Actions if the rollout of a chart fails because of a missing image.
There's a pretty neat solution at https://github.com/helm-lab/helm-images/blob/master/images.sh using a dry-run to extract the image refs.
The text was updated successfully, but these errors were encountered: