Skip to content
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

WIP: Remove build artifacts in pipeline after tests finish #3117

Conversation

AdamBrousseau
Copy link
Contributor

  • Delete all Build artifacts after the overall pipeline has completed
  • Always remove artifacts even if tests fail

Related #1017
[skip ci]

Signed-off-by: Adam Brousseau adam.brousseau88@gmail.com

@AdamBrousseau
Copy link
Contributor Author

A hack way to remove the SDKs after the tests finish. You would only be able to access them from the build jobs when the tests are running. Would currently save us 35G on master.
Thoughts @DanHeidinga @pshipton @smlambert @jdekonin ?

@AdamBrousseau
Copy link
Contributor Author

I could also wrap this in a parameter so we have the option to turn it off at launch time or via variables config file.

@DanHeidinga
Copy link
Member

Reading #1017, it mentions a buildDiscarder. Does this replace the buildDiscarder or in addition to it? Or was that never completed?

#1017 (comment)

@AdamBrousseau AdamBrousseau force-pushed the cleanup_build_artifact_after_test branch 4 times, most recently from 3c48d43 to e15a465 Compare October 13, 2018 02:35
- Delete all Build artifacts after the overall pipeline has completed
- Always remove artifacts even if tests fail

Related eclipse-openj9#1017
[skip ci]

Signed-off-by: Adam Brousseau <adam.brousseau88@gmail.com>
@AdamBrousseau AdamBrousseau force-pushed the cleanup_build_artifact_after_test branch from e15a465 to 1d1603f Compare October 13, 2018 03:02
@AdamBrousseau
Copy link
Contributor Author

Closing this for now as I don't believe it is desired behaviour.

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

Successfully merging this pull request may close these issues.

3 participants