Skip to content

expose boot disk size for VAI configs#454

Merged
mkolodner-sc merged 1 commit intomainfrom
zfan3/config
Jan 22, 2026
Merged

expose boot disk size for VAI configs#454
mkolodner-sc merged 1 commit intomainfrom
zfan3/config

Conversation

@zfan3-sc
Copy link
Copy Markdown
Collaborator

@zfan3-sc zfan3-sc commented Jan 21, 2026

Scope of work done
Expose boot_disk_size_gb for VertexAI job configs.
ran make compile_protos

Where is the documentation for this feature?: N/A

Did you add automated tests or write a test plan?

Updated Changelog.md? NO

Ready for code review?: NO

Co-authored-by: kmontemayor <kyle.e.montemayor@gmail.com>
Copy link
Copy Markdown
Collaborator

@kmontemayor2-sc kmontemayor2-sc left a comment

Choose a reason for hiding this comment

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

Thanks!

@kmontemayor2-sc
Copy link
Copy Markdown
Collaborator

nit. maybe a different PR title, I was very confused why you were sending some pr about the gs examples :P

@zfan3-sc zfan3-sc changed the title Setup e2e integration test for graph store homogeneous inference (#433) expose boot disk size for VAI configs Jan 21, 2026
Copy link
Copy Markdown
Collaborator

@mkolodner-sc mkolodner-sc left a comment

Choose a reason for hiding this comment

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

Thank Zihao! May want to update the PR name but otherwise LGTM

@zfan3-sc
Copy link
Copy Markdown
Collaborator Author

Yep PR title updated

@zfan3-sc zfan3-sc added this pull request to the merge queue Jan 21, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jan 21, 2026
@mkolodner-sc mkolodner-sc added this pull request to the merge queue Jan 21, 2026
Merged via the queue into main with commit 7c5ce20 Jan 22, 2026
6 checks passed
@mkolodner-sc mkolodner-sc deleted the zfan3/config branch January 22, 2026 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants