)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"d516eb396a40a5f835d809bce2e640811a8832b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6a265ffb_e6c1d457","updated":"2023-11-30 13:28:28.000000000","message":"Does this affect other services too? I see share.py which follows the same pattern, but I haven\u0027t checked.\n\nWould it make sense to add unit tests?\nAnd a release note?","commit_id":"aa037e9ce093e87a7b22000c4013be247c645b71"},{"author":{"_account_id":35007,"name":"Chi Wai CHAN","display_name":"Ray Chan","email":"chiwai.chan@canonical.com","username":"Ray"},"change_message_id":"e015f071f5879031b28d01608f5b2524f9f1d628","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4a531354_ea420ed5","updated":"2023-11-29 03:38:35.000000000","message":"Hi Martin,\n\nIn our cloud, we\u0027ve configured our endpoints to have path based suffixes (e.g. `http://IP:PORT/\u003csome_path\u003e/v2`), and the discovery tool isn\u0027t working properly with this kind of endpoints because it cuts too much (`\u003csome_path\u003e/v2` this is example) when the port is **not** None. I tried to work around this by using `utils.get_base_url` instead of `no_port_cut_url` to properly remove the version in the service URL, this should also be backward-compatible. \n\nCan I ask you opinion regarding this approach / issue?\n\nThank you.","commit_id":"aa037e9ce093e87a7b22000c4013be247c645b71"},{"author":{"_account_id":32363,"name":"Robert Gildein","email":"robert.gildein@canonical.com","username":"rgildein"},"change_message_id":"48ef412f87f630a21b8a715b29d7a64de47d0558","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4ef014d5_9412aa54","updated":"2023-11-29 21:43:48.000000000","message":"These changes seem reasonable to me.","commit_id":"aa037e9ce093e87a7b22000c4013be247c645b71"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"a328a5d9679fbb1241513d6ae1015bfbc619c41a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cbae9b89_70d0c807","updated":"2023-11-30 13:02:07.000000000","message":"check experimental","commit_id":"aa037e9ce093e87a7b22000c4013be247c645b71"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"1221023acc9fab382f57617fd07f84900e8c4f59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5792922a_05c60e16","updated":"2023-11-30 13:02:00.000000000","message":"looks reasonable and the gates seem to agree","commit_id":"aa037e9ce093e87a7b22000c4013be247c645b71"},{"author":{"_account_id":35007,"name":"Chi Wai CHAN","display_name":"Ray Chan","email":"chiwai.chan@canonical.com","username":"Ray"},"change_message_id":"b435073fece04ccf747b24f4aff413c5080ad468","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"65c1ea69_a5626f67","updated":"2023-12-07 06:05:45.000000000","message":"would appreciate any follow up on this, thanks!","commit_id":"aa037e9ce093e87a7b22000c4013be247c645b71"},{"author":{"_account_id":35007,"name":"Chi Wai CHAN","display_name":"Ray Chan","email":"chiwai.chan@canonical.com","username":"Ray"},"change_message_id":"7fc02b3449acd16b80b16911def664c0eea2b68e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"25c8b899_cb9349f8","in_reply_to":"4cad41a6_bed2ff87","updated":"2023-12-12 07:28:08.000000000","message":"I\u0027ve included the fix for share service, as well as some unit test for `get_base_url` since it\u0027s the primary function used the `set_versions` method","commit_id":"aa037e9ce093e87a7b22000c4013be247c645b71"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"02aef18924a3781293ae243846e3d8c8f289efac","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4cad41a6_bed2ff87","in_reply_to":"54f42403_2af2e22a","updated":"2023-12-11 08:47:48.000000000","message":"I think that if this issue influences other services (ShareService) we should fix it for this service as well (if it is an easy fix, which it seems to be in this case).","commit_id":"aa037e9ce093e87a7b22000c4013be247c645b71"},{"author":{"_account_id":35007,"name":"Chi Wai CHAN","display_name":"Ray Chan","email":"chiwai.chan@canonical.com","username":"Ray"},"change_message_id":"70104398c4fbb9613b2af0919bdf3016673f6db0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"54f42403_2af2e22a","in_reply_to":"6a265ffb_e6c1d457","updated":"2023-12-01 03:31:37.000000000","message":"\u003e Does this affect other services too?\n\nPossibly yes, but in our cloud we don\u0027t have `ShareService` so I did not see it as part of the story.\n\n\u003e Would it make sense to add unit tests?\n\nYes, I initially thought of adding one, but not exactly sure how can I do it easily without copy-and-pasting the test because the test is intended to test *one* possible pattern of [FAKE_URL](https://opendev.org/openinfra/python-tempestconf/src/branch/master/config_tempest/tests/services/test_compute.py#L28), do you happen to have some suggestions?\n\n\u003e And a release note?\n\nI am assuming it\u0027s for the maintainer?","commit_id":"aa037e9ce093e87a7b22000c4013be247c645b71"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"75396034d0a701b51322796119b223326e13ddb5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c7215437_4fcb58f3","updated":"2023-12-13 09:47:43.000000000","message":"Looks good to me!","commit_id":"3366caea2c44af16674e3c3b5361d127150330a6"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"8f4315cb13c1c15807ec37108d76d9dc4004e59c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3f9da03c_384b372d","updated":"2023-12-13 16:31:35.000000000","message":"belated +2, thanks for the unit tests","commit_id":"3366caea2c44af16674e3c3b5361d127150330a6"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"3bc81daf44f5d9a861c9c2ab5bcc0b293eca95f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9df72ea4_0db5dfe5","updated":"2023-12-13 14:10:12.000000000","message":"recheck timeout","commit_id":"3366caea2c44af16674e3c3b5361d127150330a6"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"0ffa07ecb45112566b57aba880e9b48e2cb9022d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4e281643_3d63f06e","updated":"2023-12-12 13:22:14.000000000","message":"thanks, looks good to me now","commit_id":"3366caea2c44af16674e3c3b5361d127150330a6"}]}