)]}'
{"id":"openstack%2Fopenstack-helm~559351","triplet_id":"openstack%2Fopenstack-helm~master~Ibcde4942dcd081205634be3b9a819a3802bf5784","project":"openstack/openstack-helm","branch":"master","topic":"bug/1761604","hashtags":[],"change_id":"Ibcde4942dcd081205634be3b9a819a3802bf5784","subject":"remove pdb for barbican-api","status":"ABANDONED","created":"2018-04-06 15:10:15.000000000","updated":"2020-07-15 19:29:20.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9096d7a605a0ddd31235791f536464e2764f506f","_number":559351,"virtual_id_number":559351,"owner":{"_account_id":20889,"name":"Niketu Parekh","email":"niketu.parekh@ericsson.com","username":"niketukparekh"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":18675,"name":"Sharath Rao","email":"sharath.rao@ericsson.com","username":"sharath.rao"},{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"value":1,"date":"2018-04-06 19:46: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":"","value":1,"default_value":0,"optional":true},"Code-Review":{"rejected":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":18675,"name":"Sharath Rao","email":"sharath.rao@ericsson.com","username":"sharath.rao"},{"value":-2,"date":"2018-04-06 18:19:13.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":18675,"name":"Sharath Rao","email":"sharath.rao@ericsson.com","username":"sharath.rao"},{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"_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":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"_account_id":18675,"name":"Sharath Rao","email":"sharath.rao@ericsson.com","username":"sharath.rao"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-04-06 15:12:00.000000000","updated_by":{"_account_id":18675,"name":"Sharath Rao","email":"sharath.rao@ericsson.com","username":"sharath.rao"},"reviewer":{"_account_id":18675,"name":"Sharath Rao","email":"sharath.rao@ericsson.com","username":"sharath.rao"},"state":"REVIEWER"},{"updated":"2018-04-06 15:59:31.000000000","updated_by":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"reviewer":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"state":"REVIEWER"},{"updated":"2018-04-06 18:19:13.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-04-06 19:46: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":"b6b365a67d1d326d7fb9d79df8c81bdfc070cf07","author":{"_account_id":20889,"name":"Niketu Parekh","email":"niketu.parekh@ericsson.com","username":"niketukparekh"},"date":"2018-04-06 15:10:15.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"d48ce98b9170b6738423e31ac101ffcec77adecf","author":{"_account_id":18675,"name":"Sharath Rao","email":"sharath.rao@ericsson.com","username":"sharath.rao"},"date":"2018-04-06 15:12:00.000000000","message":"Patch Set 1: Code-Review-1\n\nPlease format the commit message to make it readable.","accounts_in_message":[],"_revision_number":1},{"id":"781a55eac256ec213a7a461218accc0ce1939261","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2018-04-06 15:59:31.000000000","message":"Patch Set 1:\n\nexpanding on that, where possible, something like:\n\n\n\u003e something: one line explanation of what the change is\n\u003e \n\u003e if needed a longer explanation on line below\n\nthere is a convention that the first line should *not* having a trailing period (.)","accounts_in_message":[],"_revision_number":1},{"id":"ab9b0e067c7de86d81ffaa2ab8a65ab03f290f90","author":{"_account_id":20889,"name":"Niketu Parekh","email":"niketu.parekh@ericsson.com","username":"niketukparekh"},"date":"2018-04-06 17:26:07.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"2955ee1b167692b167d55b988e20be88fa960e29","author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"date":"2018-04-06 18:18:57.000000000","message":"Patch Set 2: Code-Review-1\n\nWe shouldn\u0027t be removing the PDB for the charts, as the pod disruption budget and the configured update strategy are two disjoint things. The pod disruption budget puts a limit on the number of unavailable pods during an intentional disruption, while the update strategy configured in the replication controller is specific to behavior during an upgrade.  \nTo read more: https://kubernetes.io/docs/concepts/workloads/pods/disruptions/#how-disruption-budgets-work","accounts_in_message":[],"_revision_number":2},{"id":"336eae4fa2bb11465253010f1482f943cf90aa14","author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"date":"2018-04-06 18:19:13.000000000","message":"Patch Set 2: Code-Review-2","accounts_in_message":[],"_revision_number":2},{"id":"36531550a70c06c54312f8c6b1f20d8a327c3322","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-04-06 19:46:14.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/51/559351/2/check/build-openstack-sphinx-docs/85a2a20/html/ : SUCCESS in 3m 27s\n- openstack-helm-linter http://logs.openstack.org/51/559351/2/check/openstack-helm-linter/ca2f777/ : SUCCESS in 1m 22s\n- openstack-helm-dev-deploy-ceph-radosgw-ovs http://logs.openstack.org/51/559351/2/check/openstack-helm-dev-deploy-ceph-radosgw-ovs/d60a59d/ : SUCCESS in 1h 01m 24s\n- openstack-helm-dev-deploy-ceph-rbd-ovs http://logs.openstack.org/51/559351/2/check/openstack-helm-dev-deploy-ceph-rbd-ovs/ef431e7/ : SUCCESS in 1h 08m 55s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/51/559351/2/check/openstack-helm-dev-deploy-nfs-ovs/1efecc0/ : SUCCESS in 53m 21s\n- openstack-helm-dev-deploy-ceph-lb http://logs.openstack.org/51/559351/2/check/openstack-helm-dev-deploy-ceph-lb/d563f93/ : FAILURE in 46m 19s (non-voting)\n- openstack-helm-dev-deploy-nfs-lb http://logs.openstack.org/51/559351/2/check/openstack-helm-dev-deploy-nfs-lb/5db539f/ : SUCCESS in 53m 56s (non-voting)\n- openstack-helm-dev-deploy-nfs-ldap http://logs.openstack.org/51/559351/2/check/openstack-helm-dev-deploy-nfs-ldap/3a76398/ : SUCCESS in 51m 38s (non-voting)\n- openstack-helm-multinode-ubuntu http://logs.openstack.org/51/559351/2/check/openstack-helm-multinode-ubuntu/0f4094c/ : SUCCESS in 1h 58m 19s (non-voting)\n- openstack-helm-ironic-ubuntu http://logs.openstack.org/51/559351/2/check/openstack-helm-ironic-ubuntu/73f55ad/ : SUCCESS in 1h 12m 30s (non-voting)\n- openstack-helm-armada-fullstack-deploy http://logs.openstack.org/51/559351/2/check/openstack-helm-armada-fullstack-deploy/15ad63e/ : SUCCESS in 2h 17m 41s (non-voting)\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/51/559351/2/check/openstack-helm-kolla-deploy-ceph-ovs/85dae15/ : SUCCESS in 1h 16m 54s\n- openstack-helm-ocata-loci-deploy-ceph http://logs.openstack.org/51/559351/2/check/openstack-helm-ocata-loci-deploy-ceph/ccac908/ : SUCCESS in 1h 05m 36s (non-voting)\n- openstack-helm-ocata-kolla-deploy-ceph http://logs.openstack.org/51/559351/2/check/openstack-helm-ocata-kolla-deploy-ceph/a45c87f/ : SUCCESS in 1h 15m 00s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"c004bf6e05d87b5b5802a68ca79b8e298474defa","author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"date":"2020-07-15 19:29:20.000000000","message":"Abandoned\n\nAbandoning since there hasn\u0027t been any recent activity, if anyone wants to continue this work, please feel free to restore this.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"e3f5f90d8c4374c8e1af4f0667c6e02c30e6fd34","revisions":{"79eb3e446361e9538c840e47a910508085640616":{"kind":"REWORK","_number":1,"created":"2018-04-06 15:10:15.000000000","uploader":{"_account_id":20889,"name":"Niketu Parekh","email":"niketu.parekh@ericsson.com","username":"niketukparekh"},"ref":"refs/changes/51/559351/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/51/559351/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/51/559351/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/51/559351/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/51/559351/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/51/559351/1"}}},"commit":{"parents":[{"commit":"44841fbffcf75321243cd2352025b5f33d84feca","subject":"Storage: increase robustness of storage clean jobs","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/44841fbffcf75321243cd2352025b5f33d84feca"}]}],"author":{"name":"Niketu Parekh","email":"niketu.parekh@ericsson.com","date":"2018-04-06 15:03:05.000000000","tz":-300},"committer":{"name":"Niketu Parekh","email":"niketu.parekh@ericsson.com","date":"2018-04-06 15:03:05.000000000","tz":-300},"subject":"remove pdb for barbican-api","message":"remove pdb for barbican-api\n\nThe pdb is redundant for k8s Deployment. The barbican-api is being deployed as k8s Deployment with rollingupdate strategy, maxUnavailable and maxSurge values. The barbican-api pdb is redundant. This fix removes the pdb for barbican-api.\n\nChange-Id: Ibcde4942dcd081205634be3b9a819a3802bf5784\nCloses-Bug: 1761604\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/79eb3e446361e9538c840e47a910508085640616"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/79eb3e446361e9538c840e47a910508085640616"}]},"branch":"refs/heads/master"},"e3f5f90d8c4374c8e1af4f0667c6e02c30e6fd34":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2018-04-06 17:26:07.000000000","uploader":{"_account_id":20889,"name":"Niketu Parekh","email":"niketu.parekh@ericsson.com","username":"niketukparekh"},"ref":"refs/changes/51/559351/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/51/559351/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/51/559351/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/51/559351/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/51/559351/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/51/559351/2"}}},"commit":{"parents":[{"commit":"44841fbffcf75321243cd2352025b5f33d84feca","subject":"Storage: increase robustness of storage clean jobs","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/44841fbffcf75321243cd2352025b5f33d84feca"}]}],"author":{"name":"Niketu Parekh","email":"niketu.parekh@ericsson.com","date":"2018-04-06 15:03:05.000000000","tz":-300},"committer":{"name":"Niketu Parekh","email":"niketu.parekh@ericsson.com","date":"2018-04-06 17:17:02.000000000","tz":-300},"subject":"remove pdb for barbican-api","message":"remove pdb for barbican-api\n\nThe pdb is redundant for kubernetes Deployment.\nThe barbican-api is being deployed as Deployment with rollingupdate strategy,\nmaxUnavailable and maxSurge values.\n\nThis fix removes the pdb for barbican-api.\n\nChange-Id: Ibcde4942dcd081205634be3b9a819a3802bf5784\nCloses-Bug: 1761604\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/e3f5f90d8c4374c8e1af4f0667c6e02c30e6fd34"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/e3f5f90d8c4374c8e1af4f0667c6e02c30e6fd34"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}