-
Notifications
You must be signed in to change notification settings - Fork 57
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 docs extra disks #159
Fix docs extra disks #159
Conversation
These can be defined using the [disk_attachment](#disk_attachment) block in the configuration. | ||
|
||
Note that this is an array, and therefore in HCL2 can be defined as multiple blocks, each | ||
one corresponding to a disk that will be attached to the instance you are booting. |
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.
Would it be helpful to add an example hcl2 block here?
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.
Done, I added one example with two disks defined in the template. Also to make sure it works well, I added an acceptance test for this case.
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.
This looks good to me. I left a question about adding an example. There looks to be a number of configuration attributes. So I'm not sure if an example would help or add confusion for a new user.
739d208
to
26c56d6
Compare
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.
Looks good. Love the tests.
When the
disk_attachment
block was introduced to the builder, the docs weren't updated, making the feature essentially invisible for users.This commit adds a section to document those, and links for easier navigation in the docs.