Sync changes from develop #3278
Merged
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
SlurmGCP. Add workaround for GoogleCloudPlatform#3242 to examples
SlurmGCP. Add `set -e` to prolog mux
SlurmGCP. Escape GCP error reasons that may cause malformed CLI args
…warning Fix formatting of Docker config warning
Filestore deletion protection ensures that instances are not unintentionally deleted. A typical lifecycle for a user will look like: 1. Deploy a blueprint with deletion protection enabled 2. Disable deletion protection in blueprint 3. Redeploy blueprint 4. Destroy deployment In particular, enabling Filestore deletion protection does not prevent Terraform from destroying other resources. So a `gcluster destroy` command will destroy all resources except the Filestore and its dependencies.
…egration-tests Script for Python Tests
This is a temporary workaround to prevent failures while we debug the issue.
…able_lustre Don't install Lustre in A3 tests
Co-authored-by: Rachael Tamakloe <[email protected]>
…eletion Add support for Filestore deletion protection
…ate_docs Remove CentOS 7 from list of supported images
…health add gpu health check in prolog and epilog
…g-latest Fix a bug where try was hiding extraction of gpu driver version
SlurmGCP. Don't use remote module to create controller instance
…t-version-2 Fix the gpu_installation_config default for case where no customer input
add firewall to allow tcp traffic for parallelstore
…m-node-pool-disk Allow specifying GKE's system node pool disk properties
…better performance"
…-doc-update Add documentation on opportunistic GCP maintenance support in Slurm
…revert-3232-develop Revert "update a3 machines local ssd to use nvme instead of scsi for better performance"
…_dockerfile Adds Cluster Toolkit Dockerfile for backend integration with XPK
…l-kueue add support for kueue v0.9.0 to enable TAS
…ud_ops Add cluster and hostname as cloud ops labels
Add support for kueue v0.9.0 to enable TAS
ankitkinra
approved these changes
Nov 18, 2024
annuay-google
merged commit 5a82958
into
GoogleCloudPlatform:experimental
5 of 55 checks passed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Back merge of most recent release
Submission Checklist
NOTE: Community submissions can take up to 2 weeks to be reviewed.
Please take the following actions before submitting this pull request.