)]}'
{"id":"openstack%2Fopenstack-ansible-security~326705","triplet_id":"openstack%2Fopenstack-ansible-security~master~I724d6eb3b716bd9b0006d0d2e5ad201481d52e59","project":"openstack/openstack-ansible-security","branch":"master","topic":"bug/1590102","hashtags":[],"change_id":"I724d6eb3b716bd9b0006d0d2e5ad201481d52e59","subject":"Fix grub configuration file path in RHEL/CentOS 7","status":"MERGED","created":"2016-06-07 19:14:08.000000000","updated":"2016-06-08 10:05:36.000000000","submitted":"2016-06-08 10:05:36.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"139a462e6ac29f9bc8f2f9682dd545c05787a339","_number":326705,"virtual_id_number":326705,"owner":{"_account_id":21656,"name":"Mark Hooper","email":"hoopernet@gmail.com","username":"h00pz"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},{"value":0,"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"value":0,"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},{"value":0,"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":2,"date":"2016-06-08 10:05:36.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"recommended":{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},"all":[{"value":1,"date":"2016-06-07 20:05:20.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},{"value":2,"date":"2016-06-07 19:49:12.000000000","_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"value":0,"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},{"value":2,"date":"2016-06-08 10:01:34.000000000","_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":{"approved":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"all":[{"value":0,"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},{"value":0,"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"value":0,"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},{"value":1,"date":"2016-06-08 10:01:34.000000000","_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-06-07 19:14:14.000000000","updated_by":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"reviewer":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"state":"REVIEWER"},{"updated":"2016-06-07 19:45:43.000000000","updated_by":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"reviewer":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"state":"REVIEWER"},{"updated":"2016-06-07 19:49:12.000000000","updated_by":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"reviewer":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"state":"REVIEWER"},{"updated":"2016-06-07 20:05:20.000000000","updated_by":{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},"reviewer":{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},"state":"REVIEWER"},{"updated":"2016-06-08 10:01:34.000000000","updated_by":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"reviewer":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"state":"REVIEWER"},{"updated":"2016-06-08 10:05:36.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"984ea73f15bf9f49c842eee1e87163a422d65651","author":{"_account_id":21656,"name":"Mark Hooper","email":"hoopernet@gmail.com","username":"h00pz"},"date":"2016-06-07 19:14:08.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"41c05cfb2dd7dbc32fde72d3d9a6863afe77bc74","author":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"date":"2016-06-07 19:14:14.000000000","message":"Patch Set 1:\n\nThank you for your first contribution to OpenStack.\n\n    Your patch will now be tested automatically by OpenStack testing frameworks\n    and once the automatic tests pass, it will be reviewed by other friendly\n    developers. They will give you feedback and may require you to refine it.\n\n    People seldom get their patch approved on the first try, so don\u0027t be\n    concerned if requested to make corrections. Feel free to modify your patch\n    and resubmit a new change-set.\n\n    Patches usually take 3 to 7 days to be reviewed so be patient and be\n    available on IRC to ask and answer questions about your work. Also it\n    takes generally at least a couple of weeks for cores to get around to\n    reviewing code. The more you participate in the community the more\n    rewarding it is for you. You may also notice that the more you get to know\n    people and get to be known, the faster your patches will be reviewed and\n    eventually approved. Get to know others and become known by doing code\n    reviews: anybody can do it, and it\u0027s a great way to learn the code base.\n\n    Thanks again for supporting OpenStack, we look forward to working with you.\n\n    IRC: https://wiki.openstack.org/wiki/IRC\n    Workflow: http://docs.openstack.org/infra/manual/developers.html\n    Commit Messages: https://wiki.openstack.org/wiki/GitCommitMessages","accounts_in_message":[],"_revision_number":1},{"id":"0c16a14af0f3fb0f14c16b397463583351dee3c1","author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"date":"2016-06-07 19:31:56.000000000","message":"Patch Set 1: Code-Review-1\n\n(4 comments)","accounts_in_message":[],"_revision_number":1},{"id":"2eb791df372ef313f20475278c94f873b7c92ae7","author":{"_account_id":21656,"name":"Mark Hooper","email":"hoopernet@gmail.com","username":"h00pz"},"date":"2016-06-07 19:44:12.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"00ce4423cd14cec1af6d690954a7360604453fc8","author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"date":"2016-06-07 19:45:43.000000000","message":"Patch Set 2: Code-Review+1\n\nThanks for the quick turnaround on Major\u0027s feedback. Looks good, but waiting for Gerrit to report in before I consider a +2","accounts_in_message":[],"_revision_number":2},{"id":"87961adf98f5e411bf422e8d18105a2120b23db6","author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"date":"2016-06-07 19:48:14.000000000","message":"Patch Set 3: Published edit on patch set 2","accounts_in_message":[],"_revision_number":3},{"id":"5fbe374b89377ab17b65ef86de111c2f1cc5a065","author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"date":"2016-06-07 19:49:12.000000000","message":"Patch Set 3: Code-Review+2\n\nLooks good! :)","accounts_in_message":[],"_revision_number":3},{"id":"18b77c7716b94438d41acf2e6b5d8f383c2f8f97","author":{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},"date":"2016-06-07 20:05:20.000000000","message":"Patch Set 3: Code-Review+1\n\nWelcome!","accounts_in_message":[],"_revision_number":3},{"id":"6c27e25912ea94821d37ea8a11c3a1e6dec974cb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-07 20:22:48.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-openstack-ansible-security-docs http://docs-draft.openstack.org/05/326705/3/check/gate-openstack-ansible-security-docs/8a1c414//doc/build/html/ : SUCCESS in 6m 16s\n- gate-openstack-ansible-security-linters http://logs.openstack.org/05/326705/3/check/gate-openstack-ansible-security-linters/81bec2b/ : SUCCESS in 1m 29s\n- gate-openstack-ansible-security-ansible-func-centos-7-nv http://logs.openstack.org/05/326705/3/check/gate-openstack-ansible-security-ansible-func-centos-7-nv/bf4de9c/ : SUCCESS in 2m 38s (non-voting)\n- gate-openstack-ansible-security-ansible-func-ubuntu-trusty http://logs.openstack.org/05/326705/3/check/gate-openstack-ansible-security-ansible-func-ubuntu-trusty/c724af8/ : SUCCESS in 2m 05s\n- gate-openstack-ansible-security-ansible-func-ubuntu-xenial-nv http://logs.openstack.org/05/326705/3/check/gate-openstack-ansible-security-ansible-func-ubuntu-xenial-nv/383cf89/ : SUCCESS in 3m 10s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"ee60426a35aa45c6f9f09cc890daf7ad98f09b9e","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2016-06-08 10:01:34.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"329966c35d956a860504b4b5478cb9c2f5c9633f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-08 10:01:39.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"16b9ba352681e465ea6e7bf068713dbc576194f8","date":"2016-06-08 10:05:36.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":3},{"id":"39d4cf4f890e2c1fcc9071551698a1808a13f056","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-08 10:05:36.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-openstack-ansible-security-docs http://docs-draft.openstack.org/05/326705/3/gate/gate-openstack-ansible-security-docs/4569ee4//doc/build/html/ : SUCCESS in 3m 27s\n- gate-openstack-ansible-security-linters http://logs.openstack.org/05/326705/3/gate/gate-openstack-ansible-security-linters/d5f185a/ : SUCCESS in 1m 07s\n- gate-openstack-ansible-security-ansible-func-ubuntu-trusty http://logs.openstack.org/05/326705/3/gate/gate-openstack-ansible-security-ansible-func-ubuntu-trusty/d2d290b/ : SUCCESS in 1m 23s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"03d436f86403d5635759ed4dcfc2d36567aeca20","revisions":{"3b2362f5bf42be8d8ec688de23c9bb3f5809a5be":{"kind":"REWORK","_number":1,"created":"2016-06-07 19:14:08.000000000","uploader":{"_account_id":21656,"name":"Mark Hooper","email":"hoopernet@gmail.com","username":"h00pz"},"ref":"refs/changes/05/326705/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-security","ref":"refs/changes/05/326705/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/1"}}},"commit":{"parents":[{"commit":"41260da8ac0f9aff3644a142e14b39ac33a1b9d6","subject":"Merge \"Ensure V-38574 works reliably on CentOS\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-security/commit/41260da8ac0f9aff3644a142e14b39ac33a1b9d6"}]}],"author":{"name":"Mark Hooper","email":"hoopernet@gmail.com","date":"2016-06-07 19:03:04.000000000","tz":-300},"committer":{"name":"Mark Hooper","email":"hoopernet@gmail.com","date":"2016-06-07 19:03:04.000000000","tz":-300},"subject":"adding grub conf variable since RHEL 7 is now using /boot/grub2 instead of /boot/grub","message":"adding grub conf variable since RHEL 7 is now using /boot/grub2 instead of /boot/grub\n\nChange-Id: I724d6eb3b716bd9b0006d0d2e5ad201481d52e59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-security/commit/3b2362f5bf42be8d8ec688de23c9bb3f5809a5be"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-security/commit/3b2362f5bf42be8d8ec688de23c9bb3f5809a5be"}]},"branch":"refs/heads/master"},"5ca88bce763e6c65c41d4097679ec8f8876704f3":{"kind":"REWORK","_number":2,"created":"2016-06-07 19:44:12.000000000","uploader":{"_account_id":21656,"name":"Mark Hooper","email":"hoopernet@gmail.com","username":"h00pz"},"ref":"refs/changes/05/326705/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-security","ref":"refs/changes/05/326705/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/2"}}},"commit":{"parents":[{"commit":"41260da8ac0f9aff3644a142e14b39ac33a1b9d6","subject":"Merge \"Ensure V-38574 works reliably on CentOS\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-security/commit/41260da8ac0f9aff3644a142e14b39ac33a1b9d6"}]}],"author":{"name":"Mark Hooper","email":"hoopernet@gmail.com","date":"2016-06-07 19:03:04.000000000","tz":-300},"committer":{"name":"Mark Hooper","email":"hoopernet@gmail.com","date":"2016-06-07 19:40:41.000000000","tz":-300},"subject":"adding grub conf variable since RHEL 7 is now using /boot/grub2 instead of /boot/grub","message":"adding grub conf variable since RHEL 7 is now using /boot/grub2 instead of /boot/grub\n\nChange-Id: I724d6eb3b716bd9b0006d0d2e5ad201481d52e59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-security/commit/5ca88bce763e6c65c41d4097679ec8f8876704f3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-security/commit/5ca88bce763e6c65c41d4097679ec8f8876704f3"}]},"branch":"refs/heads/master"},"03d436f86403d5635759ed4dcfc2d36567aeca20":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2016-06-07 19:48:14.000000000","uploader":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"ref":"refs/changes/05/326705/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-security","ref":"refs/changes/05/326705/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-security refs/changes/05/326705/3"}}},"commit":{"parents":[{"commit":"41260da8ac0f9aff3644a142e14b39ac33a1b9d6","subject":"Merge \"Ensure V-38574 works reliably on CentOS\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-security/commit/41260da8ac0f9aff3644a142e14b39ac33a1b9d6"}]}],"author":{"name":"Mark Hooper","email":"hoopernet@gmail.com","date":"2016-06-07 19:03:04.000000000","tz":-300},"committer":{"name":"Major Hayden","email":"major@mhtx.net","date":"2016-06-07 19:48:11.000000000","tz":0},"subject":"Fix grub configuration file path in RHEL/CentOS 7","message":"Fix grub configuration file path in RHEL/CentOS 7\n\nCentOS and RHEL 7 both use /boot/grub2/grub.cfg, but the tasks in the\nsecurity role expect it to be in /boot/grub/grub.cfg. This patch\nadds a variable for the grub configuration file path.\n\nCloses-bug: 1590102\n\nChange-Id: I724d6eb3b716bd9b0006d0d2e5ad201481d52e59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-security/commit/03d436f86403d5635759ed4dcfc2d36567aeca20"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-security/commit/03d436f86403d5635759ed4dcfc2d36567aeca20"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}