-
Notifications
You must be signed in to change notification settings - Fork 8
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
patch(integration_test_charm.yaml): Free up disk space #104
Conversation
613adba
to
6aaafa6
Compare
depends on #105 |
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, but:
-
Are we cleaning the same pathes for Canonical runners GH runners? It looks strange, but code is correct. LGTM.
-
we will have surprises if AGENT_TOOLSDIRECTORY=/
Better safe then sorry (c) MrMEEE/bumblebee-Old-and-abbandoned#123.
IMHO, at least:
: "${AGENT_TOOLSDIRECTORY?ERROR: variable 'AGENT_TOOLSDIRECTORY' is NOT defined, aborted.}"
...
rm -rf "${AGENT_TOOLSDIRECTORY:?}"
Reduce issues with running out of disk space (causing the runner to crash)
6aaafa6
to
98a1d45
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.
Nice improvements! It will be useful to see disk details in all test stages. Tnx!
Reduce issues with running out of disk space (causing the runner to crash)