)]}'
{"id":"openstack%2Fopenstack-helm-infra~797444","triplet_id":"openstack%2Fopenstack-helm-infra~master~I26f7fa79c5c851070e94af758d0a0438aa7efa52","project":"openstack/openstack-helm-infra","branch":"master","hashtags":[],"change_id":"I26f7fa79c5c851070e94af758d0a0438aa7efa52","subject":"[ceph-rgw] Fix a bug in placement target deletion for new targets","status":"MERGED","created":"2021-06-22 13:02:47.000000000","updated":"2021-07-14 18:14:58.000000000","submitted":"2021-06-22 18:43:26.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":3,"unresolved_comment_count":1,"has_review_started":true,"submission_id":"797444","meta_rev_id":"386c60f4b4827f712c06f423df93528fa6191792","_number":797444,"virtual_id_number":797444,"owner":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},{"value":0,"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-06-22 18:43:26.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":32190,"name":"Frank Ritchie","email":"fr801x@att.com","username":"fr801x"},{"value":0,"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"}],"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":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"recommended":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"all":[{"value":1,"date":"2021-06-22 14:28:09.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},{"value":2,"date":"2021-06-22 14:45:19.000000000","_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-06-22 15:00:12.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":32190,"name":"Frank Ritchie","email":"fr801x@att.com","username":"fr801x"},{"value":2,"date":"2021-06-22 17:54:36.000000000","_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"}],"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":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"all":[{"value":0,"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},{"value":0,"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":32190,"name":"Frank Ritchie","email":"fr801x@att.com","username":"fr801x"},{"value":1,"date":"2021-06-22 17:54:36.000000000","_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[{"_account_id":33135,"name":"kiran surapathi","email":"ks342f@att.com","username":"kiransurapathi"}],"reviewers":{"CC":[{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"}],"REVIEWER":[{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},{"_account_id":32190,"name":"Frank Ritchie","email":"fr801x@att.com","username":"fr801x"},{"_account_id":33135,"name":"kiran surapathi","email":"ks342f@att.com","username":"kiransurapathi"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-06-22 14:22:35.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"},{"updated":"2021-06-22 14:24:21.000000000","updated_by":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"reviewer":{"_account_id":33135,"name":"kiran surapathi","email":"ks342f@att.com","username":"kiransurapathi"},"state":"REVIEWER"},{"updated":"2021-06-22 14:24:21.000000000","updated_by":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"reviewer":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"state":"REVIEWER"},{"updated":"2021-06-22 14:24:21.000000000","updated_by":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"reviewer":{"_account_id":32190,"name":"Frank Ritchie","email":"fr801x@att.com","username":"fr801x"},"state":"REVIEWER"},{"updated":"2021-06-22 14:24:21.000000000","updated_by":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"reviewer":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"state":"REVIEWER"},{"updated":"2021-06-22 14:45:19.000000000","updated_by":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"reviewer":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"state":"REVIEWER"},{"updated":"2021-07-14 17:39:17.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":"CC"}],"messages":[{"id":"6584488ffd27a68f2001d925b1c1016d7871e459","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"date":"2021-06-22 13:02:47.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"bde170a081fc044501a71cbb74cdcde5b165bdab","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"date":"2021-06-22 13:11:33.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"f470c7f2322b75763a96fa3df9433278a54b04fc","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-22 14:22:35.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/94e4e30066154cd4a9fc924a609e97a8 : SUCCESS in 5m 11s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/495218d195b3447d8f360c9d18f35791 : SUCCESS in 4m 04s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/c759f1e4c08f4b92b2fd59c4a3d901e4 : SUCCESS in 4m 12s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/b29303eb560b4399ba1332e748237e78 : SUCCESS in 45m 43s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/30a26173f4c44e56a8c3773dacdc1f1b : SUCCESS in 28m 14s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/326b195295044e84baf457d0dfc54b94 : SUCCESS in 48m 10s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/3bcde669a852416ebd36dc066d2d99c8 : SUCCESS in 1h 05m 28s\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/2051c494fef848ac951dff768d8eb14c : SUCCESS in 14m 58s","accounts_in_message":[],"_revision_number":2},{"id":"c0bd4519448a07f06fad314159d30c81d7377463","author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"date":"2021-06-22 14:28:09.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"e3d98b73164b5ec8f78bd1898b967bee1d0591fc","author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"date":"2021-06-22 14:45:19.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"29e5361a4b9cb2910d609d8f671f879874831f10","author":{"_account_id":32190,"name":"Frank Ritchie","email":"fr801x@att.com","username":"fr801x"},"date":"2021-06-22 15:00:12.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"aa6c9831521d8934d72fdf02ab2934d1da2a24b9","author":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"date":"2021-06-22 17:54:36.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"830b32d1d00adb13c9a9049f97b58b254556e878","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-22 17:54:55.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"2dde515d5a469ca4bd1459a5a8549f079677098c","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-22 18:43:26.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3b01bf1741ae4774bd3a41108f49879e : SUCCESS in 4m 06s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/e9bb549feb184c6088b2bd4744c20f4c : SUCCESS in 4m 02s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/c25ecc4bf51d42c18f050ed646dcbdfe : SUCCESS in 4m 58s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/c7258b6509884f89917212cade59a498 : SUCCESS in 37m 39s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/84c463fdbd4c4b1989c292d408d98554 : SUCCESS in 27m 08s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/3d6dce7bf3e2460897ab7e0cf7e4c2ce : SUCCESS in 47m 28s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/a8ff23d5c6c74a538e43438be21d38bb : SUCCESS in 45m 40s","accounts_in_message":[],"_revision_number":2},{"id":"d511340da54f6acf60c577f534efb6aaf5554154","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-22 18:43:26.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"ad58d35f2e097fb4c7d3396e1a8923311b6d34eb","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-22 18:44:37.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/06e5d09a4b4844b6838ec7dc5a430adb : SUCCESS in 52s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/880f8cc1a7724441b59f4a06344c867c : SUCCESS in 53s","accounts_in_message":[],"_revision_number":2},{"id":"b4f763c81f8f7d71253c0225d41f871ab11628cc","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2021-07-14 17:39:17.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"af45553f1edbaed58afbc68cb048ae73b5e2462f","author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"date":"2021-07-14 18:05:29.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"386c60f4b4827f712c06f423df93528fa6191792","author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"date":"2021-07-14 18:14:58.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"62f5cab7707d128eb90cbe0a5651cac6b85ce0b8","revisions":{"3499b5342bfb1bda5a200c9f7f05a1418b32a770":{"kind":"REWORK","_number":1,"created":"2021-06-22 13:02:47.000000000","uploader":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"ref":"refs/changes/44/797444/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/44/797444/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/44/797444/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/44/797444/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/44/797444/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/44/797444/1"}}},"commit":{"parents":[{"commit":"196592ded62c1a2fc6fef196d28cb6a87b65dc8f","subject":"Merge \"Enable Ceph CSI Provisioner to Stand Alone\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/196592ded62c1a2fc6fef196d28cb6a87b65dc8f"}]}],"author":{"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","date":"2021-06-22 12:58:48.000000000","tz":-360},"committer":{"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","date":"2021-06-22 12:58:48.000000000","tz":-360},"subject":"[ceph-rgw] Fix a bug in placement target deletion for new targets","message":"[ceph-rgw] Fix a bug in placement target deletion for new targets\n\nA deployment that specifies a placement target with \"delete: true\"\nshould delete that placement target if it exists. For a clean\ndeployment the expectation is that the placement target should be\ncreated and immediately deleted; however, the check for existence\nhappens before its creation and the delete doesn\u0027t execute as a\nresult. This change adds a recheck for existence immediately after\ncreation to remedy that.\n\nChange-Id: I26f7fa79c5c851070e94af758d0a0438aa7efa52\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/3499b5342bfb1bda5a200c9f7f05a1418b32a770"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/3499b5342bfb1bda5a200c9f7f05a1418b32a770"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"196592ded62c1a2fc6fef196d28cb6a87b65dc8f","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"62f5cab7707d128eb90cbe0a5651cac6b85ce0b8":{"kind":"REWORK","_number":2,"created":"2021-06-22 13:11:33.000000000","uploader":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"ref":"refs/changes/44/797444/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/44/797444/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/44/797444/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/44/797444/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/44/797444/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/44/797444/2"}}},"commit":{"parents":[{"commit":"196592ded62c1a2fc6fef196d28cb6a87b65dc8f","subject":"Merge \"Enable Ceph CSI Provisioner to Stand Alone\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/196592ded62c1a2fc6fef196d28cb6a87b65dc8f"}]}],"author":{"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","date":"2021-06-22 12:58:48.000000000","tz":-360},"committer":{"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","date":"2021-06-22 13:11:25.000000000","tz":-360},"subject":"[ceph-rgw] Fix a bug in placement target deletion for new targets","message":"[ceph-rgw] Fix a bug in placement target deletion for new targets\n\nA deployment that specifies a placement target with \"delete: true\"\nshould delete that placement target if it exists. For a clean\ndeployment the expectation is that the placement target should be\ncreated and immediately deleted; however, the check for existence\nhappens before its creation and the delete doesn\u0027t execute as a\nresult. This change adds a recheck for existence immediately after\ncreation to remedy that.\n\nChange-Id: I26f7fa79c5c851070e94af758d0a0438aa7efa52\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/62f5cab7707d128eb90cbe0a5651cac6b85ce0b8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/62f5cab7707d128eb90cbe0a5651cac6b85ce0b8"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"196592ded62c1a2fc6fef196d28cb6a87b65dc8f","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}