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

Reenable disableDeferredWipeout also add deleteDirs #4943

Merged
merged 1 commit into from
Mar 13, 2019

Conversation

AdamBrousseau
Copy link
Contributor

@AdamBrousseau AdamBrousseau commented Mar 1, 2019

  • This will force the job to wait for the WS to be
    deleted rather than let it execute asynchronously.
  • Asynchronous delete often leaves the WS not completely
    removed in a folder called JOB_NAME_ws-cleanup_TIMESTAMP.
    Particularily bad on Windows. This has caused us to
    create a cleanup job to the remove all workspaces on
    all nodes. Hopefully this change will elimiate the
    lingering workspaces.

JENKINS-54225
Related #3510 #3309
[skip ci]

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

@AdamBrousseau
Copy link
Contributor Author

AdamBrousseau commented Mar 1, 2019

@AdamBrousseau
Copy link
Contributor Author

AdamBrousseau commented Mar 1, 2019

@AdamBrousseau AdamBrousseau force-pushed the reenable_defered_wipeout branch from ecf6dc9 to 89dba5c Compare March 4, 2019 18:01
@AdamBrousseau AdamBrousseau marked this pull request as ready for review March 4, 2019 18:02
- This will force the job to wait for the WS to be
  deleted rather than let it execute asynchronously.
- Asynchronous delete often leaves the WS not completely
  removed in a folder called JOB_NAME_ws-cleanup_TIMESTAMP.
  Particularily bad on Windows. This has caused us to
  create a cleanup job to the remove all workspaces on
  all nodes. Hopefully this change will elimiate the
  lingering workspaces.

JENKINS-54225
Related eclipse-openj9#3510 eclipse-openj9#3309
[skip ci]

Signed-off-by: Adam Brousseau <adam.brousseau88@gmail.com>
@AdamBrousseau AdamBrousseau force-pushed the reenable_defered_wipeout branch from 89dba5c to c502dfe Compare March 4, 2019 18:08
@AdamBrousseau
Copy link
Contributor Author

@vsebe for review

Copy link
Contributor

@jdekonin jdekonin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@AdamBrousseau
Copy link
Contributor Author

@llxia for merge

@llxia llxia merged commit ee1034c into eclipse-openj9:master Mar 13, 2019
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.

4 participants