)]}'
{"id":"openstack%2Fopenstack-helm-infra~778541","triplet_id":"openstack%2Fopenstack-helm-infra~master~Ib82120611659bd36bae35f2e90054642fb8ee31f","project":"openstack/openstack-helm-infra","branch":"master","hashtags":[],"change_id":"Ib82120611659bd36bae35f2e90054642fb8ee31f","subject":"Rename mariadb backup identities","status":"MERGED","created":"2021-03-03 18:38:33.000000000","updated":"2021-03-05 19:29:07.000000000","submitted":"2021-03-05 19:29:07.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":1,"unresolved_comment_count":1,"has_review_started":true,"submission_id":"778541","meta_rev_id":"a38abb7547d0019552980fe2a7410ab6623cc2c9","_number":778541,"virtual_id_number":778541,"owner":{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"value":0,"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"value":0,"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-03-05 19:29:06.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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"recommended":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"all":[{"value":1,"date":"2021-03-04 01:00:16.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"value":2,"date":"2021-03-05 18:21:43.000000000","_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"value":2,"date":"2021-03-04 01:34:26.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"]}],"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":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"all":[{"value":0,"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"value":1,"date":"2021-03-05 18:21:43.000000000","_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"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"]}],"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":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_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-03 19:51:59.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-04 01:00:16.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-04 01:34:26.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-03-05 18:21:43.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"}],"messages":[{"id":"f46a223ca306d0be61e03e8535aa938f04ae4522","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"},"date":"2021-03-03 18:38:33.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"98b542e64a288ba3e1ba379bdd7a28e6a2040af2","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"},"date":"2021-03-03 18:47:16.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"b7a69d81afc03bf6cd6943bbf8eed2067b59bf4d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-03 19:51:59.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e1010410a4b64147be4e94e94ce9e2d1 : SUCCESS in 4m 45s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/5aae64c425574660af6c3e3200f90cb6 : SUCCESS in 4m 43s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/40beeb8ec1ba4eb49ab36c97a1ca20f2 : SUCCESS in 5m 32s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/9ab35dd24c774e7d9a080c44dfb9d9ad : SUCCESS in 47m 12s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/8e9d761d70c1431a85d803971fb185d1 : SUCCESS in 37m 06s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/1d884e3199d84b7ca54770446e613e6a : SUCCESS in 58m 41s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/bec41e52c21e4a6980a57e00571132dc : SUCCESS in 45m 09s\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/49327b3b4de247f9ab94a6e7ecfc9175 : SUCCESS in 16m 06s","accounts_in_message":[],"_revision_number":2},{"id":"d9b8ed0a9d24cdbc74b14d2b58fdd9981785d891","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-03-04 01:00:16.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"a8f42bf56cc0481d39d38348be375415f8264759","author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"date":"2021-03-04 01:34:26.000000000","message":"Patch Set 2: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"d500a65da5d313d475f354af7b2e07d75a7981ac","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-03-05 18:21:43.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"3ff94e7e71b0e12c48561118b08138a07042669c","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-05 18:22:14.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"57256ad3294694ff2e50f01f04d4b101672b20ff","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-05 19:29:06.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ed62c869c6d54e92a8f90b1f0aea0b68 : SUCCESS in 4m 51s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/46e989fc953946cc9f120da3f9d2add8 : SUCCESS in 4m 15s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/8cd46804ee1c4a9d80f0a94a360cf219 : SUCCESS in 7m 47s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/1ab16ca3913f47beb55f9a9a40832273 : SUCCESS in 37m 21s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/777e536226474122945be17a90293352 : SUCCESS in 32m 10s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/2a39585b052f4e91bb038dfcb0ff0f4d : SUCCESS in 53m 09s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/9db8a9bbc5d244adacd2de5c5c70ff73 : SUCCESS in 48m 34s","accounts_in_message":[],"_revision_number":2},{"id":"a38abb7547d0019552980fe2a7410ab6623cc2c9","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-05 19:29:07.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"5db88a5fb487521e1cf20506150d7ec77c520c7c","revisions":{"89db3881717c02326def9eff3e6bc4bc00fb6151":{"kind":"REWORK","_number":1,"created":"2021-03-03 18:38:33.000000000","uploader":{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"},"ref":"refs/changes/41/778541/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/41/778541/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/41/778541/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/41/778541/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/41/778541/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/41/778541/1"}}},"commit":{"parents":[{"commit":"618c064d2594bdbb56a2d4ca043b6268de2921ab","subject":"Merge \"Add metadata in job templates\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/618c064d2594bdbb56a2d4ca043b6268de2921ab"}]}],"author":{"name":"okozachenko","email":"okozachenko@vexxhost.com","date":"2021-03-03 18:28:05.000000000","tz":120},"committer":{"name":"okozachenko","email":"okozachenko@vexxhost.com","date":"2021-03-03 18:38:20.000000000","tz":120},"subject":"Rename mariadb backup identities","message":"Rename mariadb backup identities\n\nChallenge:\nNow remote_ks_admin and remote_rgw_user are using for user labels\nof backup target openstack cloud.\nWhen the backup user doesn\u0027t exist and we can enable job_ks_user\nmanifest.\nBut job_ks_user uses .Vaules.secrets.identity.admin and mariadb,\nwhile secret-rgw and cron-job-backup-mariadb use .Values.secrets.\nidentity.remote_ks_admin and remote_rgw_user.\nIt requires to use same values for admin and remote_ks_admin,\nand for mariadb and remote_rgw_user.\nSeems it isbreaking values consistency.\n\nSuggestion:\nNow providing 2 kinds of backup - pvc and swift.\n\"remote_\" means the swift backup.\nIn fact, mariadb chart has no case to access to keystone except\nswift backup. So we can remove remote_xx_* prefix and there is\nno confusion.\n\nChange-Id: Ib82120611659bd36bae35f2e90054642fb8ee31f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/89db3881717c02326def9eff3e6bc4bc00fb6151"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/89db3881717c02326def9eff3e6bc4bc00fb6151"}]},"branch":"refs/heads/master"},"5db88a5fb487521e1cf20506150d7ec77c520c7c":{"kind":"REWORK","_number":2,"created":"2021-03-03 18:47:16.000000000","uploader":{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"},"ref":"refs/changes/41/778541/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/41/778541/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/41/778541/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/41/778541/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/41/778541/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/41/778541/2"}}},"commit":{"parents":[{"commit":"618c064d2594bdbb56a2d4ca043b6268de2921ab","subject":"Merge \"Add metadata in job templates\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/618c064d2594bdbb56a2d4ca043b6268de2921ab"}]}],"author":{"name":"okozachenko","email":"okozachenko@vexxhost.com","date":"2021-03-03 18:28:05.000000000","tz":120},"committer":{"name":"okozachenko","email":"okozachenko@vexxhost.com","date":"2021-03-03 18:46:51.000000000","tz":120},"subject":"Rename mariadb backup identities","message":"Rename mariadb backup identities\n\nChallenge:\nNow remote_ks_admin and remote_rgw_user are using for user labels\nof backup target openstack cloud.\nWhen the backup user doesn\u0027t exist and we can enable job_ks_user\nmanifest.\nBut job_ks_user uses .Vaules.secrets.identity.admin and mariadb,\nwhile secret-rgw and cron-job-backup-mariadb use .Values.secrets.\nidentity.remote_ks_admin and remote_rgw_user.\nIt requires to use same values for admin and remote_ks_admin,\nand for mariadb and remote_rgw_user.\nSeems it isbreaking values consistency.\n\nSuggestion:\nNow providing 2 kinds of backup - pvc and swift.\n\"remote_\" means the swift backup.\nIn fact, mariadb chart has no case to access to keystone except\nswift backup. So we can remove remote_xx_* prefix and there is\nno confusion.\n\nChange-Id: Ib82120611659bd36bae35f2e90054642fb8ee31f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/5db88a5fb487521e1cf20506150d7ec77c520c7c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/5db88a5fb487521e1cf20506150d7ec77c520c7c"}]},"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":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"}}]}],"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":{}}}]}