Skip to content

Fix ordering of local SSD mounting and docker#3682

Merged
tpdownes merged 1 commit into
GoogleCloudPlatform:developfrom
tpdownes:fix_startup_race
Feb 19, 2025
Merged

Fix ordering of local SSD mounting and docker#3682
tpdownes merged 1 commit into
GoogleCloudPlatform:developfrom
tpdownes:fix_startup_race

Conversation

@tpdownes
Copy link
Copy Markdown
Contributor

Upon a reboot of a VM with Docker configured to use local SSD RAID array for its data root, the existing ordering does not guarantee the disk will be ready when docker is. This change ensure the ordering is proper.

Submission Checklist

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

Please take the following actions before submitting this pull request.

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

@tpdownes tpdownes self-assigned this Feb 18, 2025
@tpdownes tpdownes assigned harshthakkar01 and unassigned tpdownes Feb 19, 2025
@tpdownes tpdownes marked this pull request as ready for review February 19, 2025 00:56
@tpdownes tpdownes added the release-bugfix Added to release notes under the "Bug fixes" heading. label Feb 19, 2025
Upon a reboot of a VM with Docker configured to use local SSD RAID array
for its data root, the existing ordering does not guarantee the disk
will be ready when docker is. This change ensure the ordering is proper.
@tpdownes tpdownes merged commit dda8b5b into GoogleCloudPlatform:develop Feb 19, 2025
@tpdownes tpdownes deleted the fix_startup_race branch February 19, 2025 15:18
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