)]}'
{"id":"openstack%2Fdevstack~44266","triplet_id":"openstack%2Fdevstack~master~I50d475f417a1fe35e74c42e1653b9a8aa9716c37","project":"openstack/devstack","branch":"master","topic":"extra_opt","hashtags":[],"change_id":"I50d475f417a1fe35e74c42e1653b9a8aa9716c37","subject":"Allow setting of EXTRA_OPTs in categories other than DEFAULT","status":"ABANDONED","created":"2013-08-29 11:03:23.000000000","updated":"2013-09-20 06:03:09.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"c6f433455c6b4311175acb4ffbda2921c088ba3a","_number":44266,"virtual_id_number":44266,"owner":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2013-08-29 11:29:42.000000000","_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"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-09-12 15:16:50.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":-1,"date":"2013-09-04 11:08:11.000000000","permitted_voting_range":{"min":-1,"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":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2013-09-12 15:16:50.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"date":"2013-09-04 11:07:34.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,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_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"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-08-29 11:29:42.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2013-09-04 11:07:34.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-12 15:16:50.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"}],"messages":[{"id":"2182ef970efb43ad6f59ca319cb0d26be1e01baf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-29 11:29:42.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/66/44266/1/check/gate-tempest-devstack-vm-full/606a714 : SUCCESS in 25m 52s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/66/44266/1/check/gate-tempest-devstack-vm-postgres-full/f6072f3 : SUCCESS in 26m 16s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/66/44266/1/check/gate-tempest-devstack-vm-neutron/daa893e : SUCCESS in 14m 45s\n- gate-grenade-devstack-vm http://logs.openstack.org/66/44266/1/check/gate-grenade-devstack-vm/496e870 : SUCCESS in 24m 52s\n","accounts_in_message":[],"_revision_number":1},{"id":"b26494e3f69f780d77b6086ef4ee2b408dc0d6d5","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-04 11:07:34.000000000","message":"Patch Set 1:\n\nthis pushes extra opts to a new level of complexity. I know why we need the functionality, but I\u0027m starting to think maybe we should go a different direction and do something instead like a local.nova.conf file which merges in at the end of conf processing.","accounts_in_message":[],"_revision_number":1},{"id":"93b5fe51811ff79cac475c83eb56c463415d4456","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-04 11:08:11.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n-1 for commentary. I\u0027m +2 on the function, just trying to unwind the complexity for config (and chance for configuring wrong).","accounts_in_message":[],"_revision_number":1},{"id":"3a8e5b69e756935d2903bd783084cfeb2ba28c8a","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-09-12 06:03:13.000000000","message":"Patch Set 1: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1},{"id":"a2c9d7c661a8a9b145a43e705cd649570fe1daef","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-09-12 06:03:13.000000000","message":"Patch Set 1:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1},{"id":"dc85917dbd65ba2fa9f7fe780f63c21d4ce0f763","author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"date":"2013-09-12 14:32:30.000000000","message":"Patch Set 1: Restored\n\nThe -1 was for commentry - I think we need more opinions on this change as to whether this is the right way to go or if we want an override snippet and/or using a template as the original input file.","accounts_in_message":[],"_revision_number":1},{"id":"6d74ff56456c7e68e47ffd938792cdb53fa35f8e","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-12 15:16:50.000000000","message":"Patch Set 1:\n\nI think we need to put together a generalized conf file merge that lets users create a local config file and have those settings merged before the services are started.  This probably isn\u0027t the right place to discuss this, I\u0027ll kick off a discussion on the -dev ML.","accounts_in_message":[],"_revision_number":1},{"id":"321f099b28d7a5fdaf173c479bbfc774a08fd061","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-09-20 06:03:09.000000000","message":"Patch Set 1:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1},{"id":"b27e6c0d2ae40f1b50552a84b058b8526b75ba11","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-09-20 06:03:09.000000000","message":"Patch Set 1: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"a44639ab42635c54c446cb11b59ef89b6d75c5ba","revisions":{"a44639ab42635c54c446cb11b59ef89b6d75c5ba":{"kind":"REWORK","_number":1,"created":"2013-08-29 11:03:23.000000000","uploader":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"ref":"refs/changes/66/44266/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/66/44266/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/66/44266/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/66/44266/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/66/44266/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/66/44266/1"}}},"commit":{"parents":[{"commit":"5584709fa8194cb7fa69c65b535999e28278e3da","subject":"Merge \"Fix is_package_installed() check with dpkg\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5584709fa8194cb7fa69c65b535999e28278e3da"}]}],"author":{"name":"Bob Ball","email":"bob.ball@citrix.com","date":"2013-08-29 11:02:16.000000000","tz":60},"committer":{"name":"Bob Ball","email":"bob.ball@citrix.com","date":"2013-08-29 11:02:16.000000000","tz":60},"subject":"Allow setting of EXTRA_OPTs in categories other than DEFAULT","message":"Allow setting of EXTRA_OPTs in categories other than DEFAULT\n\nUseful for trusted_computing category and can be used to replace\nflags such as EXTRA_BAREMETAL_OPTS\n\nChange-Id: I50d475f417a1fe35e74c42e1653b9a8aa9716c37\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a44639ab42635c54c446cb11b59ef89b6d75c5ba"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a44639ab42635c54c446cb11b59ef89b6d75c5ba"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}