)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"273d215116ec26578bb5648ad27e835f6ec2bfd9","unresolved":true,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"[0] https://review.opendev.org/c/openstack/neutron-lib/+/965087"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Depends-On: https://review.opendev.org/c/openstack/python-openstackclient/+/1000685"},{"line_number":22,"context_line":"Related-Bug: #2107316"},{"line_number":23,"context_line":"Change-Id: Ibf82d1f9002393367221404a263a11f08d3c93ce"},{"line_number":24,"context_line":"Signed-off-by: Kyuyeong Lee \u003ckyu0.lee@samsung.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"aa8afb35_d169391c","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":83},"updated":"2026-08-14 12:18:04.000000000","message":"the dependency is the other direction, you have to have the SKD patch merged and released first to make the OSC patch progress","commit_id":"089ddf28f1163e0c6bb8835fe36c572f041ece4f"},{"author":{"_account_id":36716,"name":"Kyuyeong Lee","display_name":"Kyuyeong Lee","email":"kyu0.lee@samsung.com","username":"kyu0"},"change_message_id":"6e95ba6213d1389619523ad171406d4653fe4068","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"[0] https://review.opendev.org/c/openstack/neutron-lib/+/965087"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Depends-On: https://review.opendev.org/c/openstack/python-openstackclient/+/1000685"},{"line_number":22,"context_line":"Related-Bug: #2107316"},{"line_number":23,"context_line":"Change-Id: Ibf82d1f9002393367221404a263a11f08d3c93ce"},{"line_number":24,"context_line":"Signed-off-by: Kyuyeong Lee \u003ckyu0.lee@samsung.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"8029ce11_154229a0","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":83},"in_reply_to":"a05a51e5_7abf0dde","updated":"2026-08-19 10:39:58.000000000","message":"That was my first instinct too, but the gate says otherwise -- this change cannot merge before the OSC one.\n\nDeclaring the property adds `ip_availability_details` to to_dict(), which adds a column to `openstack ip availability show`. The column list in TestShowIPAvailability is hardcoded, so OSC master fails as soon as this merges. That is what PS1 hit, with no Depends-On:\n\n  osc-tox-py311-tips ... FAILURE\n  osc-tox-py314-tips ... FAILURE\n\n  openstackclient.tests.unit.network.v2.test_ip_availability.TestShowIPAvailability.test_show_all_options ... FAILED\n    expected \u003d (\u0027network_id\u0027, ...)\n    actual   \u003d (\u0027ip_availability_details\u0027, \u0027network_id\u0027, ...)\n\nBoth jobs are voting, so without the OSC change applied this one stays Verified-1. With the Depends-On added in PS2 they pass.\n\nThe reverse direction isn\u0027t needed: the OSC change does not require this one at runtime. NetworkIPAvailability sets _allow_unknown_attrs_in_body, so the attribute is passed through on the currently required openstacksdk (4.16.0) as well, and the OSC unit tests pass against both 4.16.0 and this change.\n\nThe requirements bump does need this to be released first, but Brian asked for that on the OSC change and we agreed to handle it in a follow-up, so it does not gate the OSC change either.\n\nHappy to restructure if you\u0027d rather not have it this way -- the alternative would be to split the test expectation update out of the OSC change into a separate precursor patch. Just let me know which you prefer.","commit_id":"089ddf28f1163e0c6bb8835fe36c572f041ece4f"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d906da042f28bb113211514b2457aeb775befe15","unresolved":true,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"[0] https://review.opendev.org/c/openstack/neutron-lib/+/965087"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Depends-On: https://review.opendev.org/c/openstack/python-openstackclient/+/1000685"},{"line_number":22,"context_line":"Related-Bug: #2107316"},{"line_number":23,"context_line":"Change-Id: Ibf82d1f9002393367221404a263a11f08d3c93ce"},{"line_number":24,"context_line":"Signed-off-by: Kyuyeong Lee \u003ckyu0.lee@samsung.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a05a51e5_7abf0dde","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":83},"in_reply_to":"aa8afb35_d169391c","updated":"2026-08-19 10:25:49.000000000","message":"Yeah, it looks like this is backwards. It seems the OSC patch depends on this patch. If so, you want a `Depends-On` in the OSC patch, not this one.","commit_id":"089ddf28f1163e0c6bb8835fe36c572f041ece4f"}]}