-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
CODENVY-2199: Do not snapshot /tmp folder #5168
Conversation
Signed-off-by: Mykola Morhun <mmorhun@codenvy.com>
Is this true for Che as well? Or only Codenvy? |
@bmicklea for both Che and Codenvy |
This will require docs changes that will have to be approved before I can approve this PR: eclipse-che/che-docs#232 |
@slemeur do you think we need to mention this in the release notes? |
@bmicklea : Yes I think we can have a section for the enhancement of the snapshotting policy. (based on the docs) |
Signed-off-by: Mykola Morhun <mmorhun@codenvy.com>
Build # 2646 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2646/ to view the results. |
Signed-off-by: Mykola Morhun <mmorhun@codenvy.com>
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/2655/ |
What does this PR do?
Prevents adding
/tmp
folder from a workspace container into a snapshot.Added ability to specify a user for docker exec.
What issues does this PR fix or reference?
codenvy/codenvy#2199
Changelog
Ignore
/tmp
folder when creating workspace snapshot.Release Notes
Snapshots No Longer Capture
tmp
We have implemented a small but important change in how we handle snapshots. In order to speed up snapshotting when stopping a workspace, and snapshot restore when workspaces startup we are no longer snapshotting the
tmp
directory. In many cases this has trimmed >50% of the total snapshot size and noticeably increased the speed of snapshotting and restoring.Docs PR
eclipse-che/che-docs#232