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

Remove command to rm non-existent directory #239

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

idlewis
Copy link
Member

@idlewis idlewis commented Jan 13, 2021

At the point in the dockerfiles where /output/workarea is removed, the directory doesn't exist, as /output is a symlink that hasn't been created yet.
However, removing the real workarea directory slows down server startup, so fix up the dockerfiles by removing the duff command (rather than correcting it to remove the real workarea directory)

@idlewis idlewis requested a review from leochr January 13, 2021 14:37
@leochr
Copy link
Member

leochr commented Mar 19, 2021

@idlewis It could be the removal of /output/workarea inside populate_scc.sh that slowed down the server startup (from this PR ).

Could you please compare with just deleting the correct directory ($WLP_OUTPUT_DIR/defaultServer/workarea) in Dockerfiles?

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

Successfully merging this pull request may close these issues.

2 participants