)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"1d46bce64444221b761253417f33bd1c1b749186","unresolved":true,"context_lines":[{"line_number":10,"context_line":"enrolling nodes to support more config fields and updating"},{"line_number":11,"context_line":"existing baremetal nodes in config."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Support enrolling other baremetal nodes into the baremetal"},{"line_number":14,"context_line":"service by creating a new `baremetal` group. By default this"},{"line_number":15,"context_line":"group only contains `baremetal-compute`; operators can extend"},{"line_number":16,"context_line":"it in their own inventory with additional groups."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":33,"id":"34bf2f3e_2054bbaf","line":13,"updated":"2026-09-21 11:48:01.000000000","message":"Dumb question. Why was the new group necessary rather than just using baremetal compute?","commit_id":"92e761b9a0ded4f8728d4e1626fa28be071e4009"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"9a4c232696254d988b281f2f9553800e3d7c8156","unresolved":true,"context_lines":[{"line_number":10,"context_line":"enrolling nodes to support more config fields and updating"},{"line_number":11,"context_line":"existing baremetal nodes in config."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Support enrolling other baremetal nodes into the baremetal"},{"line_number":14,"context_line":"service by creating a new `baremetal` group. By default this"},{"line_number":15,"context_line":"group only contains `baremetal-compute`; operators can extend"},{"line_number":16,"context_line":"it in their own inventory with additional groups."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":33,"id":"1f9d960d_be740b58","line":13,"in_reply_to":"34bf2f3e_2054bbaf","updated":"2026-09-21 13:43:08.000000000","message":"I think the idea is that `baremetal-compute` is for Ironic provisioned hypervisors. `baremetal` is the catchall for everything.","commit_id":"92e761b9a0ded4f8728d4e1626fa28be071e4009"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":39213,"name":"Jed Preist","display_name":"Jed Preist","email":"jed@stackhpc.com","username":"jed-preist"},"change_message_id":"b571a072e276a51e0b8d59895ef05e9e798d2819","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":30,"id":"e43ece1a_88c7ef86","updated":"2026-09-02 10:25:36.000000000","message":"A couple of issues:\n\n1. The nodes are already being enrolled by tenks, which would need to be disabled to test baremetal node registration properly.\n\n2. There is a mix of redfish and ipmi nodes, so the driver needs to be set per node. Right now it\u0027s failing trying to re-enrol a redfish node as an ipmi node","commit_id":"7a573698944f107379d225ae07875a28668750df"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"de07533b95f3d3de4d3290f5666ba8f15b594d66","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":30,"id":"abcc80fe_8f179fa6","in_reply_to":"5b109580_394b18b9","updated":"2026-10-19 15:57:16.000000000","message":"thanks for looking into it Jed","commit_id":"7a573698944f107379d225ae07875a28668750df"},{"author":{"_account_id":39213,"name":"Jed Preist","display_name":"Jed Preist","email":"jed@stackhpc.com","username":"jed-preist"},"change_message_id":"6945e8bf5bbddf1f0f2c48ee8a761638552b50f9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":30,"id":"5b109580_394b18b9","in_reply_to":"9ea496d7_22b75965","updated":"2026-10-13 09:10:58.000000000","message":"Disabling enrolment via tenks would mean no baremetal ports are created, so CI would need to be configured to do DHCP with ironic-dnsmasq. I think that is a big enough change that it should be a separate patch.","commit_id":"7a573698944f107379d225ae07875a28668750df"},{"author":{"_account_id":39213,"name":"Jed Preist","display_name":"Jed Preist","email":"jed@stackhpc.com","username":"jed-preist"},"change_message_id":"93eb95713651d001cf3a8ed18449ab513cc472ad","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":30,"id":"9ea496d7_22b75965","in_reply_to":"e43ece1a_88c7ef86","updated":"2026-09-11 08:23:16.000000000","message":"Host vars are set to override the driver info for the redfish node now","commit_id":"7a573698944f107379d225ae07875a28668750df"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"2fd7b31d7541ca7d54ed81e5e23bf627b73a7fd6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"9722248d_8b7ce9e8","updated":"2026-10-13 10:45:45.000000000","message":"Feature freeze. Lets merge this after 2026.2 is released.","commit_id":"26c92fd58892279738d1b94f49d86750628487c6"}],"ansible/baremetal-compute-register.yml":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"eed805186f08ba3000b0a4ffa58d16cd90a23d5f","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        deploy_interface: \"{{ baremetal_compute_deploy_interface }}\""},{"line_number":42,"context_line":"        driver: \"{{ baremetal_compute_driver }}\""},{"line_number":43,"context_line":"        driver_info: \"{{ baremetal_compute_driver_info }}\""},{"line_number":44,"context_line":"        id: \"{{ baremetal_compute_uuid }}\""},{"line_number":45,"context_line":"        inspect_interface: \"{{ baremetal_compute_inspect_interface }}\""},{"line_number":46,"context_line":"        interface: \"{{ openstack_interface | default(omit, true) }}\""},{"line_number":47,"context_line":"        management_interface: \"{{ baremetal_compute_management_interface }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"646ba762_ab131307","line":44,"updated":"2025-06-20 15:18:14.000000000","message":"I think we need to default omit this... as usually we just let ironic.","commit_id":"263a74081d793bbe11297b4b5bc257ce79b6f6df"},{"author":{"_account_id":36238,"name":"Hollie Hutchinson","display_name":"Hollie Hutchinson","email":"hollie@stackhpc.com","username":"assumptions"},"change_message_id":"be987430b0f960b2dd9f3d2f5a77348fb18e3997","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        deploy_interface: \"{{ baremetal_compute_deploy_interface }}\""},{"line_number":42,"context_line":"        driver: \"{{ baremetal_compute_driver }}\""},{"line_number":43,"context_line":"        driver_info: \"{{ baremetal_compute_driver_info }}\""},{"line_number":44,"context_line":"        id: \"{{ baremetal_compute_uuid }}\""},{"line_number":45,"context_line":"        inspect_interface: \"{{ baremetal_compute_inspect_interface }}\""},{"line_number":46,"context_line":"        interface: \"{{ openstack_interface | default(omit, true) }}\""},{"line_number":47,"context_line":"        management_interface: \"{{ baremetal_compute_management_interface }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4040d10a_c7831e20","line":44,"in_reply_to":"646ba762_ab131307","updated":"2025-06-26 16:12:56.000000000","message":"Done","commit_id":"263a74081d793bbe11297b4b5bc257ce79b6f6df"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"eed805186f08ba3000b0a4ffa58d16cd90a23d5f","unresolved":true,"context_lines":[{"line_number":55,"context_line":"        raid_interface: \"{{ baremetal_compute_raid_interface }}\""},{"line_number":56,"context_line":"        rescue_interface: \"{{ baremetal_compute_rescue_interface }}\""},{"line_number":57,"context_line":"        resource_class: \"{{ baremetal_compute_resource_class }}\""},{"line_number":58,"context_line":"        state: \"{{ \u0027absent\u0027 if baremetal_compute_delete_nodes else \u0027present\u0027 }}\""},{"line_number":59,"context_line":"        storage_interface: \"{{ baremetal_compute_storage_interface }}\""},{"line_number":60,"context_line":"        timeout: \"{{ baremetal_compute_timeout }}\""},{"line_number":61,"context_line":"        vendor_interface: \"{{ baremetal_compute_vendor_interface }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"71efe8b8_dc869356","line":58,"updated":"2025-06-20 15:18:14.000000000","message":"Lets hardwire this to \"present\" for now... We might need to rename this command eventually :)","commit_id":"263a74081d793bbe11297b4b5bc257ce79b6f6df"},{"author":{"_account_id":36238,"name":"Hollie Hutchinson","display_name":"Hollie Hutchinson","email":"hollie@stackhpc.com","username":"assumptions"},"change_message_id":"be987430b0f960b2dd9f3d2f5a77348fb18e3997","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        raid_interface: \"{{ baremetal_compute_raid_interface }}\""},{"line_number":56,"context_line":"        rescue_interface: \"{{ baremetal_compute_rescue_interface }}\""},{"line_number":57,"context_line":"        resource_class: \"{{ baremetal_compute_resource_class }}\""},{"line_number":58,"context_line":"        state: \"{{ \u0027absent\u0027 if baremetal_compute_delete_nodes else \u0027present\u0027 }}\""},{"line_number":59,"context_line":"        storage_interface: \"{{ baremetal_compute_storage_interface }}\""},{"line_number":60,"context_line":"        timeout: \"{{ baremetal_compute_timeout }}\""},{"line_number":61,"context_line":"        vendor_interface: \"{{ baremetal_compute_vendor_interface }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f3bdf7a_c09fd27e","line":58,"in_reply_to":"71efe8b8_dc869356","updated":"2025-06-26 16:12:56.000000000","message":"Done","commit_id":"263a74081d793bbe11297b4b5bc257ce79b6f6df"}],"kayobe/cli/commands.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acb9799670faf25e6a1c940c38f6e6df1a993421","unresolved":false,"context_lines":[{"line_number":2224,"context_line":"        playbooks \u003d _build_playbook_list(\"baremetal-node-register\")"},{"line_number":2225,"context_line":"        self.run_kayobe_playbooks(parsed_args, playbooks)"},{"line_number":2226,"context_line":""},{"line_number":2227,"context_line":"class BaremetalNodeRegister(KayobeAnsibleMixin, VaultMixin, Command):"},{"line_number":2228,"context_line":"    \"\"\"Register baremetal nodes in Ironic.\"\"\""},{"line_number":2229,"context_line":""},{"line_number":2230,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":22,"id":"4e3e136f_8880dc3d","line":2227,"updated":"2026-07-23 18:03:16.000000000","message":"pep8: E302 expected 2 blank lines, found 1","commit_id":"7b0093018a6df76fb25b9b639825a8cd60255b63"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acb9799670faf25e6a1c940c38f6e6df1a993421","unresolved":false,"context_lines":[{"line_number":2232,"context_line":"        playbooks \u003d _build_playbook_list(\"baremetal-node-register\")"},{"line_number":2233,"context_line":"        self.run_kayobe_playbooks(parsed_args, playbooks)"},{"line_number":2234,"context_line":""},{"line_number":2235,"context_line":"class BaremetalComputeInspect(KayobeAnsibleMixin, VaultMixin, Command):"},{"line_number":2236,"context_line":"    \"\"\"Perform hardware inspection on baremetal compute nodes.\"\"\""},{"line_number":2237,"context_line":""},{"line_number":2238,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":22,"id":"8af6b684_9a4c09ee","line":2235,"updated":"2026-07-23 18:03:16.000000000","message":"pep8: E302 expected 2 blank lines, found 1","commit_id":"7b0093018a6df76fb25b9b639825a8cd60255b63"}],"requirements.yml":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"aa0abf1f27d0a74e8d8126faae0469b67545de8f","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  # installed from Galaxy below, since it is merged upstream but not yet in"},{"line_number":12,"context_line":"  # a release. Revert to the Galaxy release of openstack.cloud when it is."},{"line_number":13,"context_line":"  - name: https://github.com/stackhpc/ansible-collections-openstack"},{"line_number":14,"context_line":"    type: git"},{"line_number":15,"context_line":"    version: feature/remove_mac_requirement_v2"},{"line_number":16,"context_line":"  - name: community.docker"},{"line_number":17,"context_line":"    version: 5.3.0"}],"source_content_type":"text/x-yaml","patch_set":39,"id":"d7175178_712d92cb","line":14,"updated":"2026-10-19 15:56:30.000000000","message":"I\u0027ve raised this on #openstack-sdks - we just need a new release and we can drop the fork","commit_id":"4a130f668a736440466966c257c13cf02606777d"}]}
