)]}'
{"id":"openstack%2Fopenstack-helm~568027","triplet_id":"openstack%2Fopenstack-helm~master~I5c5461839f2351c3955efaec613bd4bcc7174f78","project":"openstack/openstack-helm","branch":"master","topic":"nova/ironic","hashtags":[],"change_id":"I5c5461839f2351c3955efaec613bd4bcc7174f78","subject":"Heat: Dont deploy cloudwatch by default","status":"MERGED","created":"2018-05-12 22:41:07.000000000","updated":"2018-05-13 04:04:11.000000000","submitted":"2018-05-13 04:04:11.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":"568027-1526184251526-2988e340","meta_rev_id":"4471417292fb91e1bb9a875379a6cff68d5fb381","_number":568027,"virtual_id_number":568027,"owner":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"value":0,"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"value":2,"date":"2018-05-13 04:04:11.000000000","post_submit":true,"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":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"all":[{"value":2,"date":"2018-05-13 01:14:20.000000000","_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"value":2,"date":"2018-05-13 01:14:20.000000000","_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"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":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"all":[{"value":1,"date":"2018-05-13 01:14:20.000000000","_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"value":0,"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"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":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-05-12 22:45:12.000000000","updated_by":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"reviewer":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"state":"REVIEWER"},{"updated":"2018-05-12 23:17:29.000000000","updated_by":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"reviewer":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"state":"REVIEWER"},{"updated":"2018-05-13 04:04:11.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":"e8ead55199349becc46b5d2f3101e34379df51c2","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-05-12 22:41:07.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"ceeb333e6ff39bcb295955183dd234cabb2f07eb","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-05-12 22:42:21.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"1c4fdffad32f4332eb4fbf3cdd34282736cbab4a","author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"date":"2018-05-12 22:45:12.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"28c2bf64b0215bfef4b371d743439e15fd99be1c","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2018-05-12 23:17:29.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"29a1e070c8dfc5d200509b5be817fb3c4e43d786","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-05-13 00:39:03.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/27/568027/2/check/build-openstack-sphinx-docs/f176f2c/html/ : SUCCESS in 5m 15s\n- openstack-helm-linter http://logs.openstack.org/27/568027/2/check/openstack-helm-linter/142f994/ : SUCCESS in 1m 47s\n- openstack-helm-dev-deploy-ceph-radosgw-ovs http://logs.openstack.org/27/568027/2/check/openstack-helm-dev-deploy-ceph-radosgw-ovs/78115c8/ : SUCCESS in 1h 01m 05s\n- openstack-helm-dev-deploy-ceph-rbd-ovs http://logs.openstack.org/27/568027/2/check/openstack-helm-dev-deploy-ceph-rbd-ovs/6a1e03f/ : SUCCESS in 1h 01m 37s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/27/568027/2/check/openstack-helm-dev-deploy-nfs-ovs/eb849aa/ : SUCCESS in 49m 41s\n- openstack-helm-dev-deploy-ceph-lb http://logs.openstack.org/27/568027/2/check/openstack-helm-dev-deploy-ceph-lb/271340d/ : SUCCESS in 1h 00m 21s (non-voting)\n- openstack-helm-dev-deploy-nfs-lb http://logs.openstack.org/27/568027/2/check/openstack-helm-dev-deploy-nfs-lb/8d916c0/ : SUCCESS in 48m 44s (non-voting)\n- openstack-helm-dev-deploy-nfs-ldap http://logs.openstack.org/27/568027/2/check/openstack-helm-dev-deploy-nfs-ldap/348fdaf/ : SUCCESS in 51m 46s (non-voting)\n- openstack-helm-multinode-ubuntu http://logs.openstack.org/27/568027/2/check/openstack-helm-multinode-ubuntu/4c18da2/ : SUCCESS in 1h 54m 42s (non-voting)\n- openstack-helm-ironic-ubuntu http://logs.openstack.org/27/568027/2/check/openstack-helm-ironic-ubuntu/d45e49f/ : FAILURE in 1h 22m 06s (non-voting)\n- openstack-helm-armada-fullstack-deploy http://logs.openstack.org/27/568027/2/check/openstack-helm-armada-fullstack-deploy/76716c4/ : FAILURE in 59m 12s (non-voting)\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/27/568027/2/check/openstack-helm-kolla-deploy-ceph-ovs/57ddaef/ : SUCCESS in 1h 02m 39s\n- openstack-helm-ocata-loci-deploy-ceph http://logs.openstack.org/27/568027/2/check/openstack-helm-ocata-loci-deploy-ceph/fbf6acb/ : SUCCESS in 1h 05m 19s (non-voting)\n- openstack-helm-ocata-kolla-deploy-ceph http://logs.openstack.org/27/568027/2/check/openstack-helm-ocata-kolla-deploy-ceph/77b1dda/ : SUCCESS in 1h 14m 43s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"bf58c2545700f36bc840de163c6078c7cbf694d7","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-05-13 00:39:16.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"1e57a96885a0af4e2b12edfa71fc010850b0e964","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-05-13 01:14:20.000000000","message":"Uploaded patch set 3: Patch Set 2 was rebased.","accounts_in_message":[],"_revision_number":3},{"id":"65d2ae6a4763f2abc530d2fb8524544f1a21b8a6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-05-13 02:55:51.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/27/568027/3/check/build-openstack-sphinx-docs/e09eb55/html/ : SUCCESS in 3m 34s\n- openstack-helm-linter http://logs.openstack.org/27/568027/3/check/openstack-helm-linter/7ee6998/ : SUCCESS in 57s\n- openstack-helm-dev-deploy-ceph-radosgw-ovs http://logs.openstack.org/27/568027/3/check/openstack-helm-dev-deploy-ceph-radosgw-ovs/536db8d/ : SUCCESS in 1h 00m 54s\n- openstack-helm-dev-deploy-ceph-rbd-ovs http://logs.openstack.org/27/568027/3/check/openstack-helm-dev-deploy-ceph-rbd-ovs/da70239/ : SUCCESS in 1h 00m 40s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/27/568027/3/check/openstack-helm-dev-deploy-nfs-ovs/f09ff64/ : SUCCESS in 51m 44s\n- openstack-helm-dev-deploy-ceph-lb http://logs.openstack.org/27/568027/3/check/openstack-helm-dev-deploy-ceph-lb/22a7fc3/ : SUCCESS in 58m 57s (non-voting)\n- openstack-helm-dev-deploy-nfs-lb http://logs.openstack.org/27/568027/3/check/openstack-helm-dev-deploy-nfs-lb/10303c6/ : SUCCESS in 46m 15s (non-voting)\n- openstack-helm-dev-deploy-nfs-ldap http://logs.openstack.org/27/568027/3/check/openstack-helm-dev-deploy-nfs-ldap/611e462/ : SUCCESS in 48m 32s (non-voting)\n- openstack-helm-multinode-ubuntu http://logs.openstack.org/27/568027/3/check/openstack-helm-multinode-ubuntu/6302975/ : SUCCESS in 1h 40m 30s (non-voting)\n- openstack-helm-ironic-ubuntu http://logs.openstack.org/27/568027/3/check/openstack-helm-ironic-ubuntu/a8c8a4b/ : SUCCESS in 1h 10m 26s (non-voting)\n- openstack-helm-armada-fullstack-deploy http://logs.openstack.org/27/568027/3/check/openstack-helm-armada-fullstack-deploy/4a9cbe4/ : FAILURE in 1h 00m 32s (non-voting)\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/27/568027/3/check/openstack-helm-kolla-deploy-ceph-ovs/6bacf1d/ : SUCCESS in 1h 03m 32s\n- openstack-helm-ocata-loci-deploy-ceph http://logs.openstack.org/27/568027/3/check/openstack-helm-ocata-loci-deploy-ceph/b573c48/ : SUCCESS in 57m 43s (non-voting)\n- openstack-helm-ocata-kolla-deploy-ceph http://logs.openstack.org/27/568027/3/check/openstack-helm-ocata-kolla-deploy-ceph/218e624/ : SUCCESS in 59m 15s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"4ce898ba127923d06de19a59c767ece6a1cc53b9","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-05-13 02:56:04.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"fc03e4db72078d0a18dd37b9a8a960647f7e7e3b","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-05-13 04:04:11.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3},{"id":"6dbf14372cbf91b18d5c89a4f527e0ae6215f0d7","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-05-13 04:04:11.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/27/568027/3/gate/build-openstack-sphinx-docs/ba19ca0/html/ : SUCCESS in 3m 52s\n- openstack-helm-linter http://logs.openstack.org/27/568027/3/gate/openstack-helm-linter/19754e0/ : SUCCESS in 1m 45s\n- openstack-helm-dev-deploy-ceph-radosgw-ovs http://logs.openstack.org/27/568027/3/gate/openstack-helm-dev-deploy-ceph-radosgw-ovs/add9807/ : SUCCESS in 1h 02m 10s\n- openstack-helm-dev-deploy-ceph-rbd-ovs http://logs.openstack.org/27/568027/3/gate/openstack-helm-dev-deploy-ceph-rbd-ovs/403f647/ : SUCCESS in 57m 42s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/27/568027/3/gate/openstack-helm-dev-deploy-nfs-ovs/1a67c64/ : SUCCESS in 49m 21s\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/27/568027/3/gate/openstack-helm-kolla-deploy-ceph-ovs/11c5380/ : SUCCESS in 1h 03m 34s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"f8a273d384a7c1ce2d494bc18626713a46e1d285","revisions":{"14f35ab4c8f1b77c31f2040e8947b291d39a53c8":{"kind":"REWORK","_number":1,"created":"2018-05-12 22:41:07.000000000","uploader":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"ref":"refs/changes/27/568027/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/27/568027/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/1"}}},"commit":{"parents":[{"commit":"99b47be838df0a267df474e9aa708ea5051cf9d8","subject":"Heat: Correct ingress manifest conditionals","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/99b47be838df0a267df474e9aa708ea5051cf9d8"}]}],"author":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-05-12 22:37:54.000000000","tz":-300},"committer":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-05-12 22:37:54.000000000","tz":-300},"subject":"Heat: Dont deploy cloudwatch by default","message":"Heat: Dont deploy cloudwatch by default\n\nAs per the Wiki page [1] and discussions on the openstack-dev mailing list [2],\nheat-api-cloudwatch is depreciated in the liberty cycle and slated for removal.\nIt was fully removed during the queens development cycle.\n\n[1] https://wiki.openstack.org/wiki/Heat/Using-CloudWatch\n[2] http://lists.openstack.org/pipermail/openstack-dev/2015-April/061834.html\n\nChange-Id: I5c5461839f2351c3955efaec613bd4bcc7174f78\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/14f35ab4c8f1b77c31f2040e8947b291d39a53c8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/14f35ab4c8f1b77c31f2040e8947b291d39a53c8"}]},"branch":"refs/heads/master"},"7f487136e404b65c5427d633abdf74f0aaf59797":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2018-05-12 22:42:21.000000000","uploader":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"ref":"refs/changes/27/568027/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/27/568027/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/2"}}},"commit":{"parents":[{"commit":"99b47be838df0a267df474e9aa708ea5051cf9d8","subject":"Heat: Correct ingress manifest conditionals","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/99b47be838df0a267df474e9aa708ea5051cf9d8"}]}],"author":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-05-12 22:37:54.000000000","tz":-300},"committer":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-05-12 22:41:50.000000000","tz":-300},"subject":"Heat: Dont deploy cloudwatch by default","message":"Heat: Dont deploy cloudwatch by default\n\nAs per the Wiki page [1] and discussions on the openstack-dev\nmailing list [2], heat-api-cloudwatch was depreciated following\nthe liberty cycle and slated for removal. It was fully removed\nduring the queens development cycle.\n\n[1] https://wiki.openstack.org/wiki/Heat/Using-CloudWatch\n[2] http://lists.openstack.org/pipermail/openstack-dev/2015-April/061834.html\n\nChange-Id: I5c5461839f2351c3955efaec613bd4bcc7174f78\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/7f487136e404b65c5427d633abdf74f0aaf59797"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/7f487136e404b65c5427d633abdf74f0aaf59797"}]},"branch":"refs/heads/master"},"f8a273d384a7c1ce2d494bc18626713a46e1d285":{"kind":"NO_CHANGE","_number":3,"created":"2018-05-13 01:14:20.000000000","uploader":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"ref":"refs/changes/27/568027/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/27/568027/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/27/568027/3"}}},"commit":{"parents":[{"commit":"99b47be838df0a267df474e9aa708ea5051cf9d8","subject":"Heat: Correct ingress manifest conditionals","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/99b47be838df0a267df474e9aa708ea5051cf9d8"}]}],"author":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-05-12 22:37:54.000000000","tz":-300},"committer":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-05-12 22:43:29.000000000","tz":-300},"subject":"Heat: Dont deploy cloudwatch by default","message":"Heat: Dont deploy cloudwatch by default\n\nAs per the Wiki page [1] and discussions on the openstack-dev\nmailing list [2], heat-api-cloudwatch was depreciated following\nthe liberty cycle and slated for removal. It was fully removed\nduring the queens development cycle.\n\n[1] https://wiki.openstack.org/wiki/Heat/Using-CloudWatch\n[2] http://lists.openstack.org/pipermail/openstack-dev/2015-April/061834.html\n\nChange-Id: I5c5461839f2351c3955efaec613bd4bcc7174f78\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/f8a273d384a7c1ce2d494bc18626713a46e1d285"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/f8a273d384a7c1ce2d494bc18626713a46e1d285"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}