Skip to content

Add recurse to condor spool directory#4178

Merged
tpdownes merged 1 commit into
GoogleCloudPlatform:developfrom
aneo-ssam:htcondor_fix_spool_uid
May 23, 2025
Merged

Add recurse to condor spool directory#4178
tpdownes merged 1 commit into
GoogleCloudPlatform:developfrom
aneo-ssam:htcondor_fix_spool_uid

Conversation

@aneo-ssam

Copy link
Copy Markdown
Contributor

This fix ensure that the condor spool directory files are accessible even if for some reason the new image's condor UID changed when using stateful disk

Submission Checklist

NOTE: Community submissions can take up to 2 weeks to be reviewed.

Please take the following actions before submitting this pull request.

[x] Fork your PR branch from the Toolkit "develop" branch (not main)
[x] Test all changes with pre-commit in a local branch #
[x] Confirm that "make tests" passes all tests
[x] Add or modify unit tests to cover code changes
[x] Ensure that unit test coverage remains above 80%
[x] Update all applicable documentation
[x] Follow Cluster Toolkit Contribution guidelines #

@aneo-ssam aneo-ssam requested review from a team and samskillman as code owners May 22, 2025 16:14
@tpdownes

Copy link
Copy Markdown
Contributor

/gcbrun

@tpdownes tpdownes self-assigned this May 22, 2025
@tpdownes tpdownes added the release-bugfix Added to release notes under the "Bug fixes" heading. label May 23, 2025

@tpdownes tpdownes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for the submission!

@tpdownes tpdownes self-requested a review May 23, 2025 13:50
@tpdownes tpdownes enabled auto-merge May 23, 2025 13:50
@tpdownes tpdownes merged commit 123ffde into GoogleCloudPlatform:develop May 23, 2025
12 of 68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-bugfix Added to release notes under the "Bug fixes" heading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants