)]}'
{"id":"openstack%2Fdevstack~1842","triplet_id":"openstack%2Fdevstack~master~If3e8ad14e364f6296b8a911c9cb405baca8e740c","project":"openstack/devstack","branch":"master","topic":"bin-swift-curl-from-github","hashtags":[],"change_id":"If3e8ad14e364f6296b8a911c9cb405baca8e740c","subject":"use github bin/swift","status":"ABANDONED","created":"2011-11-22 10:41:15.000000000","updated":"2011-12-30 21:29:09.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"b7fb7a90351b7ed997abb8372de3ea46634cdfba","_number":1842,"virtual_id_number":1842,"owner":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"actions":{},"labels":{"Verified":{"all":[{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"}],"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":{"disliked":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"all":[{"value":0,"date":"2011-11-23 19:19:24.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"value":-1,"date":"2011-11-23 19:30:00.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"date":"2011-11-22 18:27:19.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"date":"2011-11-22 18:24:24.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"date":"2011-11-23 19:26:55.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"}],"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":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"}],"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":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"}],"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":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2011-11-22 18:24:24.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2011-11-23 19:19:24.000000000","updated_by":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"reviewer":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"state":"REVIEWER"},{"updated":"2011-11-23 19:26:55.000000000","updated_by":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"reviewer":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"state":"REVIEWER"},{"updated":"2011-11-23 19:30:00.000000000","updated_by":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"reviewer":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"state":"REVIEWER"}],"messages":[{"id":"22577572a1117f408db6345cca1ae3beea75b332","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2011-11-22 17:26:41.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"19192b0ef371e18b2c3cb81a999a28ac41a20b00","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2011-11-22 17:35:14.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t submit this\n\nCan we just use a different gerrit url instead of a github URL here? github is less reliable and has more of a chance of getting a 404 when we\u0027re running stack.sh as part of a an automated job, which would lead to false test failures.","accounts_in_message":[],"_revision_number":2},{"id":"9a9f8b528a6176e52a744a00a31d420e88e64704","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2011-11-22 17:38:40.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t submit this\n\nActually, since this is inside of enabled_services \u003d~ swift, we should know that we have a local swift checkout.  Why not just symlink it from there to /usr/local/bin?","accounts_in_message":[],"_revision_number":2},{"id":"e97af521257731742cd680013bc214b1e3c79fe2","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2011-11-22 17:40:20.000000000","message":"Patch Set 2:\n\nTotally right, I forgot we have trunk checked out in there, thanks for pointing. Will rework this.","accounts_in_message":[],"_revision_number":2},{"id":"cd328b87e9bfa7abc6fb81fd76b6e5a5401193c6","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2011-11-22 18:14:48.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"90c334ba296650b3b8c98c92017cb65248a3a8da","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2011-11-22 18:24:24.000000000","message":"Patch Set 3: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"dc7b3194d80a4c22b19827e8bd55de012f0d31e0","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2011-11-22 18:27:19.000000000","message":"Patch Set 3:\n\nWe checkout stable/diablo and the 2.0 support was only added in 1.4.7 the bin/swift utility is backward compatible with stable/diablo. This will be removed in the future.","accounts_in_message":[],"_revision_number":3},{"id":"8402b790057851a56816eacefbb9c2f26ebaf716","author":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"date":"2011-11-23 19:19:24.000000000","message":"Patch Set 3:\n\nYou might want to change this against stable/diablo then - we have a review in to make master essex","accounts_in_message":[],"_revision_number":3},{"id":"2623bd73a64cb767a96929a515d87ada726639a1","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2011-11-23 19:30:00.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t submit this\n\nI agree with Jesse, if this belongs, it probably only belongs in the stable/diablo branch.  If you\u0027re using git-review from pip, you can \u0027git review stable/diablo\u0027.\n\nHowever, I\u0027d still like to know what\u0027s driving this.\n\nMy particular concern is that if we start hard-coding checkouts of tools in the script, we lose the ability to effectively test changes.  For instance, would this change mean that bin/swift never gets tested again because it is pinned at a particular version indefinitely?","accounts_in_message":[],"_revision_number":3},{"id":"578ecf978bc5878dadb2c7bdd36e198f50a1983d","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2011-11-23 19:43:31.000000000","message":"Patch Set 3:\n\nWell we need a swift cli that has support for auth_version 2.0 I am not sure how to do it in an another way, how would you suggest we are getting a swift cli compatible with 2.0 installed on a devstack installation ?\n\nMy hope is that this is only a temporary thing to make things work until we don\u0027t need that kind of hack.","accounts_in_message":[],"_revision_number":3},{"id":"b5339430e2a7e2faaa7b348db4c9f3534131c906","author":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"date":"2011-12-17 05:06:14.000000000","message":"Patch Set 3:\n\nplease abandon this review","accounts_in_message":[],"_revision_number":3},{"id":"515f7190c22769924ecce36ff64e871c41888998","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2011-12-30 21:29:09.000000000","message":"Patch Set 3: Abandoned","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"66480b2fd2c7892123540638289c282baf9ae46d","revisions":{"5f75f013bd86c98052a3ec1a98caa70bce073cee":{"kind":"REWORK","_number":1,"created":"2011-11-22 10:41:15.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/42/1842/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/42/1842/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/1842/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/1842/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/1842/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/42/1842/1"}}},"commit":{"parents":[{"commit":"d7326d2ea98b75c5e92e7309cc49abd680476693","subject":"remove old build scripts and clean for those that work","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d7326d2ea98b75c5e92e7309cc49abd680476693"}]}],"author":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2011-11-22 10:38:52.000000000","tz":60},"committer":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2011-11-22 10:38:52.000000000","tz":60},"subject":"use github bin/swift","message":"use github bin/swift\n\nSince this is now in official 1.4.3 we use this bin/swift instead of\nthe gerrit review URL. Hopefully in the future we can switch to 1.4.3\nand we won\u0027t need this hack.\n\nChange-Id: If3e8ad14e364f6296b8a911c9cb405baca8e740c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5f75f013bd86c98052a3ec1a98caa70bce073cee"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5f75f013bd86c98052a3ec1a98caa70bce073cee"}]},"branch":"refs/heads/master"},"fed849d7226dd40c864a7965c095f0680ae5cbb8":{"kind":"REWORK","_number":2,"created":"2011-11-22 17:26:41.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/42/1842/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/42/1842/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/1842/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/1842/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/1842/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/42/1842/2"}}},"commit":{"parents":[{"commit":"d7326d2ea98b75c5e92e7309cc49abd680476693","subject":"remove old build scripts and clean for those that work","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d7326d2ea98b75c5e92e7309cc49abd680476693"}]}],"author":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2011-11-22 10:38:52.000000000","tz":60},"committer":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2011-11-22 17:25:47.000000000","tz":60},"subject":"use github bin/swift","message":"use github bin/swift\n\nSince this is now in official 1.4.5 we use this bin/swift instead of\nthe gerrit review URL. Hopefully in the future we can switch to a swift\nthat include auth_version and we won\u0027t need this hack.\n\nChange-Id: If3e8ad14e364f6296b8a911c9cb405baca8e740c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/fed849d7226dd40c864a7965c095f0680ae5cbb8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/fed849d7226dd40c864a7965c095f0680ae5cbb8"}]},"branch":"refs/heads/master"},"66480b2fd2c7892123540638289c282baf9ae46d":{"kind":"REWORK","_number":3,"created":"2011-11-22 18:14:48.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/42/1842/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/42/1842/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/1842/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/1842/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/42/1842/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/42/1842/3"}}},"commit":{"parents":[{"commit":"d7326d2ea98b75c5e92e7309cc49abd680476693","subject":"remove old build scripts and clean for those that work","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d7326d2ea98b75c5e92e7309cc49abd680476693"}]}],"author":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2011-11-22 10:38:52.000000000","tz":60},"committer":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2011-11-22 18:13:07.000000000","tz":60},"subject":"use github bin/swift","message":"use github bin/swift\n\nSince this is now in official 1.4.5 we checkout the bin/swift that has\nauth_version 2.0 support. Hopefully in the future we will have a swift\nversion that include auth_version and we won\u0027t need this hack.\n\nChange-Id: If3e8ad14e364f6296b8a911c9cb405baca8e740c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/66480b2fd2c7892123540638289c282baf9ae46d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/66480b2fd2c7892123540638289c282baf9ae46d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}