)]}'
{"id":"x%2Fpackstack~305881","triplet_id":"x%2Fpackstack~master~Ic8ed069f4991602d125f40eaad8f050284d680e1","project":"x/packstack","branch":"master","topic":"jpena/improvehttpdlisten","hashtags":[],"change_id":"Ic8ed069f4991602d125f40eaad8f050284d680e1","subject":"Simplify apache::listen for services relying on httpd","status":"MERGED","created":"2016-04-14 14:12:09.000000000","updated":"2016-05-05 17:57:48.000000000","submitted":"2016-05-05 17:57:47.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"232cde16b3ca78e6c5c2da8f2747cdfb6c658a4b","_number":305881,"virtual_id_number":305881,"owner":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},{"value":0,"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},{"value":0,"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},{"value":0,"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"value":0,"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"value":2,"date":"2016-05-05 17:57:47.000000000","_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":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"recommended":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"all":[{"value":0,"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},{"value":1,"date":"2016-04-29 12:44:06.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},{"value":0,"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},{"value":0,"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"value":2,"date":"2016-05-05 17:31:11.000000000","_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"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":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"all":[{"value":0,"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},{"value":0,"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},{"value":0,"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},{"value":0,"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"value":1,"date":"2016-05-05 17:31:11.000000000","_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"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":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-04-18 17:30:19.000000000","updated_by":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"reviewer":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"state":"REVIEWER"},{"updated":"2016-04-22 13:03:35.000000000","updated_by":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"reviewer":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"state":"REVIEWER"},{"updated":"2016-04-29 12:44:06.000000000","updated_by":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"reviewer":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"state":"REVIEWER"},{"updated":"2016-05-05 17:31:11.000000000","updated_by":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"reviewer":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"state":"REVIEWER"},{"updated":"2016-05-05 17:57:47.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"bff3fab63b738343636f5e65392dd8561fbdc530","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-14 14:12:09.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"d51e8691d4d334c79ec37c7683fe5c96caedb9f2","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-14 14:18:30.000000000","message":"Patch Set 1:\n\nNote https://review.openstack.org/303307 is addressing a similar issue. I think this is a better alternative, if merged we should abandon the other review.","accounts_in_message":[],"_revision_number":1},{"id":"e2666e00ee3a3e07065459fecc9b95633bcc63b6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-04-14 15:14:27.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-packstack-integration-scenario001-tempest-centos-7 http://logs.openstack.org/81/305881/1/check/gate-packstack-integration-scenario001-tempest-centos-7/b6b3d8a/ : SUCCESS in 20m 16s\n- gate-packstack-integration-scenario002-tempest-centos-7 http://logs.openstack.org/81/305881/1/check/gate-packstack-integration-scenario002-tempest-centos-7/275148f/ : FAILURE in 12m 39s\n- gate-packstack-integration-scenario003-tempest-centos-7 http://logs.openstack.org/81/305881/1/check/gate-packstack-integration-scenario003-tempest-centos-7/67d0f3c/ : SUCCESS in 46m 46s\n- gate-packstack-pep8 http://logs.openstack.org/81/305881/1/check/gate-packstack-pep8/89df8cb/ : SUCCESS in 36s\n- gate-packstack-python27 http://logs.openstack.org/81/305881/1/check/gate-packstack-python27/b9d7639/ : SUCCESS in 53s","accounts_in_message":[],"_revision_number":1},{"id":"1a83143f0249bebd66b2ed6221e19265097d9096","author":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"date":"2016-04-14 16:03:17.000000000","message":"Patch Set 1:\n\ni think this would be even better if apache_ports.pp was added in the postscript plugin","accounts_in_message":[],"_revision_number":1},{"id":"db880297ba9b821300a40867b8ee029caecbddc0","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-14 16:19:27.000000000","message":"Patch Set 1:\n\n\u003e i think this would be even better if apache_ports.pp was added in\n \u003e the postscript plugin\n\nI thought about it, however we need to have Keystone working before other services can do their stuff, so I think our only option is to include it in every manifest using apache.","accounts_in_message":[],"_revision_number":1},{"id":"9c6260a7c14f025a28cdfa82d103d280d0270938","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-14 16:19:49.000000000","message":"Patch Set 1:\n\nrecheck scenario002 failed due to httpd not being able to bind to port 35357 :?","accounts_in_message":[],"_revision_number":1},{"id":"961278cdd5e01922f2668feb366cdeec020776cc","author":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"date":"2016-04-14 17:16:21.000000000","message":"Patch Set 1:\n\nyou\u0027re right i forgot about that","accounts_in_message":[],"_revision_number":1},{"id":"5960cf6e50f2b0f3a77eea73ceb68b993d1df013","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-04-14 17:47:08.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-packstack-integration-scenario001-tempest-centos-7 http://logs.openstack.org/81/305881/1/check/gate-packstack-integration-scenario001-tempest-centos-7/3dfb98f/ : SUCCESS in 32m 05s\n- gate-packstack-integration-scenario002-tempest-centos-7 http://logs.openstack.org/81/305881/1/check/gate-packstack-integration-scenario002-tempest-centos-7/ac590e7/ : SUCCESS in 24m 48s\n- gate-packstack-integration-scenario003-tempest-centos-7 http://logs.openstack.org/81/305881/1/check/gate-packstack-integration-scenario003-tempest-centos-7/7e76529/ : SUCCESS in 34m 02s\n- gate-packstack-pep8 http://logs.openstack.org/81/305881/1/check/gate-packstack-pep8/6e6a299/ : SUCCESS in 50s\n- gate-packstack-python27 http://logs.openstack.org/81/305881/1/check/gate-packstack-python27/347310a/ : SUCCESS in 42s","accounts_in_message":[],"_revision_number":1},{"id":"508b60d11f2417f1e8a6047a28a467e3aa6ad925","author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"date":"2016-04-18 17:30:19.000000000","message":"Patch Set 1: Code-Review+2\n\nlgtm","accounts_in_message":[],"_revision_number":1},{"id":"7f1270f83e499cffa71120c5524e7126c24a1ec7","author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"date":"2016-04-19 13:24:50.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"a5b7318cd743c6b655a71663d605f6e96eb764b2","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-19 14:14:26.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.","accounts_in_message":[],"_revision_number":2},{"id":"e82fcae9a156f8d8229e3a7a1a21ceb94e2f6402","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-04-19 22:51:46.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-packstack-integration-scenario001-tempest-centos-7 http://logs.openstack.org/81/305881/2/check/gate-packstack-integration-scenario001-tempest-centos-7/55ff8b6/ : SUCCESS in 31m 34s\n- gate-packstack-integration-scenario002-tempest-centos-7 http://logs.openstack.org/81/305881/2/check/gate-packstack-integration-scenario002-tempest-centos-7/87b05cf/ : SUCCESS in 27m 43s\n- gate-packstack-integration-scenario003-tempest-centos-7 http://logs.openstack.org/81/305881/2/check/gate-packstack-integration-scenario003-tempest-centos-7/913d559/ : SUCCESS in 21m 29s\n- gate-packstack-pep8 http://logs.openstack.org/81/305881/2/check/gate-packstack-pep8/a6046b5/ : SUCCESS in 27s\n- gate-packstack-python27 http://logs.openstack.org/81/305881/2/check/gate-packstack-python27/5161881/ : SUCCESS in 34s","accounts_in_message":[],"_revision_number":2},{"id":"cee1819a08ad97fb9bf971b3d3e009a595d6d8d0","author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"date":"2016-04-22 13:03:35.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"1081006948cfb42ca8bf47354d7523a843ed4714","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-22 16:56:03.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"c046886cdf04d44d3f62ece06cb296cadd298054","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-22 16:56:14.000000000","message":"Patch Set 2: Workflow-1","accounts_in_message":[],"_revision_number":2},{"id":"027de6e28d619ca671c4b114a2e6abf1191cae60","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-25 14:25:24.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"e1284996165b95535161dfa5ad9aee77db7c7502","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-25 14:28:32.000000000","message":"Patch Set 3:\n\n@mmagr: I\u0027ve managed to fix the Horizon SSL issue. The problem was caused by mod_ssl being installed by horizon.pp, which sets purge_configs \u003d\u003e false when including class ::apache.\n\nTo fix this, I\u0027m pre-installing apache and making sure mod_ssl is installed if needed. In Newton, we don\u0027t need this new plugin, we can remove it and place that logic inside keystone.pp because we are removing support for eventlet. However, I can\u0027t find a better way to do this in Mitaka.","accounts_in_message":[],"_revision_number":3},{"id":"a9eebdf5181cf53ac52f11d4cbe077915ff6507a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-04-25 15:09:14.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-packstack-integration-scenario001-tempest-centos-7 http://logs.openstack.org/81/305881/3/check/gate-packstack-integration-scenario001-tempest-centos-7/acd968f/ : SUCCESS in 35m 51s\n- gate-packstack-integration-scenario002-tempest-centos-7 http://logs.openstack.org/81/305881/3/check/gate-packstack-integration-scenario002-tempest-centos-7/e178b50/ : SUCCESS in 25m 34s\n- gate-packstack-integration-scenario003-tempest-centos-7 http://logs.openstack.org/81/305881/3/check/gate-packstack-integration-scenario003-tempest-centos-7/d7d53bd/ : SUCCESS in 37m 11s\n- gate-packstack-pep8 http://logs.openstack.org/81/305881/3/check/gate-packstack-pep8/4d51ec0/ : FAILURE in 24s\n- gate-packstack-python27 http://logs.openstack.org/81/305881/3/check/gate-packstack-python27/2605bd0/ : FAILURE in 22s","accounts_in_message":[],"_revision_number":3},{"id":"e7585382fec49251c066e5f7f53045b7ba1eb651","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-04-25 15:10:46.000000000","message":"Patch Set 4: Patch Set 3 was rebased","accounts_in_message":[],"_revision_number":3},{"id":"6b7b79fa9c5b86802add4cd5558b8c4c7aaf856f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-04-25 15:47:14.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-packstack-integration-scenario001-tempest-centos-7 http://logs.openstack.org/81/305881/4/check/gate-packstack-integration-scenario001-tempest-centos-7/692ae65/ : SUCCESS in 36m 02s\n- gate-packstack-integration-scenario002-tempest-centos-7 http://logs.openstack.org/81/305881/4/check/gate-packstack-integration-scenario002-tempest-centos-7/ce7bcb1/ : SUCCESS in 29m 48s\n- gate-packstack-integration-scenario003-tempest-centos-7 http://logs.openstack.org/81/305881/4/check/gate-packstack-integration-scenario003-tempest-centos-7/c39f9c8/ : SUCCESS in 36m 19s\n- gate-packstack-pep8 http://logs.openstack.org/81/305881/4/check/gate-packstack-pep8/45812a0/ : SUCCESS in 37s\n- gate-packstack-python27 http://logs.openstack.org/81/305881/4/check/gate-packstack-python27/acc5c70/ : SUCCESS in 31s","accounts_in_message":[],"_revision_number":4},{"id":"f44fa3ad3717386267522c64fec47c149c9bf00a","author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"date":"2016-04-29 11:33:41.000000000","message":"Patch Set 4: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"eb6e6b0c0d07d90648413d80845060c23eb5e665","author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"date":"2016-04-29 12:44:06.000000000","message":"Patch Set 4: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"f98bdfd75f9c47a1bfbacd85f1911e3d7073fcf4","author":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"date":"2016-05-05 17:31:11.000000000","message":"Patch Set 4: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":4},{"id":"339a194a202587f599da7709fe6834a6763141eb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-05 17:31:23.000000000","message":"Patch Set 4: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"949d3a81440c87f8e774be63cc543182f019bdbf","author":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"date":"2016-05-05 17:39:46.000000000","message":"Patch Set 4: Cherry Picked\n\nThis patchset was cherry picked to branch stable/mitaka as commit 773a8e1b4941c34782a42ce4c0d4e462412bb2b9","accounts_in_message":[],"_revision_number":4},{"id":"dd7bf6566e065bd8bb1ac519c32ece25bc24b32e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-05 17:57:47.000000000","message":"Patch Set 4: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-packstack-integration-scenario001-tempest-centos-7 http://logs.openstack.org/81/305881/4/gate/gate-packstack-integration-scenario001-tempest-centos-7/ed2c05d/ : SUCCESS in 24m 44s\n- gate-packstack-integration-scenario002-tempest-centos-7 http://logs.openstack.org/81/305881/4/gate/gate-packstack-integration-scenario002-tempest-centos-7/7a7f2d1/ : SUCCESS in 18m 37s\n- gate-packstack-integration-scenario003-tempest-centos-7 http://logs.openstack.org/81/305881/4/gate/gate-packstack-integration-scenario003-tempest-centos-7/c1186cd/ : SUCCESS in 22m 56s\n- gate-packstack-pep8 http://logs.openstack.org/81/305881/4/gate/gate-packstack-pep8/0bfb29b/ : SUCCESS in 37s\n- gate-packstack-python27 http://logs.openstack.org/81/305881/4/gate/gate-packstack-python27/47164ff/ : SUCCESS in 31s","accounts_in_message":[],"_revision_number":4},{"id":"cf0212ab9c7de64cbee10ee27c6781db81246d3f","date":"2016-05-05 17:57:48.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"68d437f5e2719e86d5268f8bca05abbb3a4cddb2","revisions":{"f87a2e6e7842f47965be3f391ca6f3bb70054f70":{"kind":"REWORK","_number":1,"created":"2016-04-14 14:12:09.000000000","uploader":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"ref":"refs/changes/81/305881/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/81/305881/1","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/81/305881/1"}}},"commit":{"parents":[{"commit":"53cc5ae832d4df74ce3229b578edcc70d1b4b92a","subject":"Merge \"Fix bridge creation for compute nodes when using VLAN\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/53cc5ae832d4df74ce3229b578edcc70d1b4b92a"}]}],"author":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2016-04-14 14:07:03.000000000","tz":120},"committer":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2016-04-14 14:07:03.000000000","tz":120},"subject":"Simplify apache::listen for services relying on httpd","message":"Simplify apache::listen for services relying on httpd\n\nDue to the way Packstack runs Puppet manifests, we are regularly\nhaving issues related to usage of apache::listen. Using it means\nthe existing configuration in ports.conf is removed with every new\nPuppet run, so we need to maintain an ugly hack in services relying\non httpd to ensure no port is lost.\n\nTo simplify this, we are consolidating all apache::listen calls in\none template (apache_ports.pp), which will be included by every\ntemplate requiring it. While it does not fix the design issue, it\nprovides a single place to specify listen ports, and will avoid\nfurther issues.\n\nChange-Id: Ic8ed069f4991602d125f40eaad8f050284d680e1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/f87a2e6e7842f47965be3f391ca6f3bb70054f70"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/f87a2e6e7842f47965be3f391ca6f3bb70054f70"}]},"branch":"refs/heads/master"},"539e7755fac104eb98f712f86b5899ad4561bf89":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2016-04-19 14:14:26.000000000","uploader":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"ref":"refs/changes/81/305881/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/81/305881/2","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/81/305881/2"}}},"commit":{"parents":[{"commit":"37ff27318a98d057931b0c5095bd6ff9dc4c1cf4","subject":"Merge \"Add /etc/hosts and /etc/pip.conf to files retrieved for diagnostic\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/37ff27318a98d057931b0c5095bd6ff9dc4c1cf4"}]}],"author":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2016-04-14 14:07:03.000000000","tz":120},"committer":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2016-04-19 14:03:22.000000000","tz":120},"subject":"Simplify apache::listen for services relying on httpd","message":"Simplify apache::listen for services relying on httpd\n\nDue to the way Packstack runs Puppet manifests, we are regularly\nhaving issues related to usage of apache::listen. Using it means\nthe existing configuration in ports.conf is removed with every new\nPuppet run, so we need to maintain an ugly hack in services relying\non httpd to ensure no port is lost.\n\nTo simplify this, we are consolidating all apache::listen calls in\none template (apache_ports.pp), which will be included by every\ntemplate requiring it. While it does not fix the design issue, it\nprovides a single place to specify listen ports, and will avoid\nfurther issues.\n\nChange-Id: Ic8ed069f4991602d125f40eaad8f050284d680e1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/539e7755fac104eb98f712f86b5899ad4561bf89"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/539e7755fac104eb98f712f86b5899ad4561bf89"}]},"branch":"refs/heads/master"},"fac84378887f5a98ace94a73937fcc6ca26340e9":{"kind":"REWORK","_number":3,"created":"2016-04-25 14:25:24.000000000","uploader":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"ref":"refs/changes/81/305881/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/81/305881/3","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/81/305881/3"}}},"commit":{"parents":[{"commit":"53cc5ae832d4df74ce3229b578edcc70d1b4b92a","subject":"Merge \"Fix bridge creation for compute nodes when using VLAN\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/53cc5ae832d4df74ce3229b578edcc70d1b4b92a"}]}],"author":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2016-04-14 14:07:03.000000000","tz":120},"committer":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2016-04-25 14:25:15.000000000","tz":120},"subject":"Simplify apache::listen for services relying on httpd","message":"Simplify apache::listen for services relying on httpd\n\nDue to the way Packstack runs Puppet manifests, we are regularly\nhaving issues related to usage of apache::listen. Using it means\nthe existing configuration in ports.conf is removed with every new\nPuppet run, so we need to maintain an ugly hack in services relying\non httpd to ensure no port is lost.\n\nTo simplify this, we are consolidating all apache::listen calls in\none template (apache_ports.pp), which is included by any service\nrequiring Apache. Also, a new plugin (apache_060.py), will make sure\nApache is installed with a default configuration, so any service\nusing it later finds the same starting point.\n\nWhile it does not fix the design issue, it provides a single place to\nspecify listen ports, and will avoid further issues.\n\nChange-Id: Ic8ed069f4991602d125f40eaad8f050284d680e1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/fac84378887f5a98ace94a73937fcc6ca26340e9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/fac84378887f5a98ace94a73937fcc6ca26340e9"}]},"branch":"refs/heads/master"},"68d437f5e2719e86d5268f8bca05abbb3a4cddb2":{"kind":"TRIVIAL_REBASE","_number":4,"created":"2016-04-25 15:10:46.000000000","uploader":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"ref":"refs/changes/81/305881/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/81/305881/4","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/81/305881/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/81/305881/4"}}},"commit":{"parents":[{"commit":"f451f9aa5aa8a823ef9709ef81e6ae7e719a0d0f","subject":"Fix pep8 and python27 tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/f451f9aa5aa8a823ef9709ef81e6ae7e719a0d0f"}]}],"author":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2016-04-14 14:07:03.000000000","tz":120},"committer":{"name":"Javier Peña","email":"jpena@redhat.com","date":"2016-04-25 15:10:46.000000000","tz":0},"subject":"Simplify apache::listen for services relying on httpd","message":"Simplify apache::listen for services relying on httpd\n\nDue to the way Packstack runs Puppet manifests, we are regularly\nhaving issues related to usage of apache::listen. Using it means\nthe existing configuration in ports.conf is removed with every new\nPuppet run, so we need to maintain an ugly hack in services relying\non httpd to ensure no port is lost.\n\nTo simplify this, we are consolidating all apache::listen calls in\none template (apache_ports.pp), which is included by any service\nrequiring Apache. Also, a new plugin (apache_060.py), will make sure\nApache is installed with a default configuration, so any service\nusing it later finds the same starting point.\n\nWhile it does not fix the design issue, it provides a single place to\nspecify listen ports, and will avoid further issues.\n\nChange-Id: Ic8ed069f4991602d125f40eaad8f050284d680e1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/68d437f5e2719e86d5268f8bca05abbb3a4cddb2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/68d437f5e2719e86d5268f8bca05abbb3a4cddb2"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}