)]}'
{"id":"openstack%2Fdevstack~63011","triplet_id":"openstack%2Fdevstack~master~I2ab8fece5f71b70342017025f8faac9e044f33e1","project":"openstack/devstack","branch":"master","topic":"fix_heat_endpoint","hashtags":[],"change_id":"I2ab8fece5f71b70342017025f8faac9e044f33e1","subject":"Fix orchestration service endpoints in catalog template","status":"ABANDONED","created":"2013-12-18 23:01:23.000000000","updated":"2014-01-17 23:37:46.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"7b08bf631b9e51e62ba051f93242426e0cb8a576","_number":63011,"virtual_id_number":63011,"owner":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":-1,"date":"2014-01-17 13:13:41.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},{"_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":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2014-01-02 17:49:37.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":-1,"date":"2014-01-09 17:28:13.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},{"value":-1,"date":"2014-01-02 02:07:15.000000000","permitted_voting_range":{"min":-1,"max":1},"_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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"date":"2013-12-18 23:01:33.000000000","_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},{"date":"2013-12-21 10:52:46.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,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-12-18 23:01:33.000000000","updated_by":{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},"reviewer":{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},"state":"REVIEWER"},{"updated":"2013-12-21 10:52:46.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-01-17 13:13:41.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"b5825956fd64d6022fd0ecd4265fee10dad9e22b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-18 23:48:32.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/11/63011/1/check/gate-devstack-bash8/81e98b7 : SUCCESS in 7s\n- check-tempest-dsvm-full http://logs.openstack.org/11/63011/1/check/check-tempest-dsvm-full/f075b38 : SUCCESS in 46m 57s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/11/63011/1/check/check-tempest-dsvm-postgres-full/f18ae0e : SUCCESS in 39m 49s\n- check-tempest-dsvm-neutron http://logs.openstack.org/11/63011/1/check/check-tempest-dsvm-neutron/21b5521 : SUCCESS in 32m 44s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/11/63011/1/check/gate-tempest-dsvm-large-ops/0fa4c86 : SUCCESS in 13m 16s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/11/63011/1/check/gate-tempest-dsvm-neutron-large-ops/907124b : SUCCESS in 16m 20s\n- check-grenade-dsvm http://logs.openstack.org/11/63011/1/check/check-grenade-dsvm/971ffa0 : SUCCESS in 40m 12s\n- check-swift-dsvm-functional http://logs.openstack.org/11/63011/1/check/check-swift-dsvm-functional/767c642 : SUCCESS in 11m 33s\n","accounts_in_message":[],"_revision_number":1},{"id":"a7a1f69c350285840a35183314692235b547643c","author":{"_account_id":9009,"name":"Red Hat CI","email":"redhatci@redhat.com","username":"redhatci","tags":["SERVICE_USER"]},"date":"2013-12-19 00:06:05.000000000","message":"Patch Set 1:\n\nredhatci results for change 63011\n\nLogs at http://people.redhat.com/~iwienand/63011/\n\n- Patch set 1 | OSLAB RHEL64 : PASS\n- Patch set 1 | OSLAB Fedora 19 : PASS\n- Patch set 1 | OSLAB Fedora 20 : FAIL\n- Patch set 1 | OSLAB CentOS 6.4 : PASS\n- Patch set 1 | OSLAB CentOS 6.5 : PASS\n- Patch set 1 | OSLAB RHEL7 : FAIL\n- Patch set 1 | OSLAB RHEL65 : PASS\n\nend","accounts_in_message":[],"_revision_number":1},{"id":"0df7f8d8a1ed12e0bdd7f32097aa6aef0e878e71","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-12-21 10:52:46.000000000","message":"Patch Set 1:\n\ndoes this fix an issue? or is this just a cleanup.","accounts_in_message":[],"_revision_number":1},{"id":"115f1b9d50c0a471bb78baa0b27fba0522f050ac","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2014-01-02 02:07:15.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\ntagging a -1 to my previous unanswered question.","accounts_in_message":[],"_revision_number":1},{"id":"062d03693a4756545efb70d0b67b56cf7412b1c4","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2014-01-02 17:49:37.000000000","message":"Patch Set 1:\n\nI made this change when I got a comment from Steven Hardy when I tried to update keystone\u0027s sample catalog templates to match devstack\u0027s[1]. I didn\u0027t check to see if this causes an actual problem, but I\u0027ll try it out some and see if it fails. It might also turn out that the correct change is to remove these lines because maybe heat does it.\n\n[1] https://review.openstack.org/#/c/54636/","accounts_in_message":[],"_revision_number":1},{"id":"64860626f835176c67e25164b531aea951ec1636","author":{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},"date":"2014-01-09 17:28:13.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nThanks to Brant for following up on this - the entry here is definitely wrong, but I think all is working OK because of this code, which is setting up the endpoints correctly:\n\nhttps://github.com/openstack-dev/devstack/blob/master/files/keystone_data.sh#L78\n\nSo maybe the right thing is to remove the heat entry in this file?","accounts_in_message":[],"_revision_number":1},{"id":"0af603aaed5e74352192b9f7b62b573c8569153c","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2014-01-09 18:05:47.000000000","message":"Patch Set 1: Work In Progress\n\nMake updates based on Steve\u0027s comments.","accounts_in_message":[],"_revision_number":1},{"id":"1c0934d4814aaacd148be5d3e6fc151f09bf0a54","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2014-01-09 18:06:33.000000000","message":"Patch Set 1:\n\nSteve - thanks, that\u0027s something I was going to look into myself but then ran into other problems using devstack with heat, so wasn\u0027t able to try things out.","accounts_in_message":[],"_revision_number":1},{"id":"eb59118b4106f2cddc74478617173be40037a198","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2014-01-17 06:03:05.000000000","message":"Patch Set 1:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1},{"id":"067c4550ca226f04f1a1a1065471e5a9bc837179","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2014-01-17 06:03:05.000000000","message":"Patch Set 1: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1},{"id":"ff2a1d71b97db66cb2df838c012a4c1902fe2953","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2014-01-17 12:57:37.000000000","message":"Patch Set 1: Restored\n\nNeed to try out and post a bug, and also make alternative change where just remove it.","accounts_in_message":[],"_revision_number":1},{"id":"f47f314aae01dc366a44e042d275f5d927affb60","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-17 13:13:41.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\nThis change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"2c5222518804d436fbb7cf201e24039cba907689","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2014-01-17 23:37:46.000000000","message":"Patch Set 1: Abandoned\n\nThis was fixed by 1b0eccdf75cf70a26c1b2ae6b9beaa75ebaf7a6a","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"178f69f6320d3e756851827d1fba48e4bb49a745","revisions":{"178f69f6320d3e756851827d1fba48e4bb49a745":{"kind":"REWORK","_number":1,"created":"2013-12-18 23:01:23.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/11/63011/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/11/63011/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/63011/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/63011/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/63011/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/11/63011/1"}}},"commit":{"parents":[{"commit":"fedaf6f2465911cfbcef62712e12c4ac39b096b7","subject":"Merge \"Stop Neutron advanced service external processes\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/fedaf6f2465911cfbcef62712e12c4ac39b096b7"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2013-12-18 22:57:36.000000000","tz":-360},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2013-12-18 22:57:36.000000000","tz":-360},"subject":"Fix orchestration service endpoints in catalog template","message":"Fix orchestration service endpoints in catalog template\n\nThe orchestration service had 8000 for its port, but the\norchestration service actually listens on port 8004.\n\nAlso, the orchestration service requires a tenant ID.\n\nChange-Id: I2ab8fece5f71b70342017025f8faac9e044f33e1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/178f69f6320d3e756851827d1fba48e4bb49a745"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/178f69f6320d3e756851827d1fba48e4bb49a745"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}