)]}'
{"id":"openstack%2Fdevstack~401217","triplet_id":"openstack%2Fdevstack~master~I70d597fbe4a8923d937ba8432e29edefb27d1058","project":"openstack/devstack","branch":"master","topic":"reload-restart","hashtags":[],"change_id":"I70d597fbe4a8923d937ba8432e29edefb27d1058","subject":"Apache reload issue when it is stopped","status":"MERGED","created":"2016-11-23 11:50:42.000000000","updated":"2016-11-27 23:25:10.000000000","submitted":"2016-11-27 23:25:09.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"57354eb98717a6e47941ad89e53fa42dd651cfcd","_number":401217,"virtual_id_number":401217,"owner":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":16803,"name":"liujunpeng","email":"liujunpeng@inspur.com","username":"liujunpeng"},{"value":2,"date":"2016-11-27 23:25:09.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":16803,"name":"liujunpeng","email":"liujunpeng@inspur.com","username":"liujunpeng"},"all":[{"value":1,"date":"2016-11-26 08:48:37.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":16803,"name":"liujunpeng","email":"liujunpeng@inspur.com","username":"liujunpeng"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2016-11-23 14:30:38.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":2,"date":"2016-11-27 22:18:17.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"all":[{"value":0,"_account_id":16803,"name":"liujunpeng","email":"liujunpeng@inspur.com","username":"liujunpeng"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":1,"date":"2016-11-27 22:18:17.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":16803,"name":"liujunpeng","email":"liujunpeng@inspur.com","username":"liujunpeng"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":16803,"name":"liujunpeng","email":"liujunpeng@inspur.com","username":"liujunpeng"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-11-23 14:30:38.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"},{"updated":"2016-11-26 08:48:37.000000000","updated_by":{"_account_id":16803,"name":"liujunpeng","email":"liujunpeng@inspur.com","username":"liujunpeng"},"reviewer":{"_account_id":16803,"name":"liujunpeng","email":"liujunpeng@inspur.com","username":"liujunpeng"},"state":"REVIEWER"},{"updated":"2016-11-27 22:18:17.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"},{"updated":"2016-11-27 23:25:09.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"5550acc01a386ef30ee01e0e87b1270bfe82c8fb","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2016-11-23 11:50:42.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b5e5c1f4affdf039844caa7688e5a34c376c4bf3","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2016-11-23 11:56:09.000000000","message":"Patch Set 1:\n\nsystemctl has reload-or-restart action, it might be better to use.","accounts_in_message":[],"_revision_number":1},{"id":"288599b8908b3cd7f9b9837027b730274916531f","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2016-11-23 12:26:31.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"ed043857ba97aa5d5e8ba7c01634f4bf924df505","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2016-11-23 13:35:35.000000000","message":"Patch Set 2:\n\nTesting completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. Contact info: kvmpower@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/devstack/17/401217/2/check/tempest-dsvm-full-xenial/035de08/ : SUCCESS in 1h 08m 38s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"46385b3f42a6c496f4794e9cd6ff79385639d8fb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-23 14:11:44.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/17/401217/2/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bafc233/ : SUCCESS in 48m 43s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/17/401217/2/check/gate-grenade-dsvm-neutron-ubuntu-xenial/8e64c41/ : SUCCESS in 52m 59s\n- gate-grenade-dsvm-ubuntu-xenial http://logs.openstack.org/17/401217/2/check/gate-grenade-dsvm-ubuntu-xenial/4ef2605/ : SUCCESS in 27m 44s\n- gate-devstack-docs-ubuntu-xenial http://docs-draft.openstack.org/17/401217/2/check/gate-devstack-docs-ubuntu-xenial/bdecc0a//doc/build/html/ : SUCCESS in 2m 22s\n- gate-devstack-bashate http://logs.openstack.org/17/401217/2/check/gate-devstack-bashate/3341888/ : SUCCESS in 1m 23s\n- gate-devstack-unit-tests http://logs.openstack.org/17/401217/2/check/gate-devstack-unit-tests/3452faa/ : SUCCESS in 2m 48s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv http://logs.openstack.org/17/401217/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/99b97ad/ : SUCCESS in 40m 03s (non-voting)\n- gate-devstack-dsvm-updown-ubuntu-xenial http://logs.openstack.org/17/401217/2/check/gate-devstack-dsvm-updown-ubuntu-xenial/04bd5dd/ : SUCCESS in 18m 17s\n- gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv http://logs.openstack.org/17/401217/2/check/gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv/1664778/ : SUCCESS in 51m 39s (non-voting)\n- gate-swift-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/17/401217/2/check/gate-swift-dsvm-functional-ubuntu-xenial-nv/874bebf/ : SUCCESS in 20m 09s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial http://logs.openstack.org/17/401217/2/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial/ebe1b1e/ : SUCCESS in 1h 27m 46s (non-voting)\n- gate-grenade-dsvm-neutron-multinode-ubuntu-xenial http://logs.openstack.org/17/401217/2/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/83ad13a/ : SUCCESS in 40m 11s\n- gate-tempest-dsvm-platform-centos7-nv http://logs.openstack.org/17/401217/2/check/gate-tempest-dsvm-platform-centos7-nv/66afa97/ : FAILURE in 1h 43m 07s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"03488cdfcd1098069e22f8b2c2390fb590fdb7d7","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2016-11-23 14:30:38.000000000","message":"Patch Set 2: Code-Review+2\n\nToo funny, the original code was service $1 stop;service $1 start for this exact reason... :)","accounts_in_message":[],"_revision_number":2},{"id":"15d8096950c5299ccd2774fc39036cfdca6c9210","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2016-11-23 16:00:06.000000000","message":"Patch Set 2:\n\nBuild succeeded.\n\n- dsvm-tempest-neutron-network http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/17/401217/2/check/dsvm-tempest-neutron-network/ecc1bf3 : SUCCESS in 2h 21m 11s","accounts_in_message":[],"_revision_number":2},{"id":"21c10e8cffe8f1049502056a71df304375a49722","author":{"_account_id":16803,"name":"liujunpeng","email":"liujunpeng@inspur.com","username":"liujunpeng"},"date":"2016-11-26 08:48:37.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"6d0792b208627c25233a0e73fd135a48e610606d","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2016-11-27 22:18:17.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nahh systemd ... TIL about reload-or-restart.  that seems to be the logical thing for \"reload\" to do by itself, but anyway","accounts_in_message":[],"_revision_number":2},{"id":"0e642ef32fc20077dbd8ed0589dfca1be5a9d999","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-27 22:18:27.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"f0da0c9bc3f984e0af882f80e1364e8b7923a1fc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-27 23:25:09.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/17/401217/2/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/3510558/ : SUCCESS in 47m 59s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/17/401217/2/gate/gate-grenade-dsvm-neutron-ubuntu-xenial/b82812c/ : SUCCESS in 1h 06m 14s\n- gate-grenade-dsvm-ubuntu-xenial http://logs.openstack.org/17/401217/2/gate/gate-grenade-dsvm-ubuntu-xenial/1ea5733/ : SUCCESS in 27m 26s\n- gate-devstack-docs-ubuntu-xenial http://docs-draft.openstack.org/17/401217/2/gate/gate-devstack-docs-ubuntu-xenial/cecaf94//doc/build/html/ : SUCCESS in 2m 35s\n- gate-devstack-bashate http://logs.openstack.org/17/401217/2/gate/gate-devstack-bashate/78cc1a7/ : SUCCESS in 1m 46s\n- gate-grenade-dsvm-neutron-multinode-ubuntu-xenial http://logs.openstack.org/17/401217/2/gate/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/9818261/ : SUCCESS in 1h 06m 39s","accounts_in_message":[],"_revision_number":2},{"id":"c2b5c357b03b3870eaeefe11fe5b2a98543cbafb","date":"2016-11-27 23:25:10.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"bcaadd63d8739b528f0121775d6612aebd333a4f","revisions":{"881e1aed8a1a7559da50c30785ef1d4f3d1a23ab":{"kind":"REWORK","_number":1,"created":"2016-11-23 11:50:42.000000000","uploader":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"ref":"refs/changes/17/401217/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/17/401217/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/401217/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/401217/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/401217/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/17/401217/1"}}},"commit":{"parents":[{"commit":"a3bb131c138859367f4529720121fd3eb8e239fd","subject":"Merge \"Remove unused KEYSTONE_CATALOG_BACKEND\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a3bb131c138859367f4529720121fd3eb8e239fd"}]}],"author":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2016-11-23 11:43:02.000000000","tz":60},"committer":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2016-11-23 11:43:02.000000000","tz":60},"subject":"Apache fallback from reload","message":"Apache fallback from reload\n\nSince 4b49e409f853104dae021dfca1a9342ec9ac4709 devstack\nstarted to use reload instead of restart.\n\nUsing reload in devstack,\ndoes not seams to be the wisest idea,\nbecause we do not need greacful restart here.\n\nSystemd also reject to reload something,\nwhich was not loaded before.\n\n $ sudo /bin/systemctl reload httpd\n httpd.service is not active, cannot reload.\n\nThis change just adds a fallback to restart,\nin case the reload got rejected for whatever reason.\n\nChange-Id: I70d597fbe4a8923d937ba8432e29edefb27d1058\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/881e1aed8a1a7559da50c30785ef1d4f3d1a23ab"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/881e1aed8a1a7559da50c30785ef1d4f3d1a23ab"}]},"branch":"refs/heads/master"},"bcaadd63d8739b528f0121775d6612aebd333a4f":{"kind":"REWORK","_number":2,"created":"2016-11-23 12:26:31.000000000","uploader":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"ref":"refs/changes/17/401217/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/17/401217/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/401217/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/401217/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/401217/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/17/401217/2"}}},"commit":{"parents":[{"commit":"a3bb131c138859367f4529720121fd3eb8e239fd","subject":"Merge \"Remove unused KEYSTONE_CATALOG_BACKEND\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a3bb131c138859367f4529720121fd3eb8e239fd"}]}],"author":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2016-11-23 11:43:02.000000000","tz":60},"committer":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2016-11-23 12:20:33.000000000","tz":60},"subject":"Apache reload issue when it is stopped","message":"Apache reload issue when it is stopped\n\nSince 4b49e409f853104dae021dfca1a9342ec9ac4709 devstack\nstarted to use reload instead of restart.\n\nUsing reload in devstack for a fresh install,\ndoes not makes too much sense unless multiple service\nplugin touches the same service configs.\n\nSystemd rejects to reload something,\nwhich was not loaded before.\n\n $ sudo /bin/systemctl reload httpd\n httpd.service is not active, cannot reload.\n\nWe will switch to `reload-or-restart` action instead of `reload`,\nit is more likely the action what the previous patch wanted.\n\nChange-Id: I70d597fbe4a8923d937ba8432e29edefb27d1058\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/bcaadd63d8739b528f0121775d6612aebd333a4f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/bcaadd63d8739b528f0121775d6612aebd333a4f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}