-
Notifications
You must be signed in to change notification settings - Fork 450
Preserve storage after e2e tests #3303
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
Conversation
0b9922e to
e361e49
Compare
There really isn't a reason to delete it and knowing the size after the tests is helpful to understand how large of a disk is needed. Signed-off-by: Dan McPherson <[email protected]>
|
@danmcp would you mind explaining a bit more about how you're testing this? The GH Action will delete the node as soon as the e2e script completes, right? Are you doing something behind the scenes to avoid deleting the node? |
The change is useful because we print out the usage after the test is run. Example: Currently we delete the test results then delete the node immediately after. That makes it so reporting the disks after the tests isn't helpful to determine how much disk was needed. To test, I simply ran the script with the parameter to prove it works. We won't see the actual numbers until it merges since the workflows don't take effect with pull request target. |
|
Oh, got it! This would be faster as well, then. |

Updating to get a better idea over time how much storage is being used and the storage isn't needed after the test
Checklist:
conventional commits.