-
-
Notifications
You must be signed in to change notification settings - Fork 232
fix(vm): increase max disk limit to 31 #2302
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(vm): increase max disk limit to 31 #2302
Conversation
Signed-off-by: Justas Umbrasas <justas.umbrasas@hostinger.com>
Summary of ChangesHello @JustasUmbrasas, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request resolves a discrepancy between the Terraform Proxmox provider and the Proxmox VE platform regarding the maximum number of attachable disk devices. Previously, the provider limited disk devices to 14, while Proxmox VE supports up to 31. This update adjusts the provider's schema to reflect the higher limit, enabling users to configure more disks per virtual machine. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request increases the maximum number of disks for a VM from 14 to 31, which aligns with the limit for SCSI devices in Proxmox VE. This is a valuable improvement that increases the provider's capabilities. The change is straightforward and correct. I have one minor suggestion to improve code maintainability.
Signed-off-by: Justas Umbrasas <justas.umbrasas@hostinger.com>
8ef221d to
fd15cdb
Compare
bpg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix @JustasUmbrasas!
LGTM! 🚀
|
@all-contributors please add @JustasUmbrasas for code |
|
I've put up a pull request to add @JustasUmbrasas! 🎉 |
Contributor's Note
/docsfor any user-facing features or additions./fwprovider/testsfor any new or updated resources / data sources.make exampleto verify that the change works as expected.Proof of Work
We encountered an issue where the provider prevents adding more than 14 disk devices, while Proxmox VE currently supports up to 31 SCSI devices. This change increases the maximum disk limit from 14 to 31 to align with Proxmox VE's capabilities.
Community Note
Closes #0000 | Relates #0000