Repository navigation
fix: add g7e, p6-b200 and trn2 to volume size unsupported instance families - #6505
Conversation
…milies --- X-AI-Prompt: Add g7e, p6-b200, and trn2 to volume size unsupported instances X-AI-Tool: Claude
There was a problem hiding this comment.
🤖 AI code review
This PR adds g7e, p6-b200, and trn2 to the unsupported_families list in volume_size_supported(), so the SDK no longer passes volume size for these NVMe-backed instances. The change is minimal, consistent with the pattern established in PR #4828 (Source: #4828), updates the accompanying comment, and includes a parametrized unit test covering the three new families. Looks clean; no issues found.
Reviewed commit e4f161c. Automated review; verify before acting.
…nit tests --- X-AI-Prompt: Run formatters and fix issues X-AI-Tool: Claude
There was a problem hiding this comment.
🤖 AI code review
This PR adds g7e, p6-b200, and trn2 to the list of instance families that don't support EBS volume attachment in volume_size_supported(), fixing JumpStart deployment failures for NVMe-backed instances. The change follows the pattern established in PR #4828 (Source: #4828) and includes a parametrized test covering the new families. The remaining diff hunks are black reformatting of unrelated test files. The change is correct and adequately tested; no issues found.
Reviewed commit af314ab. Automated review; verify before acting.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.