Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@danmcp
Copy link
Member

@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
@danmcp danmcp force-pushed the gguf branch 3 times, most recently from 0b9922e to e361e49 Compare April 19, 2025 21:03
@mergify mergify bot added the ci-failure PR has at least one CI failure label Apr 19, 2025
@mergify mergify bot removed the ci-failure PR has at least one CI failure label 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
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
Member 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
Contributor

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

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

Labels

CI/CD Affects CI/CD configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants