)]}'
{"id":"openstack%2Fdevstack~23542","triplet_id":"openstack%2Fdevstack~master~I7d457efc6e01e4e955388f52a4907524052f0173","project":"openstack/devstack","branch":"master","hashtags":[],"change_id":"I7d457efc6e01e4e955388f52a4907524052f0173","subject":"Remove trailing slash from ceilometer endpoint URLs","status":"MERGED","created":"2013-03-05 12:47:20.000000000","updated":"2013-03-07 16:37:03.000000000","submitted":"2013-03-06 19:09:08.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"6c32573b6da324189b523d8a6259f27d2e317717","_number":23542,"virtual_id_number":23542,"owner":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-03-06 19:09:08.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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"},"recommended":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-03-05 13:20:46.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":2,"date":"2013-03-06 18:07:21.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-03-05 13:16:06.000000000","permitted_voting_range":{"min":0,"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":{"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,"date":"2013-03-05 12:49:02.000000000","_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":1,"date":"2013-03-06 18:07:21.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-03-05 13:16:06.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":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-03-05 12:49:02.000000000","updated_by":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"reviewer":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"state":"REVIEWER"},{"updated":"2013-03-05 13:16:06.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-03-06 18:07:21.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-03-06 19:09:08.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"75f8bf45cac73644dfc83e473fb5d9e234366331","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-03-05 13:16:06.000000000","message":"Patch Set 1:\n\nI\u0027m ok with this change, but it seems like the ceilo client should also take this as a bug and try to be more adaptive here. Seems fragile in the current state.","accounts_in_message":[],"_revision_number":1},{"id":"aeb3d6afeddb47fa26dfca2aca48d71fc005d455","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-03-05 13:20:46.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nI agree that we should have the client clean up the URLs, too. If I understand correctly, we\u0027re using the standard client base classes. I\u0027m not sure if we\u0027re doing our own URL munging or not.","accounts_in_message":[],"_revision_number":1},{"id":"2208ea9808d0214c09bb2135d1edab7ea325812a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-05 13:24:32.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/23542/1/check/gate-tempest-devstack-vm-full/6988 : SUCCESS\n- http://logs.openstack.org/23542/1/check/gate-tempest-devstack-vm-postgres-full/6707 : SUCCESS\n- http://logs.openstack.org/23542/1/check/gate-tempest-devstack-vm-quantum/11959 : SUCCESS\n","accounts_in_message":[],"_revision_number":1},{"id":"504717b4c660aa4e437de4794835a80a662d8165","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-03-06 18:07:21.000000000","message":"Patch Set 1: Looks good to me (core reviewer); Approved\n\nThe existing client libraries are inconsistently dirty WRT trailing slashes.  I tried cleaning that up a while back (Essex?) and it crept back in...","accounts_in_message":[],"_revision_number":1},{"id":"c354748f1b514e7f7ea0a5cb08c28ca54af7c91e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-06 18:07:23.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"ebb457ad31a12f6a29f2b832a1c4f38c64414b8c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-06 19:09:08.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/23542/1/gate/gate-tempest-devstack-vm-full/7436 : SUCCESS\n- http://logs.openstack.org/23542/1/gate/gate-tempest-devstack-vm-postgres-full/7152 : SUCCESS\n- http://logs.openstack.org/23542/1/gate/gate-tempest-devstack-vm-quantum/12509 : SUCCESS\n","accounts_in_message":[],"_revision_number":1},{"id":"da6892f7a72cf45bf603f81161f37acac1a7d76d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-06 19:09:08.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1},{"id":"6c32573b6da324189b523d8a6259f27d2e317717","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2013-03-07 16:37:03.000000000","message":"Patch Set 1:\n\nSean - there\u0027s a patch pending to clean this up in the ceilo client:\n\n  https://review.openstack.org/23549","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"21ddaac8702ade8f8c6fec558f8185419c46eec3","revisions":{"21ddaac8702ade8f8c6fec558f8185419c46eec3":{"kind":"REWORK","_number":1,"created":"2013-03-05 12:47:20.000000000","uploader":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"ref":"refs/changes/42/23542/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/42/23542/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/23542/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/23542/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/23542/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/42/23542/1"}}},"commit":{"parents":[{"commit":"09d6185b3f79f558cf6a42ca3d9b3c7d685dbce7","subject":"Set l3_agent_manager for ovs/linuxbridge plugin","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/09d6185b3f79f558cf6a42ca3d9b3c7d685dbce7"}]}],"author":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2013-03-05 12:40:59.000000000","tz":0},"committer":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2013-03-05 12:40:59.000000000","tz":0},"subject":"Remove trailing slash from ceilometer endpoint URLs","message":"Remove trailing slash from ceilometer endpoint URLs\n\nFixes bug 1146794\n\nEnsure the URLs configured for the ceilometer endpoint in the keystone\nservice catalog do not contain a trailing forward slash.\n\nOtherwise, this leads to an extra forward slash in the URL paths\nconstructed by the ceilometer client, which in turn is problematic\nfor the v2 API.\n\nChange-Id: I7d457efc6e01e4e955388f52a4907524052f0173\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/21ddaac8702ade8f8c6fec558f8185419c46eec3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/21ddaac8702ade8f8c6fec558f8185419c46eec3"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}