-
Notifications
You must be signed in to change notification settings - Fork 37
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
ci: ensure nightly builds actually runs #143
Conversation
557d19c
to
286421c
Compare
Codecov Report
@@ Coverage Diff @@
## main #143 +/- ##
==========================================
- Coverage 40.09% 39.47% -0.63%
==========================================
Files 32 32
Lines 1444 1477 +33
==========================================
+ Hits 579 583 +4
- Misses 789 818 +29
Partials 76 76
Continue to review full report at Codecov.
|
0e45af8
to
f527e32
Compare
5fb2c87
to
493c88f
Compare
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.
I'm not surrpised this never worked via action as it was only tested locally. Thanks for also adding the janitor.
ffs i swear those commits were signed when i pushed them |
- ensure TF metal create works - cleanup projects after e2e run - use the correct format for the terraform action - ensure TF outputs are correctly formatted. The TF wrapper included in the action manipulates stdout and stderr to make it compatible with use in action pipelines, but that unfortunately makes it a pain to work with if you are using the output elsewhere.
What this PR does / why we need it:
Fixes the e2e nightlies, it appears they never actually had a successful run.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #127