)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"2248683400874e62f33b0eff9cfd9ba5d59a7b60","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7c733da1_1e3aadf8","updated":"2023-09-29 09:50:48.000000000","message":"Hi Martin. Is this project listed in governance repository? Nevertheless, it\u0027s too late to add this deliverables in 2023.2 Bobcat release, but it\u0027s good to start early for 2024.1 Caracal.\n\nThe first issues need to be solved can be seen in our validator job\u0027s log [1].\n(Note that deliverables/caracal/ will be initialized after the 2023.2 Bobcat release - which is planned to happen middle of next week)\n\n[1] https://zuul.opendev.org/t/openstack/build/663e0b2586bb4541acbad308dd68dfe5/log/tox/validate/validate-request-results.log#757-759","commit_id":"860ed47715db5b43622de86edff9f37a8caa1132"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"419d174253e74629abf57dcc758a4dfd4ed851da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2cf468f6_ed753b22","updated":"2023-10-12 09:35:06.000000000","message":"I believe I fixed CI issues with following patches:\n\nhttps://review.opendev.org/c/openstack/python-observabilityclient/+/898092\nhttps://review.opendev.org/c/openstack/project-config/+/898098","commit_id":"860ed47715db5b43622de86edff9f37a8caa1132"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"902d1f5d5698eeb427406796146a7f66b37953da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1a0c2845_5c72e8e9","updated":"2023-10-11 13:24:45.000000000","message":"Martin, there are zuul errors that can be fixed already. Can you please take a look?","commit_id":"860ed47715db5b43622de86edff9f37a8caa1132"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"f016a4dd18cfc420288965229262cfaf1581f236","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c990ca13_54d2ba6f","in_reply_to":"1a0c2845_5c72e8e9","updated":"2023-10-11 13:31:26.000000000","message":"Yepp, the errors [1] look valid to me. Also note, that the patch [2] for the new cycle\u0027s deliverables is already up in gerrit, so you can either wait until that merges or rebase your patch on top of that.\n\n[1] https://zuul.opendev.org/t/openstack/build/663e0b2586bb4541acbad308dd68dfe5/log/tox/validate/validate-request-results.log#757-759\n[2] https://review.opendev.org/c/openstack/releases/+/897653","commit_id":"860ed47715db5b43622de86edff9f37a8caa1132"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"b99b7d3a87a9f794d67d83eb9b83ba39e19bc67b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5a359fff_48df8efd","in_reply_to":"7c733da1_1e3aadf8","updated":"2023-10-02 10:46:39.000000000","message":"Hi Elod, yes it is listed [1]. Yes I didn\u0027t expect this patch to get to Bobcat, but there is not new release directory prepared. Once there will be, I\u0027ll update it for Caracal and fix those two things. Thanks for the heads up.\n\n[1] https://review.opendev.org/c/openstack/governance/+/894915","commit_id":"860ed47715db5b43622de86edff9f37a8caa1132"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"e9249a6af76fd66fbfdfafa0f375e47525c1dd1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e002f638_23629223","updated":"2023-10-12 09:59:22.000000000","message":"Elod, apparently [1] need some help of zuul admin. By any chance can you help with this?\n\n[1] https://review.opendev.org/c/openstack/project-config/+/898098","commit_id":"aa51366ac0f5f0f2a24c2163f7cf4b5405d62b7a"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"d2e9b3ac0d50eae6765f5c3a5d421edb68d0a0dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"22b3e086_e7b9c291","updated":"2023-10-13 15:17:52.000000000","message":"recheck","commit_id":"aa51366ac0f5f0f2a24c2163f7cf4b5405d62b7a"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b217e2b9fad3919019ccf4ab15c02428fc3032ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ae73579b_5dcda5b7","updated":"2023-10-13 13:34:15.000000000","message":"recheck - config changes have been merged","commit_id":"aa51366ac0f5f0f2a24c2163f7cf4b5405d62b7a"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"2e5b8cd1c42f899f2d17b16d8795e8fac57979ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"44b6331c_09c6f4ec","updated":"2023-10-16 15:06:20.000000000","message":"recheck","commit_id":"810287d1750d4c7f34261fdb3eaa2ed1f17f8bea"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"31415d86e548dc823f5ff3e771fcfa7fdaa51cb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a82bcd86_d4c714ae","updated":"2023-10-17 08:20:09.000000000","message":"Thanks for fixing all the issues and configuring all the necessary things. I have one comment, which should be addressed, otherwise this is getting fine. Giving -1 to highlight my comment.","commit_id":"701a080aa0e0824b745e59589de168053c68ea26"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"acffc3fb8ef89a9550b8cf0eff9decc1cef39c7a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"06123ad3_1d17cacd","updated":"2023-10-17 12:40:59.000000000","message":"thank you Martin","commit_id":"b065222c2b1268bf4cee08a59bcb4e3609172ef5"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"317aa6c772eeb2d9c68b8b45aff71f57a87b4521","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"672e6146_4baeb1db","updated":"2023-10-17 14:55:28.000000000","message":"(I\u0027ve removed the whitespace only)","commit_id":"5ebb311212665fc22f9f4d5eb7b2cc7e6f76c09d"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"7c9f00839587dc257d8f321be5e91b203526537f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e511ee7a_d2d04b23","updated":"2023-10-18 06:26:28.000000000","message":"Thank you for your review. I think I can address your concerns for now.\nVersion numbers are just arbitrary numbers. I agree that jumping from 0.0.4 to 1.0.0 sounds like a huge change, and I also agree that 0.1.0 looks much better.","commit_id":"5ebb311212665fc22f9f4d5eb7b2cc7e6f76c09d"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"2421b8fa8443fe488648662833c1dbc12fdd70ac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a7626385_44b0b30b","updated":"2023-10-17 15:08:26.000000000","message":"thank you,","commit_id":"5ebb311212665fc22f9f4d5eb7b2cc7e6f76c09d"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"41ca93a08b0410b04679a0c10017e764b8e96817","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"7b21ea9e_adebb1cd","updated":"2023-10-19 00:08:54.000000000","message":"LGTM the hash is valid and the versioning makes sense. Thanks all.","commit_id":"123dcab982070b767d3060e10e0c01311fc9d561"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"6ee6728940bf196c30b84d8f2e5ac9a25e051def","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ae6500dd_16968b94","updated":"2023-10-18 12:52:25.000000000","message":"Thanks for the reviews \u0026 updates. LGTM. Giving +1 for now, but will update as soon as we\u0027ll get a review +1 from release liaison or PTL.","commit_id":"123dcab982070b767d3060e10e0c01311fc9d561"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"89f7af53cd43d4cdcce5ab90230ea8ac080d32eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"699771fb_de3dbe98","updated":"2023-10-19 07:46:12.000000000","message":"Upgrading to +2","commit_id":"123dcab982070b767d3060e10e0c01311fc9d561"}],"deliverables/caracal/python-observabilityclient.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"31415d86e548dc823f5ff3e771fcfa7fdaa51cb2","unresolved":true,"context_lines":[{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/python-observabilityclient"},{"line_number":13,"context_line":"        hash: 26687730b3c0f9c95fadfd378c9887cbd0c16ba4"},{"line_number":14,"context_line":"branches:"},{"line_number":15,"context_line":"  - name: stable/2024.1"},{"line_number":16,"context_line":"    location: 1.0.0"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"182dcf30_e7f5b46e","line":16,"range":{"start_line":14,"start_character":0,"end_line":16,"end_character":19},"updated":"2023-10-17 08:20:09.000000000","message":"let\u0027s not cut branch yet for stable/2024.1 (please see the schedule [1] of 2024.1 Caracal cycle about when the cut should happen).","commit_id":"701a080aa0e0824b745e59589de168053c68ea26"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"3615fc0eb8f77323b2f0e8ad9e29e481f92f280c","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/python-observabilityclient: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 1.0.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/python-observabilityclient"},{"line_number":13,"context_line":"        hash: 26687730b3c0f9c95fadfd378c9887cbd0c16ba4"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3ccb3ffe_aed109af","line":10,"updated":"2023-10-18 04:44:44.000000000","message":"Going from 0.0.4 to 1.0.0 seems a very big step and is sending a stability assumption to potential users that you may not be able to keep. So my suggestion would be to start with 0.1.0 and possibly target a major release towards the end of the cycle.\n\nOnce concern is the use of the \"openstack metric\" namespace, which we might want to be shared somehow between multiple metrics backends/clients. IMO this should be discussed with the OSC team before doing a major release.","commit_id":"5ebb311212665fc22f9f4d5eb7b2cc7e6f76c09d"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"7c9f00839587dc257d8f321be5e91b203526537f","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/python-observabilityclient: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 1.0.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/python-observabilityclient"},{"line_number":13,"context_line":"        hash: 26687730b3c0f9c95fadfd378c9887cbd0c16ba4"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"e1e01fb8_42245a93","line":10,"in_reply_to":"3ccb3ffe_aed109af","updated":"2023-10-18 06:26:28.000000000","message":"At this point, the library is not intented as a replacement for gnocchiclient, thus not interfering with osc.\nIt may be a good idea to add this though. However, you can have both: sending metrics to gnocchi and exposing metrics for prometheus to scrape (and also aodh pulling metrics from both).","commit_id":"5ebb311212665fc22f9f4d5eb7b2cc7e6f76c09d"}]}