)]}'
{"id":"openstack%2Fdevstack~48088","triplet_id":"openstack%2Fdevstack~master~I4ec2b45688da3fa6c0d43e8be60885774cfbffd6","project":"openstack/devstack","branch":"master","topic":"bp/1229780","hashtags":[],"change_id":"I4ec2b45688da3fa6c0d43e8be60885774cfbffd6","subject":"Expose all versions of Ironic API","status":"MERGED","created":"2013-09-24 16:11:25.000000000","updated":"2013-09-29 11:36:24.000000000","submitted":"2013-09-29 11:36:24.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0545bf22ac4462f08a1d4984ddd5f4c525714e55","_number":48088,"virtual_id_number":48088,"owner":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-09-29 11:36:24.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"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"},{"value":0,"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"}],"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"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-09-25 14:53:01.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-09-29 11:06:02.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"date":"2013-09-24 16:53:56.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-09-25 14:53:01.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-09-29 11:06:02.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-09-25 14:53:01.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-09-29 11:06:02.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-09-29 11:36:24.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"dad12b11710b876e022ef51007f17a080b2224b5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-24 16:41:47.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-tempest-devstack-vm-full http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-full/bd02cde : SUCCESS in 30m 19s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-postgres-full/37dcede : SUCCESS in 28m 50s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-neutron/48a1c67 : FAILURE in 23m 58s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-large-ops/3954522 : SUCCESS in 10m 22s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/88/48088/1/check/gate-grenade-devstack-vm/ae58320 : SUCCESS in 25m 21s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/88/48088/1/check/gate-swift-devstack-vm-functional/bec4267 : SUCCESS in 10m 08s\n","accounts_in_message":[],"_revision_number":1},{"id":"2534bf6fb6a8ebde6d77dee21691be089731bb80","author":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"date":"2013-09-24 16:53:56.000000000","message":"Patch Set 1:\n\nRecheck no bug","accounts_in_message":[],"_revision_number":1},{"id":"5363e5a71454ed7937e89e69dd2c90583773ea01","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-24 17:24:23.000000000","message":"Patch Set 1:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-full/a5b9573 : SUCCESS in 28m 58s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-postgres-full/29790c2 : SUCCESS in 30m 18s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-neutron/3a3bf29 : FAILURE in 30m 23s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-large-ops/f5c4342 : SUCCESS in 11m 47s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/88/48088/1/check/gate-grenade-devstack-vm/1c05925 : SUCCESS in 23m 37s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/88/48088/1/check/gate-swift-devstack-vm-functional/cde6c84 : SUCCESS in 10m 45s\n","accounts_in_message":[],"_revision_number":1},{"id":"1a76e55a65889e2e88a48562d0bd03f2f969422b","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2013-09-24 19:14:40.000000000","message":"Patch Set 1:\n\nI noticed tempest failed, I think you hit bug https://bugs.launchpad.net/bugs/1223161","accounts_in_message":[],"_revision_number":1},{"id":"e80603a2ee0cd1d0f182d5cdbf152dd1aeff0322","author":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"date":"2013-09-25 09:27:37.000000000","message":"Patch Set 1:\n\nRecheck no bug","accounts_in_message":[],"_revision_number":1},{"id":"3a25077d51995bdbfa3f569e3dc795fac7a59a7f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-25 09:57:07.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-full/47cc73f : SUCCESS in 28m 12s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-postgres-full/ce1a416 : SUCCESS in 29m 27s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-neutron/9adbbb7 : SUCCESS in 19m 38s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/88/48088/1/check/gate-tempest-devstack-vm-large-ops/448cb31 : SUCCESS in 10m 55s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/88/48088/1/check/gate-grenade-devstack-vm/f2abe47 : SUCCESS in 23m 33s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/88/48088/1/check/gate-swift-devstack-vm-functional/aa151bf : SUCCESS in 11m 21s\n","accounts_in_message":[],"_revision_number":1},{"id":"e073908f920394cc42470f2eb9ebb10c1c4ac6be","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-25 11:55:25.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nthere aren\u0027t other versions yet, so there is no need for this.\n\nAlso, matt\u0027s bot actually told you which bug this was to recheck, so you should not have rechecked with no bug","accounts_in_message":[],"_revision_number":1},{"id":"bf37459641a6fdd26b0454d4d1359077d809d5e5","author":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"date":"2013-09-25 14:33:39.000000000","message":"Patch Set 1:\n\nSean, thank you for you review. I was not too familiar with Gerrit commands, sorry for \"no bug\".\n\nCurrently in Ironic team we are trying to make some tempest tests so if we do not expose a http://host:port endpoint now we have to update tempest tests later.\n\nAlso there are some API discovery features that cannot be accessed (and tested) with the http://host:port/v1 endpoint.","accounts_in_message":[],"_revision_number":1},{"id":"516bbeeaf33ba51c1a8c1aa851f1057f9db89999","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-25 14:53:01.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\nHonestly, I\u0027m strongly in favor of versions in endpoints going away.  Otherwise we wind up with computev3 and computev4, etc...  It\u0027ll take a while for the existing services to finally get there but let\u0027s not start the new ones like that if possible.","accounts_in_message":[],"_revision_number":1},{"id":"f4894dd91d94b0b9c1bf063a09a6cc120eb93e61","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-29 11:06:02.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"df5db53eb38cd9db97f81e2c3e86f1c3f2f69ae1","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-29 11:06:02.000000000","message":"Patch Set 1: Looks good to me (core reviewer); Approved\n\nok, makes sense","accounts_in_message":[],"_revision_number":1},{"id":"7528d5b7f0d30e1954791ec372bf976cc1245ae4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-29 11:36:24.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1},{"id":"f1120b09c2ae4443bf9fd689d0c45e94fd2616b7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-29 11:36:24.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/88/48088/1/gate/gate-tempest-devstack-vm-full/135bb90 : SUCCESS in 30m 15s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/88/48088/1/gate/gate-tempest-devstack-vm-postgres-full/7dd4815 : SUCCESS in 29m 11s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/88/48088/1/gate/gate-tempest-devstack-vm-neutron/ce33d69 : SUCCESS in 18m 53s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/88/48088/1/gate/gate-tempest-devstack-vm-large-ops/669d908 : SUCCESS in 10m 25s\n- gate-grenade-devstack-vm http://logs.openstack.org/88/48088/1/gate/gate-grenade-devstack-vm/472219d : SUCCESS in 23m 54s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/88/48088/1/gate/gate-swift-devstack-vm-functional/5d66890 : SUCCESS in 11m 19s\n","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"f5002ef12a890fd3110782c873d99487a4d05b17","revisions":{"f5002ef12a890fd3110782c873d99487a4d05b17":{"kind":"REWORK","_number":1,"created":"2013-09-24 16:11:25.000000000","uploader":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"ref":"refs/changes/88/48088/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/88/48088/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/88/48088/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/88/48088/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/88/48088/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/88/48088/1"}}},"commit":{"parents":[{"commit":"d24674ed22687159ced65613c714a075d9804279","subject":"Merge \"Modified ceilometer alarm evaluator console script\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d24674ed22687159ced65613c714a075d9804279"}]}],"author":{"name":"Roman Prykhodchenko","email":"rprikhodchenko@mirantis.com","date":"2013-09-24 16:09:26.000000000","tz":180},"committer":{"name":"Roman Prykhodchenko","email":"rprikhodchenko@mirantis.com","date":"2013-09-24 16:11:09.000000000","tz":180},"subject":"Expose all versions of Ironic API","message":"Expose all versions of Ironic API\n\nIn the observable future new versions of the Ironic\nAPI will appear. That\u0027s why it\u0027s reasonable to expose\nthe endpoint that will provide access to all versions\nof the API.\n\nCloses-Bug: #1229780\nChange-Id: I4ec2b45688da3fa6c0d43e8be60885774cfbffd6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f5002ef12a890fd3110782c873d99487a4d05b17"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f5002ef12a890fd3110782c873d99487a4d05b17"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}