Update H4D blueprint: disable automatic updates, provide image info, and delete duplicate filestore#4644
Merged
Neelabh94 merged 1 commit intoSep 24, 2025
Conversation
c319207 to
3aeecfa
Compare
bytetwin
previously requested changes
Sep 12, 2025
3aeecfa to
567ba4e
Compare
samskillman
previously requested changes
Sep 22, 2025
Collaborator
samskillman
left a comment
There was a problem hiding this comment.
Left some inline suggestions
…and delete duplicate filestore
8b5a78b to
de52e41
Compare
arpit974
approved these changes
Sep 24, 2025
All requested changes made.
8183d57
into
GoogleCloudPlatform:develop
13 of 64 checks passed
This file contains hidden or 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
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.
The PR makes the following changes to the H4D blueprint:
1> Disable automatic updates to prevent unattended updates leading to unstable clusters.
2> Explicitly mention the image used for nodeset, controller and login nodes within the blueprint instead of relying solely on the default in CTK.
3> Removes duplicate file store "appfs". There's no benefit of having two separate file store instances in the base blueprint. There are probably use cases of using separate file stores for specific apps but is not a general practice and hence needs to be removed.
Submission Checklist
NOTE: Community submissions can take up to 2 weeks to be reviewed.
Please take the following actions before submitting this pull request.