gluster / gluster-ansible-infra

Ansible role to enable and deploy the backend of a Gluster cluster.
GNU General Public License v3.0
28 stars 36 forks source link

backend setup is not idempotent. #51

Closed kristvanbesien closed 5 years ago

kristvanbesien commented 5 years ago

The backend setup process is not idempotent.

How to reproduce:

Expected result:

Actual result:

Running the playbook with --skip-tages=fscreate is possible as a workaround, but is not optimal.

sac commented 5 years ago

The backend setup process is not idempotent.

Thanks for reporting this. I was able to debug this issue. The `force' by default in: https://github.com/gluster/gluster-ansible-infra/blob/master/roles/backend_setup/tasks/fscreate.yml#L25 was causing the issue.

It was forcing the module to re-run the mkfs.xfs whether filesystem was already present or not.

PR: https://github.com/gluster/gluster-ansible-infra/pull/52/files fixes the issue.

sac commented 5 years ago

@kristvanbesien request to review https://github.com/gluster/gluster-ansible-infra/pull/52 if you have some time.

sac commented 5 years ago

PR: https://github.com/gluster/gluster-ansible-infra/pull/52 resolves this issue.