Description of problem: If volume creation fails, glusterd reports bricks to be part of volume. Version-Release number of selected component (if applicable): How reproducible: Always Steps to Reproduce: 1.Fail to create a volume for some reason. 2 Trigger the create command again with same bricks. Actual results: [root@dhcp42-109 glusterfs]# gluster v info No volumes present [root@dhcp42-109 glusterfs]# gluster v create test-vol1 replica 3 10.70.42.109:/home/gluster/b8 10.70.42.109:/home/gluster/b7 10.70.42.109:/home/gluster/b6 volume create: test-vol1: failed: Multiple bricks of a replicate volume are present on the same server. This setup is not optimal. Bricks should be on different nodes to have best fault tolerant configuration. Use 'force' at the end of the command if you want to override this behavior. [root@dhcp42-109 glusterfs]# gluster v create test-vol1 replica 3 10.70.42.109:/home/gluster/b8 10.70.42.109:/home/gluster/b7 10.70.42.109:/home/gluster/b6 volume create: test-vol1: failed: /home/gluster/b8 is already part of a volume [root@dhcp42-109 glusterfs]# gluster v info No volumes present Expected results: If no volume is created, the brick should be available Additional info:
Hi Sheetal, Every cli transaction goes through four phases: locking, staging, commit, unlock. We do all kinds of validations in staging phase of transaction, once the validation fails we error out. Any modifications done in regards to the current transaction, will not be reverted as we don't have roll-back mechanism in glusterd architecture. When volume create operation is issued, we set xattrs on the bricks. But, if the transaction fails, the xattrs will remain on the bricks. This causes subsequent volume create transactions to fail if the force option is not used. I will send out a patch to fix this. Thanks, Sanju
REVIEW: https://review.gluster.org/23760 (glusterd: set xaatrs afer checking the brick order) posted (#1) for review on master by Sanju Rakonde
REVISION POSTED: https://review.gluster.org/23760 (glusterd: set xaatrs after checking the brick order) posted (#3) for review on master by Sanju Rakonde
The patch merged more than a month ago, not sure why the bot didn't close the bug.