)]}'
{"id":"openstack%2Fdevstack~109714","triplet_id":"openstack%2Fdevstack~master~Ib35ffc09f6582354010ac8ca35da581ea508877a","project":"openstack/devstack","branch":"master","topic":"bug/1348786","hashtags":[],"change_id":"Ib35ffc09f6582354010ac8ca35da581ea508877a","subject":"Fix cinder volume type creation and rename variable","status":"MERGED","created":"2014-07-25 20:37:41.000000000","updated":"2014-07-31 06:09:13.000000000","submitted":"2014-07-31 06:09:13.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":6,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"5c8a33e55bfb0608f995d30c68928d87062658af","_number":109714,"virtual_id_number":109714,"owner":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"recommended":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"all":[{"value":1,"date":"2014-07-25 23:12:09.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":2,"date":"2014-07-31 06:09:13.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":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":0,"date":"2014-07-27 16:05:48.000000000","_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":4159,"name":"Cory Stone","email":"corystone@gmail.com","username":"corystone"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"all":[{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2014-07-25 22:31:15.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":1,"date":"2014-07-28 17:30:50.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":2,"date":"2014-07-25 21:51:18.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2014-07-26 19:46:51.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":2,"date":"2014-07-30 21:03:28.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":4159,"name":"Cory Stone","email":"corystone@gmail.com","username":"corystone"}],"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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"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":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":1,"date":"2014-07-30 21:03:28.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":4159,"name":"Cory Stone","email":"corystone@gmail.com","username":"corystone"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"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":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":4159,"name":"Cory Stone","email":"corystone@gmail.com","username":"corystone"}],"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":4159,"name":"Cory Stone","email":"corystone@gmail.com","username":"corystone"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-07-25 21:06:06.000000000","updated_by":{"_account_id":4159,"name":"Cory Stone","email":"corystone@gmail.com","username":"corystone"},"reviewer":{"_account_id":4159,"name":"Cory Stone","email":"corystone@gmail.com","username":"corystone"},"state":"REVIEWER"},{"updated":"2014-07-25 21:51:18.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":"2014-07-25 22:31:15.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":"2014-07-25 23:12:09.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2014-07-26 19:46:51.000000000","updated_by":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"reviewer":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"state":"REVIEWER"},{"updated":"2014-07-28 17:30:50.000000000","updated_by":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"reviewer":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"state":"REVIEWER"},{"updated":"2014-07-30 21:03: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":"2014-07-31 06:09:13.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"27ad4c70c01e9c1723389292633209d3ddc62711","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2014-07-25 20:37:41.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"5eb714111d0983e6f5a1e941e87d67f955e72255","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-07-25 20:37:42.000000000","message":"Patch Set 1:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"a815e0a7e4cb86768a7d07c231af127cb7e3580f","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2014-07-25 20:48:22.000000000","message":"Patch Set 1: Code-Review-1\n\nI\u0027d actually started on this but you can have it.\n\nThe problem here is that you don\u0027t want to use the type (unless you change some other things in the code).  The new model goes through and creates a type for each backend, so for example if you do:\n\nCINDER_ENABLED_BACKENDS\u003dlvm:lvmdriver-1,lvm:lvmdriver-2\n\nIt will get the first one, but fail because it will try and create another type for lvmdriver2 with the same name (lvm).","accounts_in_message":[],"_revision_number":1},{"id":"ebab2e2ddad0ab32f5d64fa9929710ae29576141","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2014-07-25 20:51:06.000000000","message":"Patch Set 1:\n\nie:\n\ndiff --git a/lib/cinder b/lib/cinder\nindex a51e4a0..449bde3 100644\n--- a/lib/cinder\n+++ b/lib/cinder\n@@ -472,6 +472,9 @@ function create_volume_types {\n             BE_NAME\u003d${be##*:}\n             if type configure_cinder_backend_${BE_TYPE} \u003e/dev/null 2\u003e\u00261; then\n                 # openstack volume type create --property volume_backend_name\u003d\"${BE_TYPE}\" ${BE_NAME}\n+                # huh ^^ ?\n+\n+                # FIXME(jdg): Used type-id not name\n                 cinder type-create ${BE_NAME} \u0026\u0026 \\\n                     cinder type-key ${BE_NAME} set volume_backend_name\u003d\"${BE_NAME}\"\n             fi","accounts_in_message":[],"_revision_number":1},{"id":"a164d0c7abfff4cfaed264a2d0784cfc867cd188","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2014-07-25 20:51:51.000000000","message":"Patch Set 1:\n\narghhh wrong diff","accounts_in_message":[],"_revision_number":1},{"id":"8d80c17163c9d072a2a282a764d7c2ac9e047e22","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2014-07-25 20:52:56.000000000","message":"Patch Set 1:\n\ndiff --git a/lib/cinder b/lib/cinder\nindex 449bde3..61149ee 100644\n--- a/lib/cinder\n+++ b/lib/cinder\n@@ -246,7 +246,7 @@ function configure_cinder {\n                 configure_cinder_backend_${BE_TYPE} ${BE_NAME}\n             fi\n             if [[ -z \"$default_type\" ]]; then\n-                default_type\u003d$BE_TYPE\n+                default_type\u003d$BE_NAME\n             fi\n             enabled_backends+\u003d$BE_NAME,\n         done","accounts_in_message":[],"_revision_number":1},{"id":"12b5140e63b437268c5a76deb2706df938302720","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2014-07-25 20:56:44.000000000","message":"Patch Set 1:\n\nJohn, thanks, default_type\u003d$BE_NAME was my other thought originally, was hoping for cinder people to let me know if that was wrong or not.","accounts_in_message":[],"_revision_number":1},{"id":"4a4208f85916634992f735255eea1e7037c4df0f","author":{"_account_id":4159,"name":"Cory Stone","email":"corystone@gmail.com","username":"corystone"},"date":"2014-07-25 20:59:07.000000000","message":"Patch Set 1: Code-Review-1\n\nYes, BE_NAME is better and won\u0027t blow up. :)","accounts_in_message":[],"_revision_number":1},{"id":"1a1a61a4b5ba823933e0f4697773428859d8bc43","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2014-07-25 21:04:09.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"7c49af846a3895d304cfbc1bfed6c187e0eaa1fd","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-07-25 21:04:10.000000000","message":"Patch Set 2:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"adc3fb761044bfb96bd273ead944570b343eac19","author":{"_account_id":4159,"name":"Cory Stone","email":"corystone@gmail.com","username":"corystone"},"date":"2014-07-25 21:06:06.000000000","message":"Patch Set 2: Code-Review+1\n\nThat might work!","accounts_in_message":[],"_revision_number":2},{"id":"0477a79b86576d0b1ff55d981435eb04bd515fb2","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2014-07-25 21:12:52.000000000","message":"Patch Set 2: Code-Review+1\n\nThanks Matt!","accounts_in_message":[],"_revision_number":2},{"id":"35486a331e5065e0121d5be6933b7653cb946f83","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-07-25 21:21:52.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)\n\nApparently I mistakenly thought that \u0027type\u0027 and \u0027name\u0027 meant different things to Cinder.  I equated \u0027volume type\u0027 to the volume_backend_name as what users see is called \u0027volume type name\u0027...\n\nOne note that I think will make this a bit clearer","accounts_in_message":[],"_revision_number":2},{"id":"ec1e10eceed02ac75fd0f56589de2bc387217349","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2014-07-25 21:31:08.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"9f49bedcd3e6eb990d7192c004814df84f0cdd49","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2014-07-25 21:33:31.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"00db6248e2843f07d46a614463808bb235f9f607","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-07-25 21:41:36.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"fd0b7b35a3816b44bcdf99c285f2b1d4de6f9c66","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2014-07-25 21:47:15.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"b9ecf7c3c9ba6a1fff652bd6d5beabbb9cf1deaa","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-07-25 21:47:16.000000000","message":"Patch Set 3:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"56aea4593080b80af33c38e0089661fb7bc57c74","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2014-07-25 21:47:45.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"8340d04240547014b134afc17b76ea2a90857066","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-07-25 21:51:18.000000000","message":"Patch Set 3: Code-Review+2\n\nThanks for indulging me here.","accounts_in_message":[],"_revision_number":3},{"id":"a0f74f8202c1ca7b7ff7794867601d9e44392e14","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2014-07-25 22:31:15.000000000","message":"Patch Set 3: Code-Review+1\n\nNot that big a deal to me.... but comments inline anyway.","accounts_in_message":[],"_revision_number":3},{"id":"af89f303fa9def935c01d6e394d9b778522e703a","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2014-07-25 22:31:34.000000000","message":"Patch Set 2:\n\n(1 comment)\n\nOops... in patch set 2","accounts_in_message":[],"_revision_number":2},{"id":"1d806238f469c9446c7db6a45f211a9c23508ca6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-07-25 22:48:11.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded.\n\n- check-tempest-dsvm-full http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-full/e26e787 : SUCCESS in 38m 42s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-postgres-full/5390c05 : SUCCESS in 39m 49s\n- check-tempest-dsvm-neutron http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-neutron/34deeef : SUCCESS in 48m 03s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-neutron-heat-slow/0c8775b : SUCCESS in 25m 13s\n- check-grenade-dsvm http://logs.openstack.org/14/109714/3/check/check-grenade-dsvm/06fe7b2 : SUCCESS in 40m 13s\n- gate-devstack-bashate http://logs.openstack.org/14/109714/3/check/gate-devstack-bashate/739b62e : SUCCESS in 33s\n- check-devstack-dsvm-cells http://logs.openstack.org/14/109714/3/check/check-devstack-dsvm-cells/caf8636 : SUCCESS in 16m 55s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-neutron-full/00d39eb : SUCCESS in 46m 11s (non-voting)\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/14/109714/3/check/gate-tempest-dsvm-large-ops/2ac0882 : SUCCESS in 20m 25s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/14/109714/3/check/gate-tempest-dsvm-neutron-large-ops/7b75c69 : SUCCESS in 24m 23s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/14/109714/3/check/check-grenade-dsvm-partial-ncpu/b1e0bac : SUCCESS in 33m 48s\n- check-neutron-dsvm-functional http://logs.openstack.org/14/109714/3/check/check-neutron-dsvm-functional/cc4211e : SUCCESS in 17m 40s\n- check-swift-dsvm-functional http://logs.openstack.org/14/109714/3/check/check-swift-dsvm-functional/7c8e9e5 : SUCCESS in 20m 19s\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-ironic-nv/1a817f7 : SUCCESS in 31m 46s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-ironic-postgres-nv/a1ef9cc : SUCCESS in 34m 54s (non-voting)\n- check-tempest-dsvm-virtual-ironic-nv http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-virtual-ironic-nv/3cf8c02 : SUCCESS in 37m 19s (non-voting)\n- check-tempest-dsvm-f20 http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-f20/b4f2b12 : SUCCESS in 1h 00m 46s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"6a05c4ca11e5345315b192595437ab2cc0aac4a4","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2014-07-25 23:12:09.000000000","message":"Patch Set 3: Verified+1\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/14/109714/3/19348/results.html\n\nStandard recheck supported; use \"recheck xenserver\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":3},{"id":"ccf6209d0161d2a1d7f0f734399ebda885d1bc67","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2014-07-26 19:46:51.000000000","message":"Patch Set 3: Code-Review+1\n\nThanks Matt.","accounts_in_message":[],"_revision_number":3},{"id":"066d1442a5520ddd48fab279a56121526cfcb23d","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2014-07-27 16:05:48.000000000","message":"Patch Set 3:\n\nThe VolumeTypeNotFoundByName trace is gone:\n\nhttp://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-full/e26e787/logs/screen-c-api.txt.gz?level\u003dTRACE","accounts_in_message":[],"_revision_number":3},{"id":"19596b459530116e5eb2723dee0011161a090b74","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2014-07-28 17:30:50.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"ab84e31be1e54b7c6796d78137dd69f0bc1fcb26","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2014-07-30 21:03:28.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"97b59f2c2ee2e73fdb614ff93ebca66f6cb17fb2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-07-30 21:03:31.000000000","message":"Patch Set 3: -Verified\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"a00c3da34a28748682bf246387fdc8b93437f0c9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-07-30 22:24:47.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded.\n\n- check-tempest-dsvm-full http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-full/3c0859a : SUCCESS in 41m 17s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-postgres-full/59cedc0 : SUCCESS in 45m 59s\n- check-tempest-dsvm-neutron http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-neutron/8435ae5 : SUCCESS in 56m 07s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-neutron-heat-slow/7766806 : SUCCESS in 26m 01s\n- check-grenade-dsvm http://logs.openstack.org/14/109714/3/check/check-grenade-dsvm/983ffd2 : SUCCESS in 41m 52s\n- gate-devstack-bashate http://logs.openstack.org/14/109714/3/check/gate-devstack-bashate/454fc29 : SUCCESS in 20s\n- check-devstack-dsvm-cells http://logs.openstack.org/14/109714/3/check/check-devstack-dsvm-cells/d70818a : SUCCESS in 15m 28s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-neutron-full/d88b1fa : SUCCESS in 59m 33s (non-voting)\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/14/109714/3/check/gate-tempest-dsvm-large-ops/fe27468 : SUCCESS in 23m 48s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/14/109714/3/check/gate-tempest-dsvm-neutron-large-ops/e12ca17 : SUCCESS in 24m 58s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/14/109714/3/check/check-grenade-dsvm-partial-ncpu/9481a31 : SUCCESS in 42m 16s\n- check-neutron-dsvm-functional http://logs.openstack.org/14/109714/3/check/check-neutron-dsvm-functional/1f4a301 : SUCCESS in 18m 19s\n- check-swift-dsvm-functional http://logs.openstack.org/14/109714/3/check/check-swift-dsvm-functional/27049de : SUCCESS in 19m 06s\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-ironic-nv/c6134ba : SUCCESS in 38m 05s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-ironic-postgres-nv/517cd1e : SUCCESS in 36m 57s (non-voting)\n- check-tempest-dsvm-virtual-ironic-nv http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-virtual-ironic-nv/499ee63 : SUCCESS in 31m 10s (non-voting)\n- check-tempest-dsvm-f20 http://logs.openstack.org/14/109714/3/check/check-tempest-dsvm-f20/a4bb89a : SUCCESS in 59m 14s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"b37923271f5341ed1c636f229dbb18951b6e0d52","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-07-30 22:24:50.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"71c1329a633c46655598f2a4166b1dc44dd7c633","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-07-31 06:09:13.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded.\n\n- gate-tempest-dsvm-full http://logs.openstack.org/14/109714/3/gate/gate-tempest-dsvm-full/9c0a3f9 : SUCCESS in 39m 45s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/14/109714/3/gate/gate-tempest-dsvm-postgres-full/b4fa2c7 : SUCCESS in 48m 36s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/14/109714/3/gate/gate-tempest-dsvm-neutron/5dd9a93 : SUCCESS in 49m 29s\n- gate-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/14/109714/3/gate/gate-tempest-dsvm-neutron-heat-slow/7605ecc : SUCCESS in 18m 52s\n- gate-grenade-dsvm http://logs.openstack.org/14/109714/3/gate/gate-grenade-dsvm/2a8a6a7 : SUCCESS in 32m 44s\n- gate-devstack-bashate http://logs.openstack.org/14/109714/3/gate/gate-devstack-bashate/38d5144 : SUCCESS in 17s\n- gate-devstack-dsvm-cells http://logs.openstack.org/14/109714/3/gate/gate-devstack-dsvm-cells/67e28ee : SUCCESS in 13m 31s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/14/109714/3/gate/gate-tempest-dsvm-large-ops/44f69a8 : SUCCESS in 19m 52s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/14/109714/3/gate/gate-tempest-dsvm-neutron-large-ops/e61765e : SUCCESS in 19m 32s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/14/109714/3/gate/gate-grenade-dsvm-partial-ncpu/19c21a9 : SUCCESS in 34m 18s\n- gate-neutron-dsvm-functional http://logs.openstack.org/14/109714/3/gate/gate-neutron-dsvm-functional/326f57f : SUCCESS in 15m 01s\n- gate-swift-dsvm-functional http://logs.openstack.org/14/109714/3/gate/gate-swift-dsvm-functional/cd25bcf : SUCCESS in 20m 12s","accounts_in_message":[],"_revision_number":3},{"id":"58c7a88c712987ba1711edb1e99b265b3a949544","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-07-31 06:09:13.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"6a4aa78fa3728f1b0a98850f432211da5c8caedf","revisions":{"64c7b9e9deac2fd6ca3142fe4a580ac17f8ba708":{"kind":"REWORK","_number":1,"created":"2014-07-25 20:37:41.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/14/109714/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/14/109714/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/14/109714/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/14/109714/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/14/109714/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/14/109714/1"}}},"commit":{"parents":[{"commit":"3348ca40f5c82250e8fd8022165ad544f4dcfff3","subject":"Merge \"debian: do not enbable grizzly repository from gplhost\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3348ca40f5c82250e8fd8022165ad544f4dcfff3"}]}],"author":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2014-07-25 20:35:53.000000000","tz":-420},"committer":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2014-07-25 20:35:53.000000000","tz":-420},"subject":"Fix cinder volume type creation","message":"Fix cinder volume type creation\n\nCommit 7062b12b9b91a1eb405d7079c62b62b0de59109f teased out\na bug where the default_volume_type is set to \u0027lvm\u0027 but\nthat is not the volume-type that\u0027s actually created, that\nis \u0027lvmdriver-1\u0027 in gate runs, so we see a ton of\nVolumeTypeNotFoundByName for the \u0027lvm\u0027 volume type in the\ncinder-api logs.\n\nChange-Id: Ib35ffc09f6582354010ac8ca35da581ea508877a\nCloses-Bug: #1348786\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/64c7b9e9deac2fd6ca3142fe4a580ac17f8ba708"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/64c7b9e9deac2fd6ca3142fe4a580ac17f8ba708"}]},"branch":"refs/heads/master"},"8beb0ed697f93457ce7ff67931d9320cf3e64e12":{"kind":"REWORK","_number":2,"created":"2014-07-25 21:04:09.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/14/109714/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/14/109714/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/14/109714/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/14/109714/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/14/109714/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/14/109714/2"}}},"commit":{"parents":[{"commit":"3348ca40f5c82250e8fd8022165ad544f4dcfff3","subject":"Merge \"debian: do not enbable grizzly repository from gplhost\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3348ca40f5c82250e8fd8022165ad544f4dcfff3"}]}],"author":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2014-07-25 20:35:53.000000000","tz":-420},"committer":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2014-07-25 21:02:53.000000000","tz":-420},"subject":"Fix cinder volume type creation","message":"Fix cinder volume type creation\n\nCommit 7062b12b9b91a1eb405d7079c62b62b0de59109f introduced\na bug where the default_volume_type is set to \u0027lvm\u0027 but\nthat is not the volume-type that\u0027s actually created, that\nis \u0027lvmdriver-1\u0027 in gate runs, so we see a ton of\nVolumeTypeNotFoundByName for the \u0027lvm\u0027 volume type in the\ncinder-api logs.\n\nThis simply sets the default_volume_type to the volume\ntype that\u0027s created in the create_volume_types function.\n\nChange-Id: Ib35ffc09f6582354010ac8ca35da581ea508877a\nCloses-Bug: #1348786\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/8beb0ed697f93457ce7ff67931d9320cf3e64e12"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/8beb0ed697f93457ce7ff67931d9320cf3e64e12"}]},"branch":"refs/heads/master"},"6a4aa78fa3728f1b0a98850f432211da5c8caedf":{"kind":"REWORK","_number":3,"created":"2014-07-25 21:47:15.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/14/109714/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/14/109714/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/14/109714/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/14/109714/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/14/109714/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/14/109714/3"}}},"commit":{"parents":[{"commit":"3348ca40f5c82250e8fd8022165ad544f4dcfff3","subject":"Merge \"debian: do not enbable grizzly repository from gplhost\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3348ca40f5c82250e8fd8022165ad544f4dcfff3"}]}],"author":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2014-07-25 20:35:53.000000000","tz":-420},"committer":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2014-07-25 21:46:12.000000000","tz":-420},"subject":"Fix cinder volume type creation and rename variable","message":"Fix cinder volume type creation and rename variable\n\nCommit 7062b12b9b91a1eb405d7079c62b62b0de59109f introduced\na bug where the default_volume_type is set to \u0027lvm\u0027 but\nthat is not the volume-type that\u0027s actually created, that\nis \u0027lvmdriver-1\u0027 in gate runs, so we see a ton of\nVolumeTypeNotFoundByName for the \u0027lvm\u0027 volume type in the\ncinder-api logs.\n\nThis simply sets the default_volume_type to the volume\ntype that\u0027s created in the create_volume_types function.\n\nAlso renames the default_type variable to default_name to\nbe consistent with variable naming in the rest of the\ncinder script.\n\nChange-Id: Ib35ffc09f6582354010ac8ca35da581ea508877a\nCloses-Bug: #1348786\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6a4aa78fa3728f1b0a98850f432211da5c8caedf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6a4aa78fa3728f1b0a98850f432211da5c8caedf"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
