Skip to content
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

Add missing mount_options parameter in filesystem #80

Merged

Conversation

X-Cli
Copy link

@X-Cli X-Cli commented May 15, 2024

Closes #79

There was some mixup between the "mount" option from Ignition v2 and the "options" and "mountOptions" from Ignition v3.1+

This commit fixes the names of the related functions, adds mountOptions support and updates the documentation accordingly.

This is my very first PR on a terraform provider, so there might be some mistakes or oversights. Please do not hesitate to tell me how to improve it :)

Closes community-terraform-providers#79

There was some mixup between the "mount" option from Ignition v2 and
the "options" and "mountOptions" from Ignition v3.1+

This commit fixes the names of the related functions, adds mountOptions
support and updates the documentation accordingly.

Signed-off-by: Florian Maury <[email protected]>
@yussufsh yussufsh merged commit 1bb816a into community-terraform-providers:v2 May 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Mix up between filesystem options and mountOptions
2 participants