)]}'
{"id":"openstack%2Fheat~60486","triplet_id":"openstack%2Fheat~stable%2Fhavana~I563db129e63817ad55165e318f164f06b141ee33","project":"openstack/heat","branch":"stable/havana","topic":"bug/1251757","hashtags":[],"change_id":"I563db129e63817ad55165e318f164f06b141ee33","subject":"Properly reconnect subscribing clients when QPID broker restarts","status":"MERGED","created":"2013-12-06 11:46:34.000000000","updated":"2013-12-08 00:11:59.000000000","submitted":"2013-12-08 00:11:59.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"7f0e5837dc6dd2ab43b39bb86e3cc150ce4c9d85","_number":60486,"virtual_id_number":60486,"owner":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},{"value":0,"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},{"value":0,"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"value":2,"date":"2013-12-08 00:11:59.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"recommended":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"all":[{"value":0,"date":"2013-12-06 13:15:25.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},{"value":1,"date":"2013-12-06 17:57:47.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},{"value":2,"date":"2013-12-07 23:33:35.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"all":[{"value":0,"date":"2013-12-06 13:15:25.000000000","_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},{"value":0,"date":"2013-12-06 11:46:34.000000000","_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},{"value":1,"date":"2013-12-07 23:33:35.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-12-06 11:46:34.000000000","updated_by":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"reviewer":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"state":"REVIEWER"},{"updated":"2013-12-07 23:33:35.000000000","updated_by":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"reviewer":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"state":"REVIEWER"},{"updated":"2013-12-08 00:11:59.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"3513a86462d6fecf24c22edf8bfdb6481b85b192","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-06 12:35:34.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-heat-pep8 http://logs.openstack.org/86/60486/1/check/gate-heat-pep8/e6535c3 : SUCCESS in 1m 44s\n- gate-heat-docs http://docs-draft.openstack.org/86/60486/1/check/gate-heat-docs/8021629/doc/build/html/ : SUCCESS in 2m 20s\n- gate-heat-python26 http://logs.openstack.org/86/60486/1/check/gate-heat-python26/63f2372 : SUCCESS in 2m 24s\n- gate-heat-python27 http://logs.openstack.org/86/60486/1/check/gate-heat-python27/6a13fcd : SUCCESS in 2m 13s\n- check-tempest-dsvm-full http://logs.openstack.org/86/60486/1/check/check-tempest-dsvm-full/bddabcb : SUCCESS in 47m 45s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/86/60486/1/check/check-tempest-dsvm-postgres-full/5faf960 : SUCCESS in 33m 59s\n- check-tempest-dsvm-neutron http://logs.openstack.org/86/60486/1/check/check-tempest-dsvm-neutron/a649f07 : FAILURE in 48m 53s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/60486/1/check/gate-tempest-dsvm-large-ops/773143d : SUCCESS in 11m 54s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/60486/1/check/gate-tempest-dsvm-neutron-large-ops/e796a35 : SUCCESS in 13m 51s\n","accounts_in_message":[],"_revision_number":1},{"id":"cbc426ea96edd47093b28a5c6a0fdd8bbdf22d26","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2013-12-06 12:36:26.000000000","message":"Patch Set 1:\n\nI noticed tempest failed, I think you hit bug(s):\n\n- https://bugs.launchpad.net/bugs/1253896\n\nWe don\u0027t automatically recheck or reverify, so please consider\ndoing that manually if someone hasn\u0027t already. For a code review\nwhich is not yet approved, you can recheck by leaving a code\nreview comment with just the text:\n\n    recheck bug 1253896\n\nFor a code review which has been approved but failed to merge,\nyou can reverify by leaving a comment like this:\n\n    reverify bug 1253896","accounts_in_message":[],"_revision_number":1},{"id":"86a4dcb0bda50e114db2af5b38d82cd8e96c9b06","author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"date":"2013-12-06 13:15:25.000000000","message":"Patch Set 1:\n\nrecheck bug 1253896","accounts_in_message":[],"_revision_number":1},{"id":"1f07ee03b94bede98ac00cc8e6aedb264119340f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-06 13:54:42.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-heat-pep8 http://logs.openstack.org/86/60486/1/check/gate-heat-pep8/ef177b2 : SUCCESS in 2m 08s\n- gate-heat-docs http://docs-draft.openstack.org/86/60486/1/check/gate-heat-docs/4ca9018/doc/build/html/ : SUCCESS in 2m 20s\n- gate-heat-python26 http://logs.openstack.org/86/60486/1/check/gate-heat-python26/1971e50 : SUCCESS in 2m 14s\n- gate-heat-python27 http://logs.openstack.org/86/60486/1/check/gate-heat-python27/d739c56 : SUCCESS in 1m 53s\n- check-tempest-dsvm-full http://logs.openstack.org/86/60486/1/check/check-tempest-dsvm-full/40ba41d : SUCCESS in 39m 09s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/86/60486/1/check/check-tempest-dsvm-postgres-full/097eebe : SUCCESS in 29m 48s\n- check-tempest-dsvm-neutron http://logs.openstack.org/86/60486/1/check/check-tempest-dsvm-neutron/8a313cc : SUCCESS in 29m 23s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/60486/1/check/gate-tempest-dsvm-large-ops/9add45c : SUCCESS in 11m 42s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/60486/1/check/gate-tempest-dsvm-neutron-large-ops/e62c673 : SUCCESS in 14m 50s\n","accounts_in_message":[],"_revision_number":1},{"id":"c98ae0501623bc47313220df4806445ffc86b42d","author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"date":"2013-12-06 17:57:47.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"cac7a10c8f6e885a5174b766cf52df85a9860dec","author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"date":"2013-12-07 23:33:35.000000000","message":"Patch Set 1: Looks good to me (core reviewer); Approved\n\n(1 inline comment)\n\nApproving as freeze exception for 2013.2.1","accounts_in_message":[],"_revision_number":1},{"id":"c66f63071734255f411db01dc55c1b74376b24ec","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-07 23:33:36.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"6d115403ca23898ee525ab0abbf5cc7cae21d09a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-08 00:11:59.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- gate-heat-pep8 http://logs.openstack.org/86/60486/1/gate/gate-heat-pep8/382eb40 : SUCCESS in 3m 39s\n- gate-heat-docs http://docs-draft.openstack.org/86/60486/1/gate/gate-heat-docs/796a0ad/doc/build/html/ : SUCCESS in 2m 22s\n- gate-heat-python26 http://logs.openstack.org/86/60486/1/gate/gate-heat-python26/5b44eb3 : SUCCESS in 2m 12s\n- gate-heat-python27 http://logs.openstack.org/86/60486/1/gate/gate-heat-python27/ccc181d : SUCCESS in 2m 11s\n- gate-tempest-dsvm-full http://logs.openstack.org/86/60486/1/gate/gate-tempest-dsvm-full/fae2f00 : SUCCESS in 38m 16s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/60486/1/gate/gate-tempest-dsvm-postgres-full/1850f2b : SUCCESS in 28m 12s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/86/60486/1/gate/gate-tempest-dsvm-neutron/3c8b8ed : SUCCESS in 28m 23s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/60486/1/gate/gate-tempest-dsvm-large-ops/e1c7bdb : SUCCESS in 12m 52s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/60486/1/gate/gate-tempest-dsvm-neutron-large-ops/0f33ab4 : SUCCESS in 19m 15s\n","accounts_in_message":[],"_revision_number":1},{"id":"9e2d9686d95b48983fed0b6527eca7e0868a77b5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-08 00:11:59.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"6ccbf6e7abbdb518599f943fb743e538b1f81708","revisions":{"6ccbf6e7abbdb518599f943fb743e538b1f81708":{"kind":"REWORK","_number":1,"created":"2013-12-06 11:46:34.000000000","uploader":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"ref":"refs/changes/86/60486/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/heat","ref":"refs/changes/86/60486/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/heat refs/changes/86/60486/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/heat refs/changes/86/60486/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/heat refs/changes/86/60486/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/heat refs/changes/86/60486/1"}}},"commit":{"parents":[{"commit":"a8c0b1107a62d5a145cf9ff58b85d9e3210fe95c","subject":"Use all available headroom for autoscaling","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/heat/commit/a8c0b1107a62d5a145cf9ff58b85d9e3210fe95c"}]}],"author":{"name":"Steven Dake","email":"sdake@redhat.com","date":"2013-12-05 18:34:38.000000000","tz":-420},"committer":{"name":"Flavio Percoco","email":"flaper87@gmail.com","date":"2013-12-06 11:46:31.000000000","tz":60},"subject":"Properly reconnect subscribing clients when QPID broker restarts","message":"Properly reconnect subscribing clients when QPID broker restarts\n\nThis is a cherrypick of oslo-incubator:\ne227c0ed7e0ed1f9b8d029336f8aeb60e38c23df\n\nFrom the oslo-incubator bug:\nWhen the QPID broker is restarted (or fails over), subscribed clients\nwill attempt to re-establish their connections.  In the case of fanout\nsubscriptions, this reconnection functionality is broken. For version\n1 topologies, the clients attempt to reconnect twice to the same\nexclusive address - which is illegal.  In the case of version 2\ntopologies, the address parsing is broken and an illegal address is\ncreated on reconnect.  This fix avoids the problem by removing the\nspecial-case reconnect code that manages UUID addresses; it is\nunnecessary as the QPID broker will generate unique queue names\nautomatically when the clients reconnect.\n\nChange-Id: I563db129e63817ad55165e318f164f06b141ee33\nCloses-Bug: 1251757\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/heat/commit/6ccbf6e7abbdb518599f943fb743e538b1f81708"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/heat/commit/6ccbf6e7abbdb518599f943fb743e538b1f81708"}]},"branch":"refs/heads/stable/havana"}},"requirements":[],"submit_records":[],"submit_requirements":[]}