Skip to content
This repository was archived by the owner on Apr 23, 2026. It is now read-only.

Preserve storage after e2e tests#3303

Merged
mergify[bot] merged 1 commit intoinstructlab:mainfrom
danmcp:gguf
Apr 28, 2025
Merged

Preserve storage after e2e tests#3303
mergify[bot] merged 1 commit intoinstructlab:mainfrom
danmcp:gguf

Conversation

@danmcp
Copy link
Copy Markdown
Contributor

@danmcp danmcp commented Apr 18, 2025

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

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

@danmcp danmcp marked this pull request as draft April 18, 2025 21:44
@mergify mergify Bot added CI/CD Affects CI/CD configuration ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Apr 18, 2025
@danmcp danmcp changed the title Avoid temp file on gguf write due to out of disk errors Preserve storage after medium test Apr 19, 2025
@mergify mergify Bot added ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Apr 19, 2025
@danmcp danmcp changed the title Preserve storage after medium test Preserve storage after e2e tests Apr 19, 2025
@danmcp danmcp marked this pull request as ready for review April 19, 2025 21:47
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]>
@ktdreyer
Copy link
Copy Markdown
Contributor

@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?

@danmcp
Copy link
Copy Markdown
Contributor Author

danmcp commented Apr 28, 2025

@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:

image

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.

@mergify mergify Bot added the one-approval PR has one approval from a maintainer label Apr 28, 2025
@booxter booxter requested a review from a team April 28, 2025 21:19
@mergify mergify Bot merged commit d3a4190 into instructlab:main Apr 28, 2025
24 checks passed
@mergify mergify Bot removed the one-approval PR has one approval from a maintainer label Apr 28, 2025
@ktdreyer
Copy link
Copy Markdown
Contributor

Oh, got it! This would be faster as well, then.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CI/CD Affects CI/CD configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants