)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"11653260006c131e89d9e330abd6e3aff13825e5","unresolved":true,"context_lines":[{"line_number":20,"context_line":"subnode."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"In CI, we need to wait for placement to list the resource provider"},{"line_number":23,"context_line":"otherwise it will fail the cyborg agent service on compute node."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Closes-Bug: #2148278"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9b461de7_43297df4","line":23,"updated":"2026-05-08 17:19:12.000000000","message":"am this is ture in the sensce of ci stablity\n\nthe compute agent actully has a retry loop loop that can tolerate this\nrace condition breifly\n\n\ni belive its 3 retries with an expenetial backoff\n\nbut  since cyborg is a plugin it is start after all the in tree service supprote by devstafck\n\nso nova should alwasy start before cybrog does and the nova supprot in devstack waits for the compute node to show up in the compute service list\nwhich i belive is guareenteed to alwasy be after the resouce provdier is created in placment\n\nso in reality this should jsut work due to how this is sequenced in devstack.\n\nyou dont actullly have any code in the chagne to enforce this so this last part is kind fo irrelevnet.","commit_id":"3aa6854544666077ca160e5530b32fe1346f34a0"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"11653260006c131e89d9e330abd6e3aff13825e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3144d4ba_70fe6543","updated":"2026-05-08 17:19:12.000000000","message":"+1 becase this need a rebase but since hte security isssue is not resovled we can move forward with this\ni didnt want this to cause mege conflict which is why i was holding off on thsi for the last week or two.","commit_id":"3aa6854544666077ca160e5530b32fe1346f34a0"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"71ddc66fb5a8fd2f92e551827100bf22bc5872c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"cf57b398_d4180ed1","updated":"2026-05-21 08:24:37.000000000","message":"https://review.opendev.org/c/openstack/cyborg/+/989165 will likely resolve the unit test failure so we can recheck after that lands","commit_id":"a2bbd5ac278dfd8d57dfd6b4a8f2aaa903a5ec33"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"d1b796d9d4fc8ab1f1e741b0b2de6048d075b314","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2ab7c363_67632024","updated":"2026-05-21 12:23:52.000000000","message":"recheck","commit_id":"a2bbd5ac278dfd8d57dfd6b4a8f2aaa903a5ec33"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"696d2c7b27acf81b4bb8635c9fa2b4feb1a0013c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a2e8d353_42b68ca8","updated":"2026-05-19 14:28:25.000000000","message":"thanks Chandan!","commit_id":"a2bbd5ac278dfd8d57dfd6b4a8f2aaa903a5ec33"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ce0a92188714edfe83d68605fd2e1258c20f101d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"31bd9ce8_73d7883f","updated":"2026-05-19 14:28:31.000000000","message":"yep this also looks good thanks","commit_id":"a2bbd5ac278dfd8d57dfd6b4a8f2aaa903a5ec33"}],"devstack/lib/cyborg":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"924bcadaa8125f5fb27f726e33d4dc39caec01d5","unresolved":true,"context_lines":[{"line_number":139,"context_line":"    # Uses placement service user credentials (not cyborg user) per"},{"line_number":140,"context_line":"    # the configure_cyborg_other_client_section convention."},{"line_number":141,"context_line":"    # It is needed for multinode deployment"},{"line_number":142,"context_line":"    configure_cyborg_other_client_section placement"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    # Add retries so that placement registers the resource provider"},{"line_number":145,"context_line":"    # Set higher than the default (3) because in CI the subnode agent"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"b955b84b_323f7a7e","line":142,"updated":"2026-04-28 14:16:48.000000000","message":"We could switch the custom `configure_cyborg_other_client_section` with the more generic function provided by devstack https://github.com/openstack/devstack/blob/03ece8f88040be9b0b14dd1cfe93076ad2419a80/lib/keystone#L431","commit_id":"3aa6854544666077ca160e5530b32fe1346f34a0"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"a8b22d3c1400cac5b59867eef2ba741fabee4617","unresolved":false,"context_lines":[{"line_number":139,"context_line":"    # Uses placement service user credentials (not cyborg user) per"},{"line_number":140,"context_line":"    # the configure_cyborg_other_client_section convention."},{"line_number":141,"context_line":"    # It is needed for multinode deployment"},{"line_number":142,"context_line":"    configure_cyborg_other_client_section placement"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    # Add retries so that placement registers the resource provider"},{"line_number":145,"context_line":"    # Set higher than the default (3) because in CI the subnode agent"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"0074f4b5_24e0424e","line":142,"in_reply_to":"5b131eaf_58b1326c","updated":"2026-05-19 06:54:13.000000000","message":"Done","commit_id":"3aa6854544666077ca160e5530b32fe1346f34a0"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3c31493dd80c039a86cb04bfedd07214226e8788","unresolved":true,"context_lines":[{"line_number":139,"context_line":"    # Uses placement service user credentials (not cyborg user) per"},{"line_number":140,"context_line":"    # the configure_cyborg_other_client_section convention."},{"line_number":141,"context_line":"    # It is needed for multinode deployment"},{"line_number":142,"context_line":"    configure_cyborg_other_client_section placement"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    # Add retries so that placement registers the resource provider"},{"line_number":145,"context_line":"    # Set higher than the default (3) because in CI the subnode agent"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"5b131eaf_58b1326c","line":142,"in_reply_to":"b955b84b_323f7a7e","updated":"2026-05-08 17:20:12.000000000","message":"i also agree with that.","commit_id":"3aa6854544666077ca160e5530b32fe1346f34a0"}]}