Stop deleting all plan working directories at scheduler start #598
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jherbel
force-pushed
the
dev/joerg/CMK-18459_setup_plan_directories
branch
14 times, most recently
from
4485f3f
to
9aeff05
Compare
Since we don't delete the entire working directory anymore at scheduler start, it's not enough to grant permissions to the user configured for headed execution (if any). We also have to ensure that no other, previously configured user has access. CMK-18459
jherbel
force-pushed
the
dev/joerg/CMK-18459_setup_plan_directories
branch
from
9aeff05
to
4faf074
Compare
1) Working directories of plans that are no long configured should be removed. 2) Already existing plan directories of configured plans should not be emptied. CMK-18459
We create another user and grant this user full access to one of the plan working directories. This access should be removed during the scheduler setup. CMK-18459
jherbel
force-pushed
the
dev/joerg/CMK-18459_setup_plan_directories
branch
from
4faf074
to
9a9d9af
Compare
CMK-18459
racicLuka
approved these changes
Aug 26, 2024
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.