)]}'
{"id":"openstack%2Fopenstack-helm~642005","triplet_id":"openstack%2Fopenstack-helm~master~I3c72f959e97be92cc1cf6a7269354d588e3dddd9","project":"openstack/openstack-helm","branch":"master","topic":"fix_vhost_dirs","hashtags":[],"change_id":"I3c72f959e97be92cc1cf6a7269354d588e3dddd9","subject":"use proper dir for virtualhost config for ceilometer and placement api","status":"ABANDONED","created":"2019-03-08 12:57:03.000000000","updated":"2019-03-21 10:19:10.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"1110f59f04b75ba28d061609c6afa23b59df1017","_number":642005,"virtual_id_number":642005,"owner":{"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},{"value":-1,"date":"2019-03-08 13:36:48.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":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"all":[{"value":2,"date":"2019-03-08 13:31:35.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":-1,"date":"2019-03-08 13:33:49.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_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":{"rejected":{"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":-1,"date":"2019-03-08 13:33:49.000000000","permitted_voting_range":{"min":-1,"max":0},"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},{"_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":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-03-08 13:31:35.000000000","updated_by":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"reviewer":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"state":"REVIEWER"},{"updated":"2019-03-08 13:36:48.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":"1a487ce612507e0652eee0b03dc9d61317b75bc3","author":{"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},"date":"2019-03-08 12:57:03.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3279f1db49b49300dbcdfbf85731d869dfe58167","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2019-03-08 13:31:35.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"c47465ca7f011a316c448de04153b5142ce75a4f","author":{"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},"date":"2019-03-08 13:33:49.000000000","message":"Patch Set 1: Code-Review-1 Workflow-1\n\nwe may have the same issue here as in the keystone patch, where based on distro we may want to use one dir or the other, as sites-enabled also doesnt seem to work but vhosts.d kind of does.\n\nrequires more testing","accounts_in_message":[],"_revision_number":1},{"id":"688d6a4bb3eb773b42aea7d730c7d55bc5015b98","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-03-08 13:36:48.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-tox-docs http://logs.openstack.org/05/642005/1/check/openstack-tox-docs/55f5f1d/html/ : SUCCESS in 5m 09s\n- openstack-helm-lint http://logs.openstack.org/05/642005/1/check/openstack-helm-lint/ff91469/ : SUCCESS in 1m 58s\n- openstack-helm-keystone http://logs.openstack.org/05/642005/1/check/openstack-helm-keystone/8c1da61/ : SUCCESS in 24m 26s\n- openstack-helm-keystone-ldap http://logs.openstack.org/05/642005/1/check/openstack-helm-keystone-ldap/ee59f82/ : FAILURE in 13m 36s\n- openstack-helm-glance http://logs.openstack.org/05/642005/1/check/openstack-helm-glance/a4368d4/ : SUCCESS in 24m 46s\n- openstack-helm-heat http://logs.openstack.org/05/642005/1/check/openstack-helm-heat/ad52da1/ : SUCCESS in 30m 53s\n- openstack-helm-cinder http://logs.openstack.org/05/642005/1/check/openstack-helm-cinder/f122ae9/ : SUCCESS in 37m 20s\n- openstack-helm-compute-kit http://logs.openstack.org/05/642005/1/check/openstack-helm-compute-kit/38a9c66/ : SUCCESS in 36m 32s","accounts_in_message":[],"_revision_number":1},{"id":"45029138b0bc429ef5f99968c33475417131cc68","author":{"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},"date":"2019-03-21 10:19:10.000000000","message":"Abandoned\n\nwill resend a different patch with a more proper multi-os approach","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"a6e7fc8ebbfa3721e16457756eb84ed9f6a85cd2","revisions":{"a6e7fc8ebbfa3721e16457756eb84ed9f6a85cd2":{"kind":"REWORK","_number":1,"created":"2019-03-08 12:57:03.000000000","uploader":{"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},"ref":"refs/changes/05/642005/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/05/642005/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/05/642005/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/05/642005/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/05/642005/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/05/642005/1"}}},"commit":{"parents":[{"commit":"85899d2fc8aee99f2235dbaccb61cb3d8b392523","subject":"Merge \"readOnlyFilesystem: true for heat chart\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/85899d2fc8aee99f2235dbaccb61cb3d8b392523"}]}],"author":{"name":"Itxaka","email":"igarcia@suse.com","date":"2019-03-08 12:55:32.000000000","tz":60},"committer":{"name":"Itxaka","email":"igarcia@suse.com","date":"2019-03-08 12:55:32.000000000","tz":60},"subject":"use proper dir for virtualhost config for ceilometer and placement api","message":"use proper dir for virtualhost config for ceilometer and placement api\n\nIts currently using the conf-enabled dir for both of them, but that\ndirectory is supposed to be used for confs only and they should be\nsoft links\n\nInstead use the sites-available for those virtualhost definitions\nand get also in sync with horizon, which is using the proper dir\n\nChange-Id: I3c72f959e97be92cc1cf6a7269354d588e3dddd9\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/a6e7fc8ebbfa3721e16457756eb84ed9f6a85cd2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/a6e7fc8ebbfa3721e16457756eb84ed9f6a85cd2"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}