)]}'
{"id":"openstack%2Fcookbook-openstack-compute~674283","triplet_id":"openstack%2Fcookbook-openstack-compute~stable%2Fqueens~Id9f1d165411fc5b3cc73b29a36840cf4dc63d81b","project":"openstack/cookbook-openstack-compute","branch":"stable/queens","topic":"disable_vanilla_placement_queens","hashtags":[],"change_id":"Id9f1d165411fc5b3cc73b29a36840cf4dc63d81b","subject":"placement_api: create valid apache config before installing package","status":"MERGED","created":"2019-08-02 12:00:33.000000000","updated":"2019-08-06 13:05:14.000000000","submitted":"2019-08-06 13:05:14.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"674283-1565096714416-520616fe","meta_rev_id":"41ab166512073ae138ac0d3b038332ecc3a8665c","_number":674283,"virtual_id_number":674283,"owner":{"_account_id":11109,"name":"Roger Luethi","email":"rl@patchworkscience.org","username":"rluethi"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":0,"_account_id":19193,"name":"Christoph Albers","email":"c.albers@x-ion.de","username":"calbers"},{"value":2,"date":"2019-08-06 13:05:14.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"all":[{"value":2,"date":"2019-08-06 12:55:11.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":2,"date":"2019-08-06 08:11:53.000000000","_account_id":19193,"name":"Christoph Albers","email":"c.albers@x-ion.de","username":"calbers"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"all":[{"value":1,"date":"2019-08-06 12:55:11.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":0,"_account_id":19193,"name":"Christoph Albers","email":"c.albers@x-ion.de","username":"calbers"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"_account_id":19193,"name":"Christoph Albers","email":"c.albers@x-ion.de","username":"calbers"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-08-06 08:11:53.000000000","updated_by":{"_account_id":19193,"name":"Christoph Albers","email":"c.albers@x-ion.de","username":"calbers"},"reviewer":{"_account_id":19193,"name":"Christoph Albers","email":"c.albers@x-ion.de","username":"calbers"},"state":"REVIEWER"},{"updated":"2019-08-06 12:55:11.000000000","updated_by":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"reviewer":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"state":"REVIEWER"},{"updated":"2019-08-06 13:05:14.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"46a12daba12c3b5b98ea5f8361e1bbaeceadbe92","author":{"_account_id":11109,"name":"Roger Luethi","email":"rl@patchworkscience.org","username":"rluethi"},"date":"2019-08-02 12:00:33.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6643a0d6a3bc636ac82ff48db1ae5081ee83880e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-02 13:09:05.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-chef-delivery https://logs.opendev.org/83/674283/1/check/openstack-chef-delivery/3f9cdda/ : FAILURE in 6m 42s\n- openstack-chef-integration-bionic https://logs.opendev.org/83/674283/1/check/openstack-chef-integration-bionic/547ce76/ : SUCCESS in 53m 59s (non-voting)\n- openstack-chef-integration-xenial https://logs.opendev.org/83/674283/1/check/openstack-chef-integration-xenial/3584993/ : SUCCESS in 41m 19s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"6070f3e9bdea85261189b66c60ae17e590c4caf2","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2019-08-02 21:40:22.000000000","message":"Patch Set 2: Patch Set 1 was rebased","accounts_in_message":[],"_revision_number":2},{"id":"7190887463da6196274a9a61168b7a85093cb0d5","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-02 22:22:27.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-chef-delivery https://logs.opendev.org/83/674283/2/check/openstack-chef-delivery/bc4fd27/ : SUCCESS in 7m 13s\n- openstack-chef-integration-bionic https://logs.opendev.org/83/674283/2/check/openstack-chef-integration-bionic/28caff0/ : SUCCESS in 35m 45s (non-voting)\n- openstack-chef-integration-xenial https://logs.opendev.org/83/674283/2/check/openstack-chef-integration-xenial/f471fce/ : SUCCESS in 37m 11s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"e6b9fd6a770e13b7ec1e814e9b14aa1036952aad","author":{"_account_id":19193,"name":"Christoph Albers","email":"c.albers@x-ion.de","username":"calbers"},"date":"2019-08-06 08:11:53.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"f2a6564ba5a77e1b4c0a9549dc468490e8f4c19b","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2019-08-06 12:55:11.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"cfddfcc3b992019a48be5b6eba2812e1d430a839","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-06 12:55:23.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"3ae8a76e07aa27a0bf6983c80f1fa13b29616308","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-06 13:05:14.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-chef-delivery https://logs.opendev.org/83/674283/2/gate/openstack-chef-delivery/a0a753f/ : SUCCESS in 8m 20s","accounts_in_message":[],"_revision_number":2},{"id":"71c3204aa96d59bd0b561ba987ba608c26bb251a","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-06 13:05:14.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"33aa18a008d0f35cd628edc3dceaf2a17712fb52","revisions":{"475ddd6936a562e319d4b0c1e0296222b96d0eea":{"kind":"REWORK","_number":1,"created":"2019-08-02 12:00:33.000000000","uploader":{"_account_id":11109,"name":"Roger Luethi","email":"rl@patchworkscience.org","username":"rluethi"},"ref":"refs/changes/83/674283/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cookbook-openstack-compute","ref":"refs/changes/83/674283/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cookbook-openstack-compute refs/changes/83/674283/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cookbook-openstack-compute refs/changes/83/674283/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cookbook-openstack-compute refs/changes/83/674283/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cookbook-openstack-compute refs/changes/83/674283/1"}}},"commit":{"parents":[{"commit":"a6f21ae978a453281b48112cb9b8dd2305c181f2","subject":"OpenDev Migration Patch","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cookbook-openstack-compute/commit/a6f21ae978a453281b48112cb9b8dd2305c181f2"}]}],"author":{"name":"Roger Luethi","email":"rl@patchworkscience.org","date":"2019-07-16 13:50:46.000000000","tz":120},"committer":{"name":"Roger Luethi","email":"rl@patchworkscience.org","date":"2019-08-02 11:58:24.000000000","tz":120},"subject":"placement_api: create valid apache config before installing package","message":"placement_api: create valid apache config before installing package\n\nIf apache is (re)starting while the chef-client installs the\nnova-placement-api package, it may pick up the vanilla apache2 conf file\nbefore chef-client gets around to fixing it which may result in apache2\nfailing due to a non-working configuration.\n\nThis changeset eliminates the race by creating a valid configuration\nbefore installing the software. The solution is based on the assumption\nthat no reasonably configured package manager will overwrite an\nexisting, user-created configuration file.\n\nbackport: queens\n\nChange-Id: Id9f1d165411fc5b3cc73b29a36840cf4dc63d81b\n(cherry picked from commit ff074af903390c07e07514df7ffc00a6bfc1475e)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cookbook-openstack-compute/commit/475ddd6936a562e319d4b0c1e0296222b96d0eea"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cookbook-openstack-compute/commit/475ddd6936a562e319d4b0c1e0296222b96d0eea"}]},"parents_data":[{"branch_name":"refs/heads/stable/queens","commit_id":"a6f21ae978a453281b48112cb9b8dd2305c181f2","is_merged_in_target_branch":true}],"branch":"refs/heads/stable/queens"},"33aa18a008d0f35cd628edc3dceaf2a17712fb52":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2019-08-02 21:40:22.000000000","uploader":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"ref":"refs/changes/83/674283/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cookbook-openstack-compute","ref":"refs/changes/83/674283/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cookbook-openstack-compute refs/changes/83/674283/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cookbook-openstack-compute refs/changes/83/674283/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cookbook-openstack-compute refs/changes/83/674283/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cookbook-openstack-compute refs/changes/83/674283/2"}}},"commit":{"parents":[{"commit":"421ae68912782cd7f1dcfd74b7acb2c69865096b","subject":"Fix Berksfile to request stable/queens branch","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cookbook-openstack-compute/commit/421ae68912782cd7f1dcfd74b7acb2c69865096b"}]}],"author":{"name":"Roger Luethi","email":"rl@patchworkscience.org","date":"2019-07-16 13:50:46.000000000","tz":120},"committer":{"name":"Jens Harbott (frickler)","email":"j.harbott@x-ion.de","date":"2019-08-02 21:40:22.000000000","tz":0},"subject":"placement_api: create valid apache config before installing package","message":"placement_api: create valid apache config before installing package\n\nIf apache is (re)starting while the chef-client installs the\nnova-placement-api package, it may pick up the vanilla apache2 conf file\nbefore chef-client gets around to fixing it which may result in apache2\nfailing due to a non-working configuration.\n\nThis changeset eliminates the race by creating a valid configuration\nbefore installing the software. The solution is based on the assumption\nthat no reasonably configured package manager will overwrite an\nexisting, user-created configuration file.\n\nbackport: queens\n\nChange-Id: Id9f1d165411fc5b3cc73b29a36840cf4dc63d81b\n(cherry picked from commit ff074af903390c07e07514df7ffc00a6bfc1475e)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cookbook-openstack-compute/commit/33aa18a008d0f35cd628edc3dceaf2a17712fb52"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cookbook-openstack-compute/commit/33aa18a008d0f35cd628edc3dceaf2a17712fb52"}]},"parents_data":[{"branch_name":"refs/heads/stable/queens","commit_id":"421ae68912782cd7f1dcfd74b7acb2c69865096b","is_merged_in_target_branch":true,"change_id":"I7ae5cb83f6bd3774c978f3152a137c20fe7ab20c","change_number":674415,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/stable/queens"}},"requirements":[],"submit_records":[],"submit_requirements":[]}