-
Notifications
You must be signed in to change notification settings - Fork 260
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
Ansible: gluster-brick-create switch from community modules to commands #46
Conversation
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 OK :)
| cache_lvname | yes | | | Name of the Logical Volume to be used for cache. | ||
| cache_lvsize | yes | | | Size of the cache logical volume | ||
| mntpath | yes | | | Path to mount the filesystem. | ||
| pool_metadatasize | yes | | | Size of the pools metadata, should be between 2M to 16G (Example: 24M). |
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.
Just make sure it' shouldn't be 2MiB
& 16GiB
.
Verified the patch on my setup working as expected. |
/ost |
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.
+1
/ost |
This is a minor change to the gluster-brick-create ansible role. The main change is that we switched from community modules which are no longer supported to the commands. We need this change for possible ansible-core support.
Along with the changes, I have bit improved the readme and ansible lint for better readability.
I was able to test these changes on my server with multiple reruns and looked fine, but I was not able to test it with gluster.
Bug-Url: https://bugzilla.redhat.com/2052690