{"action":"closed","issue":{"url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/403","labels_url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/403/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/403/comments","events_url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/403/events","html_url":"https://github.com/ansible/ansible-modules-extras/issues/403","id":68730359,"number":403,"title":"gluster_volume should support multiple bricks","user":{"login":"rnowling","id":1114888,"avatar_url":"https://avatars.githubusercontent.com/u/1114888?v=3","gravatar_id":"","url":"https://api.github.com/users/rnowling","html_url":"https://github.com/rnowling","followers_url":"https://api.github.com/users/rnowling/followers","following_url":"https://api.github.com/users/rnowling/following{/other_user}","gists_url":"https://api.github.com/users/rnowling/gists{/gist_id}","starred_url":"https://api.github.com/users/rnowling/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rnowling/subscriptions","organizations_url":"https://api.github.com/users/rnowling/orgs","repos_url":"https://api.github.com/users/rnowling/repos","events_url":"https://api.github.com/users/rnowling/events{/privacy}","received_events_url":"https://api.github.com/users/rnowling/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ansible/ansible-modules-extras/labels/feature_idea","name":"feature_idea","color":"006b75"},{"url":"https://api.github.com/repos/ansible/ansible-modules-extras/labels/P4","name":"P4","color":"bfe5bf"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-04-15T16:33:29Z","updated_at":"2015-04-20T14:40:36Z","closed_at":"2015-04-20T14:40:36Z","body":"Issue Type:\r\nFeature idea\r\n\r\nAnsible Version:\r\n    ansible 2.0.0\r\n    configured module search path = None\r\n\r\nEnvironment:\r\nN/A\r\n\r\nSummary:\r\nThe `gluster_volume` module currently only supports one brick per volume. (In Gluster distributed file system terms, bricks are backing directories on the storage nodes.)  However, it is not uncommon to have multiple bricks per storage node (e.g., in cases where each brick is on a different disk.)  For example:\r\n\r\n    $ gluster volume create gv0 replica 2 server1:/data/brick1 server2:/data/brick1 server1:/data/brick2 server2:/data/brick2\r\n\r\nwill create a single volume `gv0` with two bricks, each of which is mirrored between the two servers.\r\n\r\nI want to add support for multiple bricks.  This will require changing the `gluster_volume` API.  Possible options include:\r\n\r\n* Modifying the `brick` keyword to take one or more bricks.\r\n* Adding a new `bricks` keyword alongside `brick` that can take 1 or more bricks.  Deprecate `brick` going forward.\r\n\r\nIn Ansible, which are preferable:  lists of strings or string with comma-separated values?\r\n\r\nThe order of the bricks is important since Gluster uses the order to group replicas.  For now, I suggest noting this in the documentation instead of using a more advanced data structure.\r\n\r\nSteps To Reproduce:\r\nJust look at code.\r\n\r\nExpected Results:\r\nBe able to pass multiple bricks into `gluster_volume`.\r\n\r\nActual Results:\r\nAssumes `brick` is a single item."}}