)]}'
{"id":"openstack%2Fopenstack-chef~3613","triplet_id":"openstack%2Fopenstack-chef~master~I9f0c3006aae6c20cda0ac1eae375ea836f3ee4bd","project":"openstack/openstack-chef","branch":"master","topic":"feature/add-swift-aio-cookbook","hashtags":[],"change_id":"I9f0c3006aae6c20cda0ac1eae375ea836f3ee4bd","subject":"Add cookbook for Swift All In One.","status":"ABANDONED","created":"2012-02-01 04:44:51.000000000","updated":"2012-03-16 06:03:02.000000000","total_comment_count":11,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"432a4c649eb90a27a18b8be613ad546374dbf2ab","_number":3613,"virtual_id_number":3613,"owner":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"actions":{},"labels":{"Verified":{"all":[{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"_account_id":537,"name":"a abes","username":"a-abes"},{"_account_id":1580,"name":"Nachi Ueno","username":"nati-ueno-s","inactive":true}],"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":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"all":[{"value":0,"date":"2012-02-28 21:25:29.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},{"value":0,"date":"2012-02-15 01:51:30.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},{"value":2,"date":"2012-02-15 01:52:16.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"value":-1,"date":"2012-02-28 22:25:16.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":537,"name":"a abes","username":"a-abes"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1580,"name":"Nachi Ueno","username":"nati-ueno-s","inactive":true}],"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":{"all":[{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},{"date":"2012-02-15 01:52:16.000000000","_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"date":"2012-02-28 21:57:56.000000000","_account_id":537,"name":"a abes","username":"a-abes"},{"_account_id":1580,"name":"Nachi Ueno","username":"nati-ueno-s","inactive":true}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"_account_id":537,"name":"a abes","username":"a-abes"},{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},{"_account_id":1580,"name":"Nachi Ueno","username":"nati-ueno-s","inactive":true},{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-02-08 03:10:15.000000000","updated_by":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"reviewer":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"state":"REVIEWER"},{"updated":"2012-02-08 03:13:12.000000000","updated_by":{"_account_id":1580,"name":"Nachi Ueno","username":"nati-ueno-s","inactive":true},"reviewer":{"_account_id":1580,"name":"Nachi Ueno","username":"nati-ueno-s","inactive":true},"state":"REVIEWER"},{"updated":"2012-02-15 01:52:16.000000000","updated_by":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"reviewer":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"state":"REVIEWER"},{"updated":"2012-02-28 21:25:29.000000000","updated_by":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"reviewer":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"state":"REVIEWER"},{"updated":"2012-02-28 21:57:56.000000000","updated_by":{"_account_id":537,"name":"a abes","username":"a-abes"},"reviewer":{"_account_id":537,"name":"a abes","username":"a-abes"},"state":"REVIEWER"}],"messages":[{"id":"7a548ff783ac70b0c479b3f709c6dcb92f364594","author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"date":"2012-02-08 03:38:19.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n(1 inline comment)\n\nLooks pretty good to me, but would be best to have a Swift expert look this over...","accounts_in_message":[],"_revision_number":1},{"id":"f5e3a8a5cb9ac87fb716a1e3e276d29bbff1f359","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2012-02-08 04:35:14.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"4a69d36845307ae2df4e5d1fb1e43eca3afa7b70","author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"date":"2012-02-08 04:36:25.000000000","message":"Patch Set 1:\n\nNah, no worries... when I read through the SAIO docs, I realized why you hard-coded Lucid :)\n\nWe can work on templatizing the recipe after the alignment currently being discussed on the ML starts happening..\n\nBest,\n\n-jay","accounts_in_message":[],"_revision_number":1},{"id":"8ad95382553d306eb41a8c900c1268ef95838748","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2012-02-15 01:47:57.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"86a11294c0f0139f7170a7ee55d2fa61a6fc55fc","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2012-02-15 01:51:30.000000000","message":"Patch Set 2:\n\nPatch 2 adds installation of openstack.nose_plugin to ensure that .unittests will still run as advertised once the tox changes land in swift.","accounts_in_message":[],"_revision_number":2},{"id":"84b3587135c8f46fa69b0314c1fad9b8d541913b","author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"date":"2012-02-15 01:52:16.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nAh, nice update.","accounts_in_message":[],"_revision_number":2},{"id":"51a1ce9736498a2a872ad219f2389392f75afefd","author":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"date":"2012-02-28 21:25:29.000000000","message":"Patch Set 2: (1 inline comment)\n\nJust want to make sure scripts that are made work on more than 1 distro since openstack needs to get out of the 1 distro mode...","accounts_in_message":[],"_revision_number":2},{"id":"f5a314ddfa00b15d2f753e505f9bf4f59154e55d","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2012-02-28 21:28:11.000000000","message":"Patch Set 2:\n\nharlowja:  As mentioned in the commit message, this script automates the documented procedure for creating a Swift AIO.  Since the documentation prescribes Ubuntu 10.04 - and relies upon a Launchpad PPA for many of its dependencies - so does this recipe.","accounts_in_message":[],"_revision_number":2},{"id":"7690fc8e9431b898666ab559c7eb072d82f7829c","author":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"date":"2012-02-28 21:30:48.000000000","message":"Patch Set 2:\n\nOk, so the desire is to keep with the status quo? Is it possible to make this chef cookbook not have that limitation (without that much work)? Wouldn\u0027t that be preferable?","accounts_in_message":[],"_revision_number":2},{"id":"f064779c363175cbe423d172fc01cb77bd55fc52","author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"date":"2012-02-28 21:32:43.000000000","message":"Patch Set 2:\n\nHi Josh, I\u0027m of the mind that such (wanted) improvements can come in a separate patch, no?\n\nI think Maru was sticking to the letter of the developer SAIO docs, which is why this cookbook looks the way it does.\n\nWould certainly love to see a patch from ya for a more agnostic setup.","accounts_in_message":[],"_revision_number":2},{"id":"757104678b1ad5e903fc5e69b0853e92feb68725","author":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"date":"2012-02-28 21:38:50.000000000","message":"Patch Set 2:\n\n;) Good call.","accounts_in_message":[],"_revision_number":2},{"id":"d283140020061e4d176dcc20ce00bd725667b0c0","author":{"_account_id":537,"name":"a abes","username":"a-abes"},"date":"2012-02-28 22:25:16.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t submit this\n\n(4 inline comments)\n\nMy overall impression is that this code would be much easier to maintain in pure bash scripts, ala devStack, rather than chef... Chef seems to just get in the way here, without actually adding much  value for the developer use cases that this cookbook addresses.\n\nSeems that what you\u0027re trying to address could be achieved with a few scripts:\n* prep_system - setup the basic pre-reqs (rsyslog, rsync,python, pip etc)\n* deploy_code - which could take a git repo or use a PPA\n* run - which just does all the things require to run stuff.\n* reset - which resets things.\n\nFrom a developer perspective, you\u0027d want to run these at different times, based on what you\u0027re trying to do. With chef, you\u0027d have to go tweak the node to adjust it\u0027s run_list just so it would run the script you want to run...\n\n\nThat said, I\u0027ve included a few inline comments..","accounts_in_message":[],"_revision_number":2},{"id":"87b6ee5aa82ff6ca500ec2cbbb43ca7fe83f6c22","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2012-02-28 22:57:47.000000000","message":"Patch Set 2:\n\nShould this change be abandoned?  I\u0027m getting the sense that nobody actually wants it.","accounts_in_message":[],"_revision_number":2},{"id":"09c68cf8b40b3ac1dc187a0eabbbbceb20b54c85","author":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"date":"2012-02-28 23:26:19.000000000","message":"Patch Set 2:\n\nI think its still useful. Its a good reference/example imho.","accounts_in_message":[],"_revision_number":2},{"id":"16bd11aec44441f70f74b18ea04c78f740adfd4a","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2012-03-01 01:25:48.000000000","message":"Patch Set 2: (4 inline comments)\n\na abes:  I have some questions about your comments in advance of submitting a revised patch.  Please reply at your convenience.","accounts_in_message":[],"_revision_number":2},{"id":"3980f26dce39f93f341c4b660e7291c9115fac10","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2012-03-08 06:03:05.000000000","message":"Patch Set 2:\n\ncode review expired after 1 week of no activity after a negative review","accounts_in_message":[],"_revision_number":2},{"id":"a5398c9bc37e6c5b1de417507a77d4800161628d","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2012-03-08 06:03:06.000000000","message":"Patch Set 2: Abandoned\n\ncode review expired after 1 week of no activity after a negative review","accounts_in_message":[],"_revision_number":2},{"id":"53a24992a1151d299d86c3b84a3c983b24310217","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2012-03-08 06:07:10.000000000","message":"Patch Set 2:\n\nSo should I resubmit this?  Or leave it be?\n\nAnd who do I talk to about getting a warning before automatic abandonment?","accounts_in_message":[],"_revision_number":2},{"id":"cbc42334280719a82de142bb4525842e8e8f9e15","author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"date":"2012-03-08 20:05:35.000000000","message":"Patch Set 2:\n\nMaru, you can chat with mtaylor and jeblair about the automated abandon stuff on IRC...\n\nI would actually like to see this go in somewhere, but I\u0027m not sure *where* based on feedback on this patchset...\n\nPerhaps we should start a thread on the ML to see what the community thinks about these options:\n\n * Create a new cookbook for SAIO (as opposed to production Swift cookbook)\n * Create a new recipe for SAIO under a generic Swift cookbook\n * Some other strategy\n\nThoughts?\n\n-jay","accounts_in_message":[],"_revision_number":2},{"id":"39d77cb94625affc046790b4b54d0284446d73bf","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2012-03-09 02:40:40.000000000","message":"Patch Set 2:\n\nJay:  I thought naming the cookbook \u0027swift-aio\u0027 would make things clear, but are you thinking that changing the name to SAIO would help?  I\u0027d be happy to ask on the mailing list if you do.","accounts_in_message":[],"_revision_number":2},{"id":"ec49b4e90167309d99962019622a3a82b7844433","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2012-03-09 02:51:04.000000000","message":"Patch Set 2: Restored\n\nStill working on this...","accounts_in_message":[],"_revision_number":2},{"id":"c54444d6d62cbcdc787fb7de8253eff3b4741243","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2012-03-16 06:03:02.000000000","message":"Patch Set 2:\n\ncode review expired after 1 week of no activity after a negative review","accounts_in_message":[],"_revision_number":2},{"id":"485305f5985bd1939799a34014e8ebc44dad25b9","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2012-03-16 06:03:02.000000000","message":"Patch Set 2: Abandoned\n\ncode review expired after 1 week of no activity after a negative review","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"20e34c9ee2fee7dd024e5ca80a11ca5e36cbee28","revisions":{"4e3609d4e2bf664be33f974add6de90e40ff1eb9":{"kind":"REWORK","_number":1,"created":"2012-02-01 04:44:51.000000000","uploader":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"ref":"refs/changes/13/3613/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-chef","ref":"refs/changes/13/3613/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-chef refs/changes/13/3613/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-chef refs/changes/13/3613/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-chef refs/changes/13/3613/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-chef refs/changes/13/3613/1"}}},"commit":{"parents":[{"commit":"61e6efcf0605015e140b9999ffccd323433091d3","subject":"Add nouse_cow_images option for nova.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-chef/commit/61e6efcf0605015e140b9999ffccd323433091d3"}]}],"author":{"name":"Maru Newby","email":"mnewby@internap.com","date":"2012-02-01 04:41:00.000000000","tz":-480},"committer":{"name":"Maru Newby","email":"mnewby@internap.com","date":"2012-02-01 04:41:00.000000000","tz":-480},"subject":"Add cookbook for Swift All In One.","message":"Add cookbook for Swift All In One.\n\n* Added a new recipe, swift-aio, that implements the OpenStack instructions\n  for creating a Swift All In One:\n\n  http://swift.openstack.org/development_saio.html\n\n  Where possible the recipes in the cookbook follow the logical flow of\n  the instructions and the resulting environment should be exactly the\n  same as if the instructions where executed manually.\n\n* Updated the apt::default recipe to reflect the latest from\n  opscode, and added a resource that allows forcing of an apt-get update\n  once the swift-core ppa repo has been configured on the system.\n\nChange-Id: I9f0c3006aae6c20cda0ac1eae375ea836f3ee4bd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-chef/commit/4e3609d4e2bf664be33f974add6de90e40ff1eb9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-chef/commit/4e3609d4e2bf664be33f974add6de90e40ff1eb9"}]},"branch":"refs/heads/master"},"20e34c9ee2fee7dd024e5ca80a11ca5e36cbee28":{"kind":"REWORK","_number":2,"created":"2012-02-15 01:47:57.000000000","uploader":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"ref":"refs/changes/13/3613/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-chef","ref":"refs/changes/13/3613/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-chef refs/changes/13/3613/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-chef refs/changes/13/3613/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-chef refs/changes/13/3613/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-chef refs/changes/13/3613/2"}}},"commit":{"parents":[{"commit":"47af02987490ba2344a702b806b293917e20bb5d","subject":"Add ability to pass in boolean flags to extra_config in nova.conf template.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-chef/commit/47af02987490ba2344a702b806b293917e20bb5d"}]}],"author":{"name":"Maru Newby","email":"mnewby@internap.com","date":"2012-02-01 04:41:00.000000000","tz":-480},"committer":{"name":"Maru Newby","email":"mnewby@internap.com","date":"2012-02-15 01:47:55.000000000","tz":-480},"subject":"Add cookbook for Swift All In One.","message":"Add cookbook for Swift All In One.\n\n* Added a new recipe, swift-aio, that implements the OpenStack instructions\n  for creating a Swift All In One:\n\n  http://swift.openstack.org/development_saio.html\n\n  Where possible the recipes in the cookbook follow the logical flow of\n  the instructions and the resulting environment should be exactly the\n  same as if the instructions where executed manually.\n\n* Updated the apt::default recipe to reflect the latest from\n  opscode, and added a resource that allows forcing of an apt-get update\n  once the swift-core ppa repo has been configured on the system.\n\nChange-Id: I9f0c3006aae6c20cda0ac1eae375ea836f3ee4bd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-chef/commit/20e34c9ee2fee7dd024e5ca80a11ca5e36cbee28"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-chef/commit/20e34c9ee2fee7dd024e5ca80a11ca5e36cbee28"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}