)]}'
{"id":"openstack%2Fopenstack-helm-infra~783737","triplet_id":"openstack%2Fopenstack-helm-infra~master~Ife63b73f89996d59b75ec617129818068b060d1c","project":"openstack/openstack-helm-infra","branch":"master","hashtags":[],"change_id":"Ife63b73f89996d59b75ec617129818068b060d1c","subject":"Allow Ceph RBD pool job to leave failed pods","status":"MERGED","created":"2021-03-29 19:39:17.000000000","updated":"2021-03-30 08:05:33.000000000","submitted":"2021-03-30 08:04:06.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":"783737","meta_rev_id":"630f94e0a87f18da8834009a537ea7cc121ee60b","_number":783737,"virtual_id_number":783737,"owner":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},{"value":0,"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"value":0,"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"value":0,"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-03-30 08:04:05.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"}],"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"},"recommended":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"all":[{"value":1,"date":"2021-03-29 19:43:57.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},{"value":1,"date":"2021-03-30 00:44:12.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"value":2,"date":"2021-03-30 07:07:38.000000000","_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"value":2,"date":"2021-03-30 03:08:26.000000000","_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-03-29 19:53:45.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"}],"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":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"all":[{"value":0,"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},{"value":0,"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"value":1,"date":"2021-03-30 07:07:38.000000000","_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"value":0,"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},{"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-03-29 19:43:57.000000000","updated_by":{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},"reviewer":{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},"state":"REVIEWER"},{"updated":"2021-03-29 19:53:45.000000000","updated_by":{"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},"reviewer":{"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},"state":"REVIEWER"},{"updated":"2021-03-29 20:27:50.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-03-30 00:44:12.000000000","updated_by":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"reviewer":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"state":"REVIEWER"},{"updated":"2021-03-30 03:08:26.000000000","updated_by":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"reviewer":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"state":"REVIEWER"},{"updated":"2021-03-30 07:07:38.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"}],"messages":[{"id":"df1dd66542c0dc4e79ec87da36b756512a530cbb","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"date":"2021-03-29 19:39:17.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"bbdbb475da3eea69bee3df346ca54c0a9265eed6","author":{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},"date":"2021-03-29 19:43:57.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"88bf251c404adf83fc7158bdcb4b7241ef8d5140","author":{"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},"date":"2021-03-29 19:53:45.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"48180985b1a62a6a327c397e0a6c5580367aacb8","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-29 20:27:50.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f38bce2077964a7382c505af261095d4 : SUCCESS in 4m 32s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/43900002dc2245298d0903126811d71f : SUCCESS in 4m 20s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/6c5f1efc78144096a234550412be5afc : SUCCESS in 4m 46s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/cb1ce4bc4fa844e8b6a7c84786f9ca69 : SUCCESS in 36m 03s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/e64bd256f88a4041b6116539139a3c7d : SUCCESS in 28m 44s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/be96fe6da1c340089587f075f6244d14 : SUCCESS in 46m 41s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/2a2a85ddf5d94efb9cf22ec81832edd5 : SUCCESS in 44m 29s\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/a5c1aaf0ab67451181e0bfae882e703e : SUCCESS in 14m 46s","accounts_in_message":[],"_revision_number":1},{"id":"0572f7618b12b924e0d125c3cec0e07a266ca556","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-03-30 00:44:12.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"839627a4eaeb1d71438fecd4d3d6d61c389ba74f","author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"date":"2021-03-30 03:08:26.000000000","message":"Patch Set 1: Code-Review+2\n\nI am ok with the changes per-se, although in order to set this to \"Never\" for debug purpose, there is really no easy way to update this value once it is deployed other than to hack the job yaml file, as there is no configmap to update, so it sort of render this change kind of moot?  You are almost better off to just update the commit msg to say you want to make the restartPolicy configurable and leave it at that.","accounts_in_message":[],"_revision_number":1},{"id":"5f3afe4b13bf9a6994c4c8ac449ff4b884b202b5","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2021-03-30 07:07:38.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"4656eb7c797151e08e086cbe18621eb25a7399c7","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-30 07:07:54.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"4569bb5f2bc1e3d6c925f26a74076553bc195e17","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-30 08:04:05.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d96e694416f34d7ead052585c9c02fad : SUCCESS in 4m 29s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/008ba25bfb194199aeb22becdebaa879 : SUCCESS in 3m 58s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/a890b3cce6af4529ae233e12cf98b20c : SUCCESS in 5m 44s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/31d8dea6d0694e74af53d6e808848b14 : SUCCESS in 39m 55s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/46297ea6653c478cbd20a05cb0651d95 : SUCCESS in 38m 12s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/eaf3463d446e40cdbd28d82b16bb1c06 : SUCCESS in 54m 16s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/f92482d22fcf49abbe0161bec96647c2 : SUCCESS in 53m 46s","accounts_in_message":[],"_revision_number":1},{"id":"fd5a3a9ccacff47a33c2452bfff613b07e3dace1","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-30 08:04:06.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"630f94e0a87f18da8834009a537ea7cc121ee60b","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-30 08:05:33.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/7007369d6866424caab8443ef92a4034 : SUCCESS in 59s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/984e191b700e4373b960d477726bfece : SUCCESS in 57s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"f20eff164f9de5c43e747826235662f9219eba58","revisions":{"f20eff164f9de5c43e747826235662f9219eba58":{"kind":"REWORK","_number":1,"created":"2021-03-29 19:39:17.000000000","uploader":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"ref":"refs/changes/37/783737/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/37/783737/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/37/783737/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/37/783737/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/37/783737/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/37/783737/1"}}},"commit":{"parents":[{"commit":"734b344bf6a2106b234bca0c927f0accb8331cf4","subject":"[ceph-provisioners] Update ceph_mon config as per new ceph clients","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/734b344bf6a2106b234bca0c927f0accb8331cf4"}]}],"author":{"name":"Parsons, Cliff (cp769u)","email":"cp769u@att.com","date":"2021-03-29 19:34:11.000000000","tz":0},"committer":{"name":"Parsons, Cliff (cp769u)","email":"cp769u@att.com","date":"2021-03-29 19:38:55.000000000","tz":0},"subject":"Allow Ceph RBD pool job to leave failed pods","message":"Allow Ceph RBD pool job to leave failed pods\n\nThis patchset will add the capability to configure the\nCeph RBD pool job to leave failed pods behind for debugging\npurposes, if it is desired. Default is to not leave them\nbehind, which is the current behavior.\n\nChange-Id: Ife63b73f89996d59b75ec617129818068b060d1c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/f20eff164f9de5c43e747826235662f9219eba58"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/f20eff164f9de5c43e747826235662f9219eba58"}]},"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":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"}}]}],"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":{}}}]}