)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"e7ad5447ea385365b553595c7b2841176ef4868b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d3daeb6c_ef446fe8","updated":"2026-09-21 09:39:31.000000000","message":"I agree with the change, but as stated on the commit message, it depends on a specific method being added to python-observabilityclient. However, in this patch, the python-observabilityclient is still \u003e\u003d1.1.0, which will not work when the current change gets merged.\n\nSo I would say that we need to update the requirements.txt to whatever new version of observabilityclient contains the change and only then can this patch be merged safely.","commit_id":"89ec1bc3e402018d28afea07ecef4df8e0e37b52"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"f3befc7fc33cba24d311f6e8abef0fb155a8be80","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"55181554_48ea83f6","updated":"2026-09-21 09:37:21.000000000","message":"Since this depends on new functionality in the observabilityclient, I think this should also bump the observabilityclient version in the requirements.txt, although I\u0027m not sure how bumping to a not yet released client version should work.","commit_id":"89ec1bc3e402018d28afea07ecef4df8e0e37b52"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"61f9c819954bc728c09d82a9d203e9a62ff55520","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"970dd378_3c88fac2","updated":"2026-09-18 11:54:59.000000000","message":"recheck","commit_id":"89ec1bc3e402018d28afea07ecef4df8e0e37b52"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"0f1b992d098c9b015f0922bb2908f6e8f69134af","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b5538cf8_30bca814","in_reply_to":"48d8f816_839403ae","updated":"2026-09-21 11:22:25.000000000","message":"ack","commit_id":"89ec1bc3e402018d28afea07ecef4df8e0e37b52"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"c69a1e16250e13ae23bc1fccd6e7f9cf6f339e40","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"48d8f816_839403ae","in_reply_to":"55181554_48ea83f6","updated":"2026-09-21 09:40:06.000000000","message":"My comment crossed with Jaromir\u0027s 😢","commit_id":"89ec1bc3e402018d28afea07ecef4df8e0e37b52"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"f65e5774782b042235f70565f9936e12ba04ec23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"af76253d_ed83eb71","updated":"2026-09-21 11:23:05.000000000","message":"Waiting for new python-observabilityclient release","commit_id":"3a341119923ed35139939643542c55646e064e9e"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"f0265a4a937620e7f612c5def61a2909d47a51e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"145d5be3_30a99493","updated":"2026-10-22 12:23:08.000000000","message":"@dpawlik@redhat.com We have a similar patch proposed for ceilometer and we\u0027re discussing whether to default to 1.3 or not. Whatever the decision, we should be consistent with the defaults and adjust this patch if needed as well https://review.opendev.org/c/openstack/ceilometer/+/1011826","commit_id":"dfdab1290d41966c8b222436fbef1af533992fd1"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4f78c07de1d77336fb97a0907a147c867d4c72e7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"7a6633fd_4f1cd195","in_reply_to":"145d5be3_30a99493","updated":"2026-10-27 16:16:48.000000000","message":"I\u0027m approving the change which uses 1.3 as default for min_version so please update this change accordingly.","commit_id":"dfdab1290d41966c8b222436fbef1af533992fd1"}]}
