)]}'
{"id":"openstack%2Fdevstack~26640","triplet_id":"openstack%2Fdevstack~master~Iaa105233c9fce8d8fda0a9ea447e045b8d010db7","project":"openstack/devstack","branch":"master","topic":"glusterfs","hashtags":[],"change_id":"Iaa105233c9fce8d8fda0a9ea447e045b8d010db7","subject":"Add support for GlusterFS Cinder driver","status":"MERGED","created":"2013-04-10 17:10:03.000000000","updated":"2013-04-23 18:07:20.000000000","submitted":"2013-04-23 18:07:20.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"19eb161e23d812719a1b7246802896540ef72060","_number":26640,"virtual_id_number":26640,"owner":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-04-23 18:07:20.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":2,"date":"2013-04-23 15:51:46.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-04-22 21:17:38.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":2,"date":"2013-04-22 21:24:28.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":1,"date":"2013-04-23 15:51:46.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-04-22 21:16:40.000000000","_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"date":"2013-04-22 21:24:28.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-04-10 18:41:52.000000000","updated_by":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"reviewer":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"state":"REVIEWER"},{"updated":"2013-04-22 21:24:28.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-04-23 15:51:46.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-04-23 18:07:20.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"1e0dcadb45a1b8687de290e67973b65f04b8ca42","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-04-10 17:33:07.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nSo it\u0027s not that I don\u0027t want to make it easy to test drivers with devstack, I just don\u0027t like doing it this way.  I think it can get out of control rather quickly in terms of lib/cinder getting very messy.\n\nI think we can do this in a more generic fashion, where localrc variable is used for *whatever* driver instead of hard-coding all of them in the cinder file.\n\nI\u0027ll let the devstack pros determine if I\u0027m off base here or not, but that\u0027s my opinion.","accounts_in_message":[],"_revision_number":1},{"id":"d69b80c0f3a1e60d07a37fb2f75b0cc5e444f39a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-10 17:53:04.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/26640/1/check/gate-tempest-devstack-vm-full/12223 : SUCCESS\n- http://logs.openstack.org/26640/1/check/gate-tempest-devstack-vm-postgres-full/11938 : SUCCESS\n- http://logs.openstack.org/26640/1/check/gate-tempest-devstack-vm-quantum/18034 : SUCCESS\n- http://logs.openstack.org/26640/1/check/gate-grenade-devstack-vm/243 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"3030776a02c0e4e9fe1678d7d79d6836985b0939","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-04-10 18:41:52.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nIt seems that my thought on this is a bit out of line, particularly since as Eric pointed out other drivers (sheepdog and xennfs) were recently added.\n\nSorry Eric... my philosophies on this are apparently not quite aligned.","accounts_in_message":[],"_revision_number":1},{"id":"245636f8426bb8081d1014100d30980da5a8b112","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-04-22 20:44:53.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nsomewhere in the devstack script you need to document what the expected config is for gluster, otherwise it\u0027s not really useful to people.","accounts_in_message":[],"_revision_number":1},{"id":"f4719accabff0d51d41abb40954f3966697328e1","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-04-22 20:45:39.000000000","message":"Patch Set 1: (1 inline comment)\n\nadded the specific location to comment","accounts_in_message":[],"_revision_number":1},{"id":"7061cac13e00e2cd78a9772044e244c86e413748","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-04-22 21:16:40.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"32ac1ea66e2c16b84d7f463eee8068df4c5e1b3a","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-04-22 21:17:38.000000000","message":"Patch Set 2:\n\nThanks Sean, added.","accounts_in_message":[],"_revision_number":2},{"id":"e1340615479283f3fe6800bae1e52d84740865fb","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-04-22 21:24:28.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"05503949eec2275fd0ddaa05e9ddf7983f62ac55","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-22 22:02:54.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/26640/2/check/gate-tempest-devstack-vm-full/13233 : SUCCESS\n- http://logs.openstack.org/26640/2/check/gate-tempest-devstack-vm-postgres-full/12948 : SUCCESS\n- http://logs.openstack.org/26640/2/check/gate-tempest-devstack-vm-quantum/19151 : SUCCESS\n- http://logs.openstack.org/26640/2/check/gate-grenade-devstack-vm/1124 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"95ff17fc38656a060d286166cdfc59424b06331e","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-04-23 15:51:46.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"e9ff48716a56f323e64887cf6bb01d4525166cf2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-23 15:51:53.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"f559141b30c7343f040c520c28de97db36c4d5e2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-23 18:07:20.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2},{"id":"56fe9da2613a752745d3dcef1e69f53df808c87f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-23 18:07:20.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/26640/2/gate/gate-tempest-devstack-vm-full/13347 : SUCCESS\n- http://logs.openstack.org/26640/2/gate/gate-tempest-devstack-vm-postgres-full/13062 : SUCCESS\n- http://logs.openstack.org/26640/2/gate/gate-tempest-devstack-vm-quantum/19285 : SUCCESS\n- http://logs.openstack.org/26640/2/gate/gate-grenade-devstack-vm/1225 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"cd26151261425ab81e678a6f6b16011ab4ca8b2f","revisions":{"50b05e5c7efc9aecd4786d186a98f24b36c94ba2":{"kind":"REWORK","_number":1,"created":"2013-04-10 17:10:03.000000000","uploader":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"ref":"refs/changes/40/26640/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/40/26640/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/26640/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/26640/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/26640/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/40/26640/1"}}},"commit":{"parents":[{"commit":"a1c183b4effc8a9859ed5529577f209bbd9bb880","subject":"Merge \"Add err()/err_if_not_set()\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a1c183b4effc8a9859ed5529577f209bbd9bb880"}]}],"author":{"name":"Eric Harney","email":"eharney@redhat.com","date":"2013-04-10 16:48:09.000000000","tz":-240},"committer":{"name":"Eric Harney","email":"eharney@redhat.com","date":"2013-04-10 17:08:25.000000000","tz":-240},"subject":"Add support for GlusterFS Cinder driver","message":"Add support for GlusterFS Cinder driver\n\nThe GlusterFS volume driver can be used with Cinder by setting the\nfollowing in localrc:\n\nCINDER_DRIVER\u003dglusterfs\nCINDER_GLUSTERFS_SHARES\u003d\"127.0.0.1:/vol1;127.0.0.1:/vol2\"\n\nShares are \u003chost\u003e:\u003cvolume\u003e and separated by semicolons.\n\nChange-Id: Iaa105233c9fce8d8fda0a9ea447e045b8d010db7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/50b05e5c7efc9aecd4786d186a98f24b36c94ba2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/50b05e5c7efc9aecd4786d186a98f24b36c94ba2"}]},"branch":"refs/heads/master"},"cd26151261425ab81e678a6f6b16011ab4ca8b2f":{"kind":"REWORK","_number":2,"created":"2013-04-22 21:16:40.000000000","uploader":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"ref":"refs/changes/40/26640/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/40/26640/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/26640/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/26640/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/26640/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/40/26640/2"}}},"commit":{"parents":[{"commit":"a1c183b4effc8a9859ed5529577f209bbd9bb880","subject":"Merge \"Add err()/err_if_not_set()\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a1c183b4effc8a9859ed5529577f209bbd9bb880"}]}],"author":{"name":"Eric Harney","email":"eharney@redhat.com","date":"2013-04-10 16:48:09.000000000","tz":-240},"committer":{"name":"Eric Harney","email":"eharney@redhat.com","date":"2013-04-22 21:16:21.000000000","tz":-240},"subject":"Add support for GlusterFS Cinder driver","message":"Add support for GlusterFS Cinder driver\n\nThe GlusterFS volume driver can be used with Cinder by setting the\nfollowing in localrc:\n\nCINDER_DRIVER\u003dglusterfs\nCINDER_GLUSTERFS_SHARES\u003d\"127.0.0.1:/vol1;127.0.0.1:/vol2\"\n\nShares are \u003chost\u003e:\u003cvolume\u003e and separated by semicolons.\n\nChange-Id: Iaa105233c9fce8d8fda0a9ea447e045b8d010db7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/cd26151261425ab81e678a6f6b16011ab4ca8b2f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/cd26151261425ab81e678a6f6b16011ab4ca8b2f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}