)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"a252e01147513852c386f8c56b5877fbab008551","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5afccd0c_2a5d0e23","updated":"2023-01-24 13:24:26.000000000","message":"Afaik it is not a doc issue, when I run\nopenstack loadbalancer healthmonitor create --help\nI see that only a single pool can be attached to the healthmonitor","commit_id":"617f01f2a3a9f5b0b0f98677162a4dc74d1b33a9"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"3887fa692d0ed00d4c418155945e95f818597190","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6dd76f70_ae0cb56b","updated":"2023-01-24 12:30:21.000000000","message":"I am not sure if I should add release notes as well","commit_id":"617f01f2a3a9f5b0b0f98677162a4dc74d1b33a9"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"a217d65fbea6140f0a495f86383282f12c5c4c3d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cc5b4d92_10322574","updated":"2023-01-24 13:10:41.000000000","message":"If it\u0027s a doc issue, the doc needs to be fixed\n","commit_id":"617f01f2a3a9f5b0b0f98677162a4dc74d1b33a9"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"58bb1fcd469fe845b06b4fbc26896d1107aa13d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"87dab5a8_c71eb6e7","updated":"2023-01-24 13:01:09.000000000","message":"hmm here you are changing the API, I think it will break projects that use Octavia.","commit_id":"617f01f2a3a9f5b0b0f98677162a4dc74d1b33a9"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"c5ceda3ac8e9104e73db310e4662dfa5b5c79147","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5da9d0af_e384ba13","in_reply_to":"5afccd0c_2a5d0e23","updated":"2023-01-24 15:20:25.000000000","message":"\"shared health monitors\" is still on the roadmap, I believe this is the reason why a HM can have multiple pools attached in the API.\n\nthis patch breaks existing clients, after applying it, when I do \"openstack loadbalancer healthmonitor show \u003cid\u003e\", the pool is missing:\n\n```\n+---------------------+--------------------------------------+\n| Field               | Value                                |\n+---------------------+--------------------------------------+\n| project_id          | 594103c8355645ffb776097e17824983     |\n| name                | hm1                                  |\n| admin_state_up      | True                                 |\n| pools               |                                      |\n  ^^^^^                 ^^^^^^^^^^\n| created_at          | 2023-01-24T12:49:02                  |\n| provisioning_status | ACTIVE                               |\n| updated_at          | 2023-01-24T12:49:05                  |\n| delay               | 5                                    |\n| expected_codes      | None                                 |\n| max_retries         | 3                                    |\n| http_method         | None                                 |\n| timeout             | 4                                    |\n| max_retries_down    | 3                                    |\n| url_path            | None                                 |\n| type                | TCP                                  |\n| id                  | 0d4a2394-8bfc-438d-a1dd-6c1aef812294 |\n| operating_status    | ONLINE                               |\n| http_version        | None                                 |\n| domain_name         | None                                 |\n| tags                |                                      |\n+---------------------+--------------------------------------+\n```","commit_id":"617f01f2a3a9f5b0b0f98677162a4dc74d1b33a9"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"2f9b22636c2748ccfe07d94d9ae771cb1b7ffa3d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b8378ed6_b90c7caf","in_reply_to":"5da9d0af_e384ba13","updated":"2023-01-24 15:36:17.000000000","message":"Thank you for testing it.\nSo as \"shared health monitors\" feature is still on the roadmap, should we change both the doc in the octavia repo and the description on the\nopenstack loadbalancer healthmonitor show \u003chm\u003e\non the python-octaviaclient repo? Because right now the octavia docs mention \"pool_id\", the response of healthmonitor show command returns \"pools\", and \nopenstack loadbalancer healthmonitor create --help shows:\n\u003cpool\u003e        Set the pool for the health monitor (name or ID).","commit_id":"617f01f2a3a9f5b0b0f98677162a4dc74d1b33a9"}]}