)]}'
{"id":"openstack%2Fopenstack-helm~961316","triplet_id":"openstack%2Fopenstack-helm~master~Ieb94bebda58d050a9698ff949f8f051ed0b6b625","project":"openstack/openstack-helm","branch":"master","attention_set":{"3009":{"account":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"last_update":"2025-09-29 18:00:54.000000000","reason":"\u003cGERRIT_ACCOUNT_5890\u003e replied on the change","reason_account":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"}}},"removed_from_attention_set":{"5890":{"account":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"last_update":"2025-09-29 18:00:54.000000000","reason":"\u003cGERRIT_ACCOUNT_5890\u003e replied on the change","reason_account":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"}}},"hashtags":[],"change_id":"Ieb94bebda58d050a9698ff949f8f051ed0b6b625","subject":"DNM: possible approach to re-orient charts so they can be split","status":"NEW","created":"2025-09-15 21:50:50.000000000","updated":"2025-09-29 18:00:54.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":false,"submittable":false,"total_comment_count":4,"unresolved_comment_count":2,"has_review_started":true,"meta_rev_id":"0bd072ed7418ba91771624780b9014e75b235ab9","_number":961316,"virtual_id_number":961316,"owner":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2025-09-15 23:22:40.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":{"all":[{"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":[{"_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":{"CC":[{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"_account_id":34520,"name":"Sergiy Markin","email":"smarkin@mirantis.com","username":"sm515x"}],"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-09-15 21:52:50.000000000","updated_by":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"reviewer":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"state":"CC"},{"updated":"2025-09-15 21:52:50.000000000","updated_by":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"reviewer":{"_account_id":34520,"name":"Sergiy Markin","email":"smarkin@mirantis.com","username":"sm515x"},"state":"CC"},{"updated":"2025-09-15 21:52:50.000000000","updated_by":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"reviewer":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"state":"CC"},{"updated":"2025-09-15 21:52:50.000000000","updated_by":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"reviewer":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"state":"CC"},{"updated":"2025-09-15 23:22:40.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":"2025-09-27 22:25:32.000000000","updated_by":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"reviewer":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"state":"CC"}],"messages":[{"id":"07db3f22fc4d23ccdbda530573833de662b7a70a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"date":"2025-09-15 21:50:50.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"15190af84230df9ad4d8c73906f4404c315477d3","author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"date":"2025-09-15 21:52:50.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"f38dc6bcd3e55589d77bab280890cc05daff7ff7","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-15 23:22:40.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/6cc631e717794fe5add980238ec88d0d\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/7fa3b81bcf0b47d892d980588532fa43 : SUCCESS in 4m 43s\n- openstack-helm-linter https://zuul.opendev.org/t/openstack/build/fdd5ae47aca847cfbe00c744505d868b : FAILURE in 2m 54s\n- openstack-helm-pre-commit https://zuul.opendev.org/t/openstack/build/8a45da2bf2a84598a93343c417ec73f4 : SUCCESS in 2m 53s\n- openstack-helm-tls-2024-1-ubuntu_jammy https://zuul.opendev.org/t/openstack/build/80adf333f42745658a08d32e0fa35916 : SUCCESS in 1h 28m 54s (non-voting)\n- openstack-helm-compute-kit-2024-1-ubuntu_jammy https://zuul.opendev.org/t/openstack/build/208cc9e664614df79eb399f6744d99cd : SUCCESS in 1h 16m 36s\n- openstack-helm-tacker-2024-1-ubuntu_jammy https://zuul.opendev.org/t/openstack/build/2b34a1fb38434dd3b8c5c07f8c7bba9e : SUCCESS in 50m 28s\n- openstack-helm-compute-kit-2024-2-ubuntu_jammy https://zuul.opendev.org/t/openstack/build/5775ebe37b684693bd62f336b60bec36 : SUCCESS in 1h 18m 12s\n- openstack-helm-compute-kit-2025-1-ubuntu_jammy https://zuul.opendev.org/t/openstack/build/023ce20073db49ad9a7f18b69f1505d3 : SUCCESS in 1h 05m 28s\n- openstack-helm-skyline-2025-1-ubuntu_jammy https://zuul.opendev.org/t/openstack/build/3fe715a217bd4979a9004d4911e49250 : SUCCESS in 44m 23s\n- openstack-helm-octavia-2025-1-ubuntu_jammy https://zuul.opendev.org/t/openstack/build/d11f2f7cb660431aa64819f2a5b0dc1f : SUCCESS in 1h 17m 44s\n- openstack-helm-compute-kit-2025-1-ubuntu_noble https://zuul.opendev.org/t/openstack/build/9026f20ee3254d99b03d9a5b17293196 : SUCCESS in 1h 20m 36s\n- openstack-helm-logging https://zuul.opendev.org/t/openstack/build/6af5515c5fe8431aa938a8c233691f59 : SUCCESS in 35m 15s\n- openstack-helm-monitoring https://zuul.opendev.org/t/openstack/build/9aa3836503124fa2b56ee38a47df54eb : SUCCESS in 31m 19s\n- openstack-helm-metacontroller https://zuul.opendev.org/t/openstack/build/e801d2f784f04369a7a8927d3d53fe37 : SUCCESS in 14m 52s","accounts_in_message":[],"_revision_number":1},{"id":"30ce4de0065215825708f4f54119cdb633f0e795","author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"date":"2025-09-29 16:53:36.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"70939b7fb2461965354dbda88854521fd3eb1315","author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"date":"2025-09-29 17:19:14.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"0bd072ed7418ba91771624780b9014e75b235ab9","author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"date":"2025-09-29 18:00:54.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"947b89544d7c6f4b42b27e22cdc097c61a350fc5","revisions":{"947b89544d7c6f4b42b27e22cdc097c61a350fc5":{"kind":"REWORK","_number":1,"created":"2025-09-15 21:50:50.000000000","uploader":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"ref":"refs/changes/16/961316/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/16/961316/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/16/961316/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/16/961316/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/16/961316/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/16/961316/1"}}},"commit":{"parents":[{"commit":"a4df9c709e0a022d14daaaa998a8d54c2ca0ab8c","subject":"Merge \"updated blazar configs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/a4df9c709e0a022d14daaaa998a8d54c2ca0ab8c"}]}],"author":{"name":"Doug Goldstein","email":"cardoe@cardoe.com","date":"2025-09-15 21:42:33.000000000","tz":-300},"committer":{"name":"Doug Goldstein","email":"cardoe@cardoe.com","date":"2025-09-15 21:50:44.000000000","tz":-300},"subject":"DNM: possible approach to re-orient charts so they can be split","message":"DNM: possible approach to re-orient charts so they can be split\n\nThis re-orients the values.yaml to follow the sub-chart style documented\nby Helm at https://helm.sh/docs/chart_template_guide/subcharts_and_globals/\nwhich utilizes the special \"globals\" key that would be shared to\nsub-charts. With this we could then split up certain charts like,\nironic, nova, cinder, and neutron for example, into sub-charts of their\nAPI and then related services. So for example with Ironic it is common\nto create Conductor Groups once you go over a certain amount of hosts.\nSo you would have an ironic-api chart and an ironic-conductor chart, you\nmight deploy the ironic-api once but the ironic-conductor chart a few\ntimes with different configurations that would utilize the same\nironic-api. The default case we would provide a chart called \u0027ironic\u0027\nwhich would include both charts and behave as things behave today. The\n\u0027ironic\u0027 chart would have keys \u0027api\u0027 and \u0027conductor\u0027 where those\nspecific configs would be passed down into the sub-chart. This is a\nchange from how the values.yaml is organized today but that\u0027s what this\nlayout demonstrates while adding a backwards compat shim which could\nexist for a release cycle or two. Similarly nova I would want to break\nout nova-cells so that a multi-cell deployment would work. With Neutron\nthe agents are deployed in different availability zones. While cinder\nhas the cinder-volume deployed multiple times.\n\nAssisted-By: Claude Code\nChange-Id: Ieb94bebda58d050a9698ff949f8f051ed0b6b625\nSigned-off-by: Doug Goldstein \u003ccardoe@cardoe.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/947b89544d7c6f4b42b27e22cdc097c61a350fc5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/947b89544d7c6f4b42b27e22cdc097c61a350fc5"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY"},{"label":"Workflow","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}