)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"fedad0be617cdf758dadcb6b123f0ac788ab6d58","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1250ea85_29c40d78","updated":"2023-08-05 16:07:05.000000000","message":"4.4.x and 4.5.x stable releases should be available next week: https://github.com/gnocchixyz/gnocchi/pull/1314#issuecomment-1666512985","commit_id":"789fbe3665c71d3f83d1d6acf1c1fd8edf39589b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e0317cecf1b8d2f7936d9d0ae942beafd3c65f4f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9c01d7bf_ce2c0b84","updated":"2023-08-05 09:06:27.000000000","message":"Agree with Pierre","commit_id":"789fbe3665c71d3f83d1d6acf1c1fd8edf39589b"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"21713f8bd77e0b01a69776a0bdf706100c592fae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"249ea690_ef2639e5","updated":"2023-08-05 09:00:35.000000000","message":"I am not keen on building from master for stable branches in general, but this one in particular is problematic because it drops support for Python 3.6.","commit_id":"789fbe3665c71d3f83d1d6acf1c1fd8edf39589b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"d0fe1cbb694f52d4591ccb2a8f47bc11a6e3903e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"71574bbd_13d6af5f","updated":"2023-08-05 18:15:22.000000000","message":"Just friendly note, if kolla had merged my patches some time ago, then we would able fix this with  a local patchfile in kolla repository :)  And remove it after they release regular version.\n\nhttps://review.opendev.org/c/openstack/kolla/+/829295\nhttps://review.opendev.org/c/openstack/kolla/+/829296/1\n\nIn my downstream repo i have this patches :\n\npatches/\npatches/gnocchi-base\npatches/gnocchi-base/fix-gnocchi-build.patch\npatches/gnocchi-base/series\npatches/openstack-base\npatches/openstack-base/disable-greenthreads-for-rabbitdriver-listen-connections.patch\npatches/openstack-base/series\n\nAnd I have no problem :) \n\n\n\n\nmichalarbet@pixla:~/ultimum/git/ultimum/kolla$ cat patches/openstack-base/disable-greenthreads-for-rabbitdriver-listen-connections.patch \nDescription: Disable greenthreads for RabbitDriver \"listen\" connections\n When enabling heartbeat_in_pthread, we were restoring the \"threading\"\n python library from eventlet to original one in RabbitDriver but we\n forgot to do the same in AMQPDriverBase (RabbitDriver is subclass of\n AMQPDriverBase).\n \n We also need to use the original \"queue\" so that queues are not going to\n use greenthreads as well.\n \n Related-bug: #1961402\n Related-bug: #1934937\n Closes-bug: #2009138\n\nAuthor: Arnaud Morin \u003carnaud.morin@gmail.com\u003e\nDate: Mon, 15 May 2023 11:16:56 +0100\nOrigin: upstream, https://review.opendev.org/c/openstack/oslo.messaging/+/880189\nChange-Id: I34ea0d1381e934297df2f793e0d2594ef8254f00\nLast-Update: 2023-08-05\n\n--- /var/lib/kolla/venv/lib/python3.9/site-packages/oslo_messaging/_drivers/impl_rabbit.py\n+++ /var/lib/kolla/venv/lib/python3.9/site-packages/oslo_messaging/_drivers/impl_rabbit.py\n@@ -541,6 +541,8 @@\n             # if it was already monkey patched by eventlet/greenlet.\n             global threading\n             threading \u003d _utils.stdlib_threading\n+            amqpdriver.threading \u003d _utils.stdlib_threading\n+            amqpdriver.queue \u003d _utils.stdlib_queue\n \n         self.direct_mandatory_flag \u003d driver_conf.direct_mandatory_flag\n \n--- /var/lib/kolla/venv/lib/python3.9/site-packages/oslo_messaging/_utils.py\n+++ /var/lib/kolla/venv/lib/python3.9/site-packages/oslo_messaging/_utils.py\n@@ -14,6 +14,7 @@\n #    under the License.\n \n import logging\n+import queue\n import threading\n \n from oslo_utils import eventletutils\nmichalarbet@pixla:~/ultimum/git/ultimum/kolla$ cat patches/openstack-base/\ndisable-greenthreads-for-rabbitdriver-listen-connections.patch  series                                                          \nmichalarbet@pixla:~/ultimum/git/ultimum/kolla$ cat patches/gnocchi-base/\nfix-gnocchi-build.patch  series                   \nmichalarbet@pixla:~/ultimum/git/ultimum/kolla$ cat patches/gnocchi-base/fix-gnocchi-build.patch \nDescription: Fix multiple CI issues\n\n The 3.1.0 release of pyparsing introduced a change\n or regression in the error message that we parse\n in the gabbit tests so we pin until that is investigated\n in the issue [1].\n .\n setup: fix usage of deprecated get_script_header()\n For now we catch the AttributeError and use the new\n function if the deprecated function does not exist\n so that we don\u0027t break older versions.\n .\n tox: remove editable wheels when installing\n We do this to unblock installation using newer\n pip versions that would require us to rewrite\n more logic.\nAuthor: Tobias Urdin\nDate: Sat, 04 Aug 2023 10:56:47 +0000\nOrigin: upstream, https://github.com/gnocchixyz/gnocchi/pull/1314\nLast-Update: 2023-08-05\n\n--- /gnocchi-base-source/gnocchi-base-archive-4.5.0/setup.cfg\n+++ /gnocchi-base-source/gnocchi-base-archive-4.5.0/setup.cfg\n@@ -47,7 +47,9 @@ install_requires \u003d\n     Paste\n     PasteDeploy\n     daiquiri\n-    pyparsing\u003e\u003d2.2.0\n+    # TODO(tobias-urdin): Evaluate unpin later\n+    # https://github.com/pyparsing/pyparsing/issues/504\n+    pyparsing\u003e\u003d2.2.0,\u003c3.1.0\n     lz4\u003e\u003d0.9.0\n     tooz\u003e\u003d1.38\n     cachetools\nindex b79eb951..e95fd038 100755\n--- /gnocchi-base-source/gnocchi-base-archive-4.5.0/setup.py\n+++ /gnocchi-base-source/gnocchi-base-archive-4.5.0/setup.py\n@@ -63,7 +63,12 @@ class local_install_scripts(install_scripts.install_scripts):\n         # replaced by the correct interpreter. We do the same here.\n         bs_cmd \u003d self.get_finalized_command(\u0027build_scripts\u0027)\n         executable \u003d getattr(bs_cmd, \u0027executable\u0027, easy_install.sys_executable)\n-        script \u003d easy_install.get_script_header(\"\", executable) + SCRIPT_TMPL\n+        try:\n+            script \u003d easy_install.get_script_header(\n+                \"\", executable) + SCRIPT_TMPL\n+        except AttributeError:\n+            script \u003d easy_install.ScriptWriter.get_header(\n+                \"\", executable) + SCRIPT_TMPL\n         if PY3:\n             script \u003d script.encode(\u0027ascii\u0027)\n         self.write_script(\"gnocchi-api\", script, \u0027b\u0027)\n@@ -74,7 +79,10 @@ class local_develop(develop.develop):\n         develop.develop.install_wrapper_scripts(self, dist)\n         if self.exclude_scripts:\n             return\n-        script \u003d easy_install.get_script_header(\"\") + SCRIPT_TMPL\n+        try:\n+            script \u003d easy_install.get_script_header(\"\") + SCRIPT_TMPL\n+        except AttributeError:\n+            script \u003d easy_install.ScriptWriter.get_header(\"\") + SCRIPT_TMPL\n         if PY3:\n             script \u003d script.encode(\u0027ascii\u0027)\n         self.write_script(\"gnocchi-api\", script, \u0027b\u0027)\nindex 7272d248..5f257f81 100644\n--- /gnocchi-base-source/gnocchi-base-archive-4.5.0/tox.ini\n+++ /gnocchi-base-source/gnocchi-base-archive-4.5.0/tox.ini\n@@ -45,7 +45,6 @@ setenv \u003d\n # coordination driver in functional tests (--coordination-driver is passed to\n # pifpaf)\n deps \u003d\n-   -e\n    .[test,redis,prometheus,amqp1,{env:GNOCCHI_STORAGE_DEPS:},{env:GNOCCHI_INDEXER_DEPS:}]\n    {env:GNOCCHI_TEST_TARBALLS:}\n    # TODO(tobias-urdin): Remove this pin and use pifpaf directly instead when this is\n@@ -127,7 +126,6 @@ basepython \u003d python3\n #        .[postgresql,doc]\n # setenv \u003d GNOCCHI_STORAGE_DEPS\u003dfile\n deps \u003d\n-    -e\n     .[test,file,postgresql,doc]\n     doc8\n setenv \u003d GNOCCHI_TEST_DEBUG\u003d1\n \n \n After regular fix will be release ... i will just delete patch files... simple and easy ...\n \n \n Maybe we can re-open this discussion, what do you think ?","commit_id":"789fbe3665c71d3f83d1d6acf1c1fd8edf39589b"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"67fa8d65f0a9402c2f6f67cc0e72faafe1ccfc54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5d00d3dd_ef274de1","updated":"2023-08-07 07:49:15.000000000","message":"let\u0027s wait for the release","commit_id":"789fbe3665c71d3f83d1d6acf1c1fd8edf39589b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"f88b29580a5e3d4da7d2fec9dbc5e3033fdb74f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f0f62395_08d97924","in_reply_to":"71574bbd_13d6af5f","updated":"2023-08-05 18:17:41.000000000","message":"Ups, Gerrit is formatting patch file :D :D :D - but it\u0027s classic patchfile generated with diff -u orig.txt orig.txt.patched  - ignore the formatting","commit_id":"789fbe3665c71d3f83d1d6acf1c1fd8edf39589b"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"c500fee8c9bf76f8e846229d83e36f44a05d6b0f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b679fe15_8c550f47","updated":"2023-09-05 13:55:11.000000000","message":"recheck packages.grafana.com not in sync","commit_id":"feed9d87f3365f400c2d25d94ccfa4ec997798b0"}]}
