)]}'
{"id":"openstack%2Fdevstack~53031","triplet_id":"openstack%2Fdevstack~master~I9f04db46706bf6453def09ca1e22e0db918d811b","project":"openstack/devstack","branch":"master","topic":"fix-docker","hashtags":[],"change_id":"I9f04db46706bf6453def09ca1e22e0db918d811b","subject":"Removed docker version requirement","status":"MERGED","created":"2013-10-21 23:19:32.000000000","updated":"2013-10-31 14:22:37.000000000","submitted":"2013-10-31 14:22:36.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"fc3eab636cc1b1ad7413e14bddb34039132eb2d1","_number":53031,"virtual_id_number":53031,"owner":{"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-10-31 14:22:36.000000000","_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":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},{"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"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-10-24 21:18:30.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-10-24 19:29:25.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},{"value":2,"date":"2013-10-31 11:45:14.000000000","_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":"","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-10-24 21:18:30.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},{"value":1,"date":"2013-10-31 11:45:14.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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},{"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-24 21:18:30.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-10-31 11:45:14.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-10-31 14:22:36.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"d795f9e11d0a08915fb11e9ea6ad97e26e2e3c72","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-22 00:05:45.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- check-tempest-devstack-vm-full http://logs.openstack.org/31/53031/1/check/check-tempest-devstack-vm-full/cb379bb : SUCCESS in 29m 02s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/31/53031/1/check/check-tempest-devstack-vm-postgres-full/1818dd6 : SUCCESS in 46m 04s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/31/53031/1/check/check-tempest-devstack-vm-neutron/788cfa3 : SUCCESS in 29m 55s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/31/53031/1/check/gate-tempest-devstack-vm-large-ops/835998f : SUCCESS in 14m 11s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/31/53031/1/check/gate-tempest-devstack-vm-neutron-large-ops/60afecf : SUCCESS in 17m 22s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/31/53031/1/check/check-grenade-devstack-vm/c18e22a : SUCCESS in 27m 28s\n- check-swift-devstack-vm-functional http://logs.openstack.org/31/53031/1/check/check-swift-devstack-vm-functional/c5466ed : SUCCESS in 12m 25s\n","accounts_in_message":[],"_revision_number":1},{"id":"81832f8d2002d87581df56ec4bd5cf4e767c1961","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-10-22 20:54:47.000000000","message":"Patch Set 1:\n\nI don\u0027t like having the version in the package name, is there any way to avoid that?","accounts_in_message":[],"_revision_number":1},{"id":"feb0c36ab82605e414de45e2512cc354a5492bbd","author":{"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},"date":"2013-10-22 20:57:51.000000000","message":"Patch Set 1:\n\nWith Docker 0.7 coming soon, I\u0027d recommend to stay on 0.6.x and test each new version before upgrading.\n\nOnce Docker gets more stable, we can remove the version and just install \"lxc-docker\" which will get the latest version everytime. The nova driver hardcodes the API version so it will not break.","accounts_in_message":[],"_revision_number":1},{"id":"4648199e8d7609fa93c53b97b08f7fe69e9d5915","author":{"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},"date":"2013-10-24 19:26:26.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"50ec2c4eef68c63a9c5d7bd05b264c1b50e1d2e6","author":{"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},"date":"2013-10-24 19:27:18.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"6b39ab03fd16f2c6e02a94c506325096c92c8717","author":{"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},"date":"2013-10-24 19:29:25.000000000","message":"Patch Set 3:\n\nI finally removed the version dependency for 2 reasons:\n\n1. The API version is hardcoded in the docker client, so we\u0027re already safe on this side (when a new version is released)\n\n2. Everytime a new version of Docker, forcing the package version on lxc-docker won\u0027t work anyway. At least with this change, devstack will be install to always install a proper docker.","accounts_in_message":[],"_revision_number":3},{"id":"a8831728c3a47de48c3e558d3d5b3676d68c00e9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-24 20:32:14.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/31/53031/3/check/gate-devstack-bash8/7acd773 : SUCCESS in 6s\n- check-tempest-devstack-vm-full http://logs.openstack.org/31/53031/3/check/check-tempest-devstack-vm-full/967dbb1 : SUCCESS in 30m 16s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/31/53031/3/check/check-tempest-devstack-vm-postgres-full/960d62e : SUCCESS in 1h 04m 30s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/31/53031/3/check/check-tempest-devstack-vm-neutron/270ae51 : SUCCESS in 19m 12s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/31/53031/3/check/gate-tempest-devstack-vm-large-ops/016f1fe : SUCCESS in 16m 48s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/31/53031/3/check/gate-tempest-devstack-vm-neutron-large-ops/74527ab : SUCCESS in 17m 15s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/31/53031/3/check/check-grenade-devstack-vm/1467e5b : SUCCESS in 24m 44s\n- check-swift-devstack-vm-functional http://logs.openstack.org/31/53031/3/check/check-swift-devstack-vm-functional/5c6936e : SUCCESS in 11m 31s\n","accounts_in_message":[],"_revision_number":3},{"id":"5b7316740af68d1eea6a1a32984be2dc68b2b5e6","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-10-24 21:18:30.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\nThanks Sam, that\u0027s exactly what I was hoping we could do.","accounts_in_message":[],"_revision_number":3},{"id":"60dac5ec641656231898b967a8969a8420b6a7bc","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-10-31 11:45:14.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":3},{"id":"8c29090aceff8a5f40ef382df28d97233da91c8b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-31 11:46:04.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"4a7afe44d7cfe347a019a948a3fac9eacca60d44","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-31 14:22:36.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/31/53031/3/gate/gate-devstack-bash8/c4b356a : SUCCESS in 8s\n- gate-tempest-devstack-vm-full http://logs.openstack.org/31/53031/3/gate/gate-tempest-devstack-vm-full/635c325 : SUCCESS in 30m 52s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/31/53031/3/gate/gate-tempest-devstack-vm-postgres-full/c882559 : SUCCESS in 36m 14s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/31/53031/3/gate/gate-tempest-devstack-vm-neutron/f837d31 : SUCCESS in 21m 35s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/31/53031/3/gate/gate-tempest-devstack-vm-large-ops/ab1c67d : SUCCESS in 19m 09s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/31/53031/3/gate/gate-tempest-devstack-vm-neutron-large-ops/a0ea7a2 : SUCCESS in 23m 55s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/31/53031/3/gate/gate-grenade-devstack-vm/91fac29 : SUCCESS in 23m 28s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/31/53031/3/gate/gate-swift-devstack-vm-functional/b380b2e : SUCCESS in 10m 52s\n","accounts_in_message":[],"_revision_number":3},{"id":"5e2dfd39905aba7a748f246843965c3aaa800fe5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-31 14:22:37.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"5a77d03addeab27704eeeb6a43ea75c1a9941764","revisions":{"a31ec13c185df201a39a29dc543845c124061e54":{"kind":"REWORK","_number":1,"created":"2013-10-21 23:19:32.000000000","uploader":{"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},"ref":"refs/changes/31/53031/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/31/53031/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/53031/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/53031/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/53031/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/31/53031/1"}}},"commit":{"parents":[{"commit":"ff38d464e41db3d8f83c2b85b48f983e424f3a23","subject":"Merge \"Use nova.conf for auth_token configs.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ff38d464e41db3d8f83c2b85b48f983e424f3a23"}]}],"author":{"name":"Sam Alba","email":"sam.alba@gmail.com","date":"2013-10-21 23:17:30.000000000","tz":-420},"committer":{"name":"Sam Alba","email":"sam.alba@gmail.com","date":"2013-10-21 23:17:30.000000000","tz":-420},"subject":"Fixed Docker package install + upgraded Docker to the latest 0.6.4","message":"Fixed Docker package install + upgraded Docker to the latest 0.6.4\n\nChange-Id: I9f04db46706bf6453def09ca1e22e0db918d811b\nCloses-Bug: #1237581\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a31ec13c185df201a39a29dc543845c124061e54"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a31ec13c185df201a39a29dc543845c124061e54"}]},"branch":"refs/heads/master"},"83c37b656d9e2022d1190fb68638f4255a441dd8":{"kind":"REWORK","_number":2,"created":"2013-10-24 19:26:26.000000000","uploader":{"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},"ref":"refs/changes/31/53031/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/31/53031/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/53031/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/53031/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/53031/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/31/53031/2"}}},"commit":{"parents":[{"commit":"0c42f250430ad2e68c0a72804179a80522a159df","subject":"Merge \"Just reset the repo instead of trying to co files\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0c42f250430ad2e68c0a72804179a80522a159df"}]}],"author":{"name":"Sam Alba","email":"sam.alba@gmail.com","date":"2013-10-21 23:17:30.000000000","tz":-420},"committer":{"name":"Sam Alba","email":"sam.alba@gmail.com","date":"2013-10-24 19:26:13.000000000","tz":-420},"subject":"Removed docker version requirement","message":"Removed docker version requirement\n\nChange-Id: I9f04db46706bf6453def09ca1e22e0db918d811b\nCloses-Bug: #1237581\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/83c37b656d9e2022d1190fb68638f4255a441dd8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/83c37b656d9e2022d1190fb68638f4255a441dd8"}]},"branch":"refs/heads/master"},"5a77d03addeab27704eeeb6a43ea75c1a9941764":{"kind":"REWORK","_number":3,"created":"2013-10-24 19:27:18.000000000","uploader":{"_account_id":7838,"name":"Sam Alba","email":"sam.alba@gmail.com","username":"samalba"},"ref":"refs/changes/31/53031/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/31/53031/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/53031/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/53031/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/53031/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/31/53031/3"}}},"commit":{"parents":[{"commit":"0c42f250430ad2e68c0a72804179a80522a159df","subject":"Merge \"Just reset the repo instead of trying to co files\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0c42f250430ad2e68c0a72804179a80522a159df"}]}],"author":{"name":"Sam Alba","email":"sam.alba@gmail.com","date":"2013-10-21 23:17:30.000000000","tz":-420},"committer":{"name":"Sam Alba","email":"sam.alba@gmail.com","date":"2013-10-24 19:27:10.000000000","tz":-420},"subject":"Removed docker version requirement","message":"Removed docker version requirement\n\nChange-Id: I9f04db46706bf6453def09ca1e22e0db918d811b\nCloses-Bug: #1237581\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5a77d03addeab27704eeeb6a43ea75c1a9941764"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5a77d03addeab27704eeeb6a43ea75c1a9941764"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}