Default values are already defined in both the blueprints. I don't think these changes are need. Please add reasons why this change is required.

It's needed because in the public docs we have mentioned that the user can replace these variables to match the specific values in the deployement.yaml file and since we are adding the disk size vars in the public docs they will be actually needed in the deployment file.

Default values are already defined in both the blueprints. I don't think these changes are need. Please add reasons why this change is required.

It's needed because in the public docs we have mentioned that the user can replace these variables to match the specific values in the deployement.yaml file and since we are adding the disk size vars in the public docs they will be actually needed in the deployment file.

Thanks. Could you share a link to the docs if they are public?

Default values are already defined in both the blueprints. I don't think these changes are need. Please add reasons why this change is required.

It's needed because in the public docs we have mentioned that the user can replace these variables to match the specific values in the deployement.yaml file and since we are adding the disk size vars in the public docs they will be actually needed in the deployment file.

Thanks. Could you share a link to the docs if they are public?

https://cloud.google.com/ai-hypercomputer/docs/create/gke-ai-hypercompute#:~:text=In%20the%20examples/gke%2Da4%2Dhighgpu/gke%2Da4%2Dhighgpu%2Ddeployment.yaml%20file%2C%20replace%20the%20following%20variables%20in%20the%20terraform_backend_defaults%20and%20vars%20sections%20to%20match%20the%20specific%20values%20for%20your%20deployment%3A

A comment above those variables describing what they are for can help, just like how the other variables have comments.