)]}'
{"id":"openstack%2Fnova~22431","triplet_id":"openstack%2Fnova~master~I80de4c09ba857d550734f9bd0dfc0c40bfd5036a","project":"openstack/nova","branch":"master","topic":"bug/1130718","hashtags":[],"change_id":"I80de4c09ba857d550734f9bd0dfc0c40bfd5036a","subject":"Allow exit code 21 for \u0027iscsiadm -m session\u0027.","status":"MERGED","created":"2013-02-20 15:22:48.000000000","updated":"2013-02-20 16:26:28.000000000","submitted":"2013-02-20 16:26:26.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d5ac3248df7c1d714ba3dac4ebc9438343bf1cec","_number":22431,"virtual_id_number":22431,"owner":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":2,"date":"2013-02-20 16:26:26.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"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":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"all":[{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":2,"date":"2013-02-20 15:41:45.000000000","_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"date":"2013-02-20 15:25:27.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"value":2,"date":"2013-02-20 15:41:04.000000000","_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"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":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"all":[{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":1,"date":"2013-02-20 15:41:54.000000000","_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"date":"2013-02-20 15:24:19.000000000","_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"value":0,"date":"2013-02-20 15:41:04.000000000","_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"date":"2013-02-20 15:25:53.000000000","_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"_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-02-20 15:23:01.000000000","updated_by":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"reviewer":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"state":"REVIEWER"},{"updated":"2013-02-20 15:23:10.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-02-20 15:25:53.000000000","updated_by":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"reviewer":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"state":"REVIEWER"},{"updated":"2013-02-20 15:41:04.000000000","updated_by":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"reviewer":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"state":"REVIEWER"},{"updated":"2013-02-20 15:41:54.000000000","updated_by":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"reviewer":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"state":"REVIEWER"},{"updated":"2013-02-20 16:26:26.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"d0a587e1b6a354346d8a7244d1ef87d0b17f3522","author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"date":"2013-02-20 15:24:19.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"f619f7d801e65c89a302af5d3254e542e3fe9623","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-02-20 15:24:35.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":2},{"id":"34e76e13631993be6d6fbe61d77cca52e464fc9c","author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"date":"2013-02-20 15:25:27.000000000","message":"Patch Set 2:\n\nHey guys: I believe should fix SmokeStack. :)","accounts_in_message":[],"_revision_number":2},{"id":"27c7f7395a3e704e64f493f7dfe9df517a22bcf4","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2013-02-20 15:41:04.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"01ded29542ad33a7c51a972aa7e83a7ba463e1a6","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-02-20 15:41:45.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nWe do this in 6 other places, obviously the right thing","accounts_in_message":[],"_revision_number":2},{"id":"ba18e0a8bfb12df36569845bda74614dbcaa8bcf","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-02-20 15:41:54.000000000","message":"Patch Set 2: Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"7f198f3d6066306cd9bee54ea5fd1ef5a1737faf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 15:41:56.000000000","message":"Patch Set 2:\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"10429aa9f01117c679761eb4bbad7c99f3a87b34","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 16:08:12.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/22431/2/check/gate-nova-pep8/21113 : SUCCESS\n- http://logs.openstack.org/22431/2/check/gate-nova-docs/15685 : SUCCESS\n- http://logs.openstack.org/22431/2/check/gate-nova-python26/17130 : SUCCESS\n- http://logs.openstack.org/22431/2/check/gate-nova-python27/17185 : SUCCESS\n- http://logs.openstack.org/22431/2/check/gate-tempest-devstack-vm-full/4515 : SUCCESS\n- http://logs.openstack.org/22431/2/check/gate-tempest-devstack-vm-postgres-full/4234 : SUCCESS\n- http://logs.openstack.org/22431/2/check/gate-tempest-devstack-vm-quantum/9043 : SUCCESS\n- http://logs.openstack.org/22431/2/check/gate-nova-pylint/8610 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"ae5a2d43d798809d8529d12bd4b89b0876282ac7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 16:26:26.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/22431/2/gate/gate-nova-docs/15686 : SUCCESS\n- http://logs.openstack.org/22431/2/gate/gate-nova-pep8/21115 : SUCCESS\n- http://logs.openstack.org/22431/2/gate/gate-nova-python26/17131 : SUCCESS\n- http://logs.openstack.org/22431/2/gate/gate-nova-python27/17186 : SUCCESS\n- http://logs.openstack.org/22431/2/gate/gate-tempest-devstack-vm-full/4516 : SUCCESS\n- http://logs.openstack.org/22431/2/gate/gate-tempest-devstack-vm-postgres-full/4235 : SUCCESS\n- http://logs.openstack.org/22431/2/gate/gate-tempest-devstack-vm-quantum/9044 : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"24040a64b51c3ee9e558870830a084e98a044cd9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 16:26:28.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"13eee3d1d7b40bc334d663b7cea7c65a8f960dfe","revisions":{"a61863ddc669331bc4630eb21555f990e9450ff6":{"kind":"REWORK","_number":1,"created":"2013-02-20 15:22:48.000000000","uploader":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"ref":"refs/changes/31/22431/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/31/22431/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/31/22431/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/31/22431/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/31/22431/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/31/22431/1"}}},"commit":{"parents":[{"commit":"0d33d2a3820cad7484c5f5eaa9aa41a325965c62","subject":"Merge \"Fix broken logging imports.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0d33d2a3820cad7484c5f5eaa9aa41a325965c62"}]}],"author":{"name":"Dan Prince","email":"dprince@redhat.com","date":"2013-02-20 15:20:03.000000000","tz":-300},"committer":{"name":"Dan Prince","email":"dprince@redhat.com","date":"2013-02-20 15:20:03.000000000","tz":-300},"subject":"Allow exit code 21 for \u0027iscsiadm -m session\u0027.","message":"Allow exit code 21 for \u0027iscsiadm -m session\u0027.\n\nUpdates the libvirt volume driver so that it properly handles\nexit code 21 in the active session check. This fixes issues\nin running the command on Fedora 17.\n\nFixes LP Bug #1130718.\n\nChange-Id: I80de4c09ba857d550734f9bd0dfc0c40bfd5036a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a61863ddc669331bc4630eb21555f990e9450ff6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a61863ddc669331bc4630eb21555f990e9450ff6"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"0d33d2a3820cad7484c5f5eaa9aa41a325965c62","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"13eee3d1d7b40bc334d663b7cea7c65a8f960dfe":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-02-20 15:24:19.000000000","uploader":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"ref":"refs/changes/31/22431/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/31/22431/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/31/22431/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/31/22431/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/31/22431/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/31/22431/2"}}},"commit":{"parents":[{"commit":"0d33d2a3820cad7484c5f5eaa9aa41a325965c62","subject":"Merge \"Fix broken logging imports.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0d33d2a3820cad7484c5f5eaa9aa41a325965c62"}]}],"author":{"name":"Dan Prince","email":"dprince@redhat.com","date":"2013-02-20 15:20:03.000000000","tz":-300},"committer":{"name":"Dan Prince","email":"dprince@redhat.com","date":"2013-02-20 15:23:29.000000000","tz":-300},"subject":"Allow exit code 21 for \u0027iscsiadm -m session\u0027.","message":"Allow exit code 21 for \u0027iscsiadm -m session\u0027.\n\nUpdates the libvirt volume driver so that it properly handles\nexit code 21 in the active session check. This fixes issues\nin running the command on Fedora 17.\n\nThis behaviour was broken in 933365a (multipath libvirt iscsi).\n\nFixes LP Bug #1130718.\n\nChange-Id: I80de4c09ba857d550734f9bd0dfc0c40bfd5036a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/13eee3d1d7b40bc334d663b7cea7c65a8f960dfe"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/13eee3d1d7b40bc334d663b7cea7c65a8f960dfe"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"0d33d2a3820cad7484c5f5eaa9aa41a325965c62","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}