)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"22f1cd8fabaa9b797deee9a69f4036fd8929745e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"25fd2e41_175e1ee9","updated":"2025-05-28 15:33:01.000000000","message":"FYI pretty sure the CI break here is systemmatic, similar to the NGS one I had to fix","commit_id":"349ffec1d23cce30e088767214da291ccb636923"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"c9b65e4f0a261945bda94e5a548364bcaef48500","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e345eeeb_e941143b","updated":"2025-11-17 21:32:27.000000000","message":"I think the name got changed with some confusion, hopefully we can sort that out.\n\nAlso, needs more unit testing and a release note. Thanks!","commit_id":"d6200d8fd1cd84f23893825b47242e86445d73ba"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"95addebf9d364b9ed2cb43641bc624df094bb80d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1bc62ab6_c3c98543","updated":"2025-05-29 17:24:50.000000000","message":"I think we should test this before landing to ensure it works as expected; I\u0027m concerned we might need additional filtering or hookups in the rpc layer.","commit_id":"d6200d8fd1cd84f23893825b47242e86445d73ba"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"e459f9000c923e354800632915da323c969b3f8f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"7f01a8ce_e850fb18","updated":"2025-11-18 22:43:07.000000000","message":"Looks like a change is still required here, please let me know if I\u0027m interpretting the api structure incorrectly or there is some sort of hidden alias in the sdk.","commit_id":"e256cae8c0915950303488654e341b5cf3582b7b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"78f680322a49ca1de6034e31a81060797222e44f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"f37f4fd1_d8a8e00a","updated":"2026-02-05 16:12:54.000000000","message":"Looks like this will need 4.9.0dev-something, so as soon as the next openstacksdk is released, we can wrap this one up.","commit_id":"e256cae8c0915950303488654e341b5cf3582b7b"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"823b65aefcf959f4710f008e5d04983304ce2c44","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e027bafa_2665cee2","updated":"2026-01-22 19:09:02.000000000","message":"This depends on I60885738db1ecb7a9525302ec1e22f7f2fbcd6c6 and we\u0027ll need a bump to the package for a release as well no? Dropping ironic-week-prio until this is resolved.","commit_id":"e256cae8c0915950303488654e341b5cf3582b7b"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"ae88681b1c0e01e7ca704a87ec96cb4d8175d807","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"56d57e37_13dcda6a","in_reply_to":"f37f4fd1_d8a8e00a","updated":"2026-02-05 18:08:37.000000000","message":"The release is already cut. Patch updated. tks! :-)","commit_id":"e256cae8c0915950303488654e341b5cf3582b7b"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"a7bbe1ac7f4a506cfffd70c11636134667a67910","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"a3aa3732_b343fda5","updated":"2026-02-25 00:56:19.000000000","message":"Recheck apparently the test is fixed now","commit_id":"079870110db97b3103244eca07dbaa50e4fecf65"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"bac90fc2be3eb68212afb1454aa72ed7a787ac5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"f737a801_84508d63","updated":"2026-02-26 14:48:13.000000000","message":"Recheck more testing fixes have landed","commit_id":"079870110db97b3103244eca07dbaa50e4fecf65"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"b30c3b8e776a52609e96b9c241bfc9829ac469aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"68931456_829257bd","updated":"2026-02-24 18:11:41.000000000","message":"recheck - neutron-dhcp race condition that\u0027s now been fixed","commit_id":"079870110db97b3103244eca07dbaa50e4fecf65"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"19f4eb608f44a1518f82f8d3ee209485909d04ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"00eac439_8ac7d336","updated":"2026-02-24 16:57:12.000000000","message":"recheck - unrelated CI failure.","commit_id":"079870110db97b3103244eca07dbaa50e4fecf65"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b94e4a0e170054f89c6ba8b38b4b8075d74b2856","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c244b9d8_7e3edd48","updated":"2026-02-25 02:50:04.000000000","message":"recheck ci should be fixed now","commit_id":"079870110db97b3103244eca07dbaa50e4fecf65"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f5ad896df732cda81b33ba8d616cde1c0468cde4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c84bf464_918b61dc","updated":"2026-02-26 00:02:18.000000000","message":"recheck multinode job fix merged","commit_id":"079870110db97b3103244eca07dbaa50e4fecf65"}],"networking_baremetal/agent/ironic_neutron_agent.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"c9b65e4f0a261945bda94e5a548364bcaef48500","unresolved":true,"context_lines":[{"line_number":221,"context_line":"    def _report_state(self):"},{"line_number":222,"context_line":"        node_states \u003d {}"},{"line_number":223,"context_line":"        shards \u003d getattr(CONF, \u0027sharding\u0027, None)"},{"line_number":224,"context_line":"        conductor_groups \u003d getattr(shards, \u0027conductor_groups\u0027, None) or []"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"        if conductor_groups:"},{"line_number":227,"context_line":"            LOG.info(\"Using conductor groups filter: %s\", conductor_groups)"}],"source_content_type":"text/x-python","patch_set":7,"id":"62dcab01_fa841597","line":224,"updated":"2025-11-17 21:32:27.000000000","message":"so, we hand the list of shards from the config along as conductor groups, that seems wrong?","commit_id":"d6200d8fd1cd84f23893825b47242e86445d73ba"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"c9b65e4f0a261945bda94e5a548364bcaef48500","unresolved":true,"context_lines":[{"line_number":227,"context_line":"            LOG.info(\"Using conductor groups filter: %s\", conductor_groups)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        ironic_ports \u003d self.ironic_client.ports("},{"line_number":230,"context_line":"            details\u003dTrue, conductor_groups\u003dconductor_groups)"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        # NOTE: the above calls returns a generator, so we need to handle"},{"line_number":233,"context_line":"        # exceptions that happen just before the first loop iteration, when"}],"source_content_type":"text/x-python","patch_set":7,"id":"528ce90a_64ab1380","line":230,"updated":"2025-11-17 21:32:27.000000000","message":"so shard is a parameter it looks like.\n\nAlso, there is no conductor_group argument on the API: https://docs.openstack.org/api-ref/baremetal/#id88","commit_id":"d6200d8fd1cd84f23893825b47242e86445d73ba"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a0ee8dc8ca90f8c6337f3c6f1d6fd216c81cf68d","unresolved":true,"context_lines":[{"line_number":227,"context_line":"            LOG.info(\"Using conductor groups filter: %s\", conductor_groups)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        ironic_ports \u003d self.ironic_client.ports("},{"line_number":230,"context_line":"            details\u003dTrue, conductor_groups\u003dconductor_groups)"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        # NOTE: the above calls returns a generator, so we need to handle"},{"line_number":233,"context_line":"        # exceptions that happen just before the first loop iteration, when"}],"source_content_type":"text/x-python","patch_set":7,"id":"e43380e4_4b6bda75","line":230,"in_reply_to":"0d7689f7_84dcf3ea","updated":"2026-01-06 17:38:14.000000000","message":"Okay, that is a start, the next issue is the sdk needs to be updated then. The max version on openstacksdk is 1.97 for the port resources and the conductor_groups option is not listed there as an available argument.","commit_id":"d6200d8fd1cd84f23893825b47242e86445d73ba"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"0c6a83d377803174fc502be630599b05ebe0932f","unresolved":true,"context_lines":[{"line_number":227,"context_line":"            LOG.info(\"Using conductor groups filter: %s\", conductor_groups)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        ironic_ports \u003d self.ironic_client.ports("},{"line_number":230,"context_line":"            details\u003dTrue, conductor_groups\u003dconductor_groups)"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        # NOTE: the above calls returns a generator, so we need to handle"},{"line_number":233,"context_line":"        # exceptions that happen just before the first loop iteration, when"}],"source_content_type":"text/x-python","patch_set":7,"id":"6625b81a_1fa69605","line":230,"in_reply_to":"4d4676c7_9adb827b","updated":"2025-11-18 22:51:27.000000000","message":"So, here\u0027s the precursor to the networking-baremetal change: https://review.opendev.org/c/openstack/ironic/+/862292.\n\nIs that not the endpoint this call hits?","commit_id":"d6200d8fd1cd84f23893825b47242e86445d73ba"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"e459f9000c923e354800632915da323c969b3f8f","unresolved":true,"context_lines":[{"line_number":227,"context_line":"            LOG.info(\"Using conductor groups filter: %s\", conductor_groups)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        ironic_ports \u003d self.ironic_client.ports("},{"line_number":230,"context_line":"            details\u003dTrue, conductor_groups\u003dconductor_groups)"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        # NOTE: the above calls returns a generator, so we need to handle"},{"line_number":233,"context_line":"        # exceptions that happen just before the first loop iteration, when"}],"source_content_type":"text/x-python","patch_set":7,"id":"4d4676c7_9adb827b","line":230,"in_reply_to":"528ce90a_64ab1380","updated":"2025-11-18 22:43:07.000000000","message":"Greetings, re-instating my -1 since it doesn\u0027t look like conductor_groups is a valid argument.","commit_id":"d6200d8fd1cd84f23893825b47242e86445d73ba"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"fdca05a1d238a96a86966c01bae4732fb69b2246","unresolved":true,"context_lines":[{"line_number":227,"context_line":"            LOG.info(\"Using conductor groups filter: %s\", conductor_groups)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        ironic_ports \u003d self.ironic_client.ports("},{"line_number":230,"context_line":"            details\u003dTrue, conductor_groups\u003dconductor_groups)"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        # NOTE: the above calls returns a generator, so we need to handle"},{"line_number":233,"context_line":"        # exceptions that happen just before the first loop iteration, when"}],"source_content_type":"text/x-python","patch_set":7,"id":"0d7689f7_84dcf3ea","line":230,"in_reply_to":"6625b81a_1fa69605","updated":"2025-12-10 19:33:46.000000000","message":"I think you\u0027re right.  The API does have support for the conductor_groups param, Jay and I found out it just wasn\u0027t correctly documented. Maybe that\u0027s the missing bit?\n\nFix here -\u003e https://review.opendev.org/c/openstack/ironic/+/970482","commit_id":"d6200d8fd1cd84f23893825b47242e86445d73ba"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"c10e82206747c70c53b13a145cd54de549f26ee3","unresolved":true,"context_lines":[{"line_number":227,"context_line":"            LOG.info(\"Using conductor groups filter: %s\", conductor_groups)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        ironic_ports \u003d self.ironic_client.ports("},{"line_number":230,"context_line":"            details\u003dTrue, conductor_groups\u003dconductor_groups)"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        # NOTE: the above calls returns a generator, so we need to handle"},{"line_number":233,"context_line":"        # exceptions that happen just before the first loop iteration, when"}],"source_content_type":"text/x-python","patch_set":7,"id":"fe8b7d46_03c24269","line":230,"in_reply_to":"e43380e4_4b6bda75","updated":"2026-01-13 18:33:18.000000000","message":"I created the PR here: https://review.opendev.org/c/openstack/openstacksdk/+/973287","commit_id":"d6200d8fd1cd84f23893825b47242e86445d73ba"}],"networking_baremetal/tests/unit/ironic_agent/test_ironic_agent.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"c9b65e4f0a261945bda94e5a548364bcaef48500","unresolved":true,"context_lines":[{"line_number":264,"context_line":"        self.agent \u003d ironic_neutron_agent.BaremetalNeutronAgent()"},{"line_number":265,"context_line":"        self.agent.ironic_client \u003d mock_conn"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"        def mock_generator(details\u003dNone):"},{"line_number":268,"context_line":"            raise sdk_exc.OpenStackCloudException()"},{"line_number":269,"context_line":"            yield"},{"line_number":270,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"450cd8e8_7b9c2b19","side":"PARENT","line":267,"updated":"2025-11-17 21:32:27.000000000","message":"Hate to say this, I think we need more exhaustive testing including simulation of a value.","commit_id":"4a33028bc4f543d77d33070ee2744330c8354f51"}],"releasenotes/notes/conductor-groups-filtering-support.yaml":[{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"baeed78d09e84c2e05e8a0a46c505d3e3208a45d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds support for filtering ports by conductor groups."}],"source_content_type":"text/x-yaml","patch_set":9,"id":"c7e12057_a30fca9e","line":1,"range":{"start_line":1,"start_character":1,"end_line":1,"end_character":2},"updated":"2026-02-05 18:04:12.000000000","message":"DOS to UNIX line ending conversion is responsible for the updated timestamp here.","commit_id":"079870110db97b3103244eca07dbaa50e4fecf65"}]}