)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"4e748c3d9c3245a9a0027c16be944151219e7609","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"de4ed38b_ba14b7e4","updated":"2026-02-16 22:10:28.000000000","message":"Raising the review priority since this is blocking the telemetry gate and we\u0027re starting to have quite a few changes potentially ready to merge.","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b3e22f335be5a00c968469ee1fc511e7e97de443","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8b552be5_f77beaff","updated":"2026-02-16 19:30:07.000000000","message":"as a temproy messure this si proably ok but if we need to this for any more plugins i think we woudl be better of addign a explcit BUILD_ISOLATION varable to devstack instead.","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8378e9dc9804ff0038129d53e88176afe06605df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1018cff6_3dfb98a5","updated":"2026-02-16 15:22:39.000000000","message":"this shoudl really be done in the relevent plugin rather then in devstck.\nis there a reason you didnt do it in in \n\nhttps://github.com/openstack/ceilometer/blob/20756c9bd8ccb5e25c61e68769d04d4913725128/devstack/plugin.sh#L139-L146","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"}],"inc/python":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"73a4750642ed4d1afc59b515925ba5cd5a1bb2aa","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"08b95e85_a5c368bc","line":214,"updated":"2026-02-16 15:27:07.000000000","message":"disablinbg build isoloation is intened to be temporay so maybe this is fine\nbut we want to reenable it once we have properly pinned setuptools or otherwise resolve the pkg_resouces problems.\n\nsince gnochi installation is not part of core devstack i was hopign we could avoid modifying devstack for it but perhaps we need to have a flag for this that you can set ot false in your jobs.","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"1ab0278eefe2fda3bb10f4ed658d89f5a24b9ba2","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ccd3aa38_3f28cc8f","line":214,"in_reply_to":"08b95e85_a5c368bc","updated":"2026-02-16 15:37:10.000000000","message":"I didn\u0027t see a way to influence this from the devstack plugin except for maybe reimplementing the pip_install, which seems like an overkill. Which is why I added the condition.","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5e73259a8412ce5fba87495862fe7c3a54c8556c","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9aea2ee9_26228f75","line":214,"in_reply_to":"08b95e85_a5c368bc","updated":"2026-02-16 15:37:37.000000000","message":"so in gnoicchi 7.2.0 they removed the usage of pkg_resocues and that was merged in 2024 \n\nhttps://github.com/gnocchixyz/python-gnocchiclient/commit/0f386b689fb0a116f9d8c2ed72983e9f01fbc5a5\n\nso we shoudl have been using that version in stable/2025.1\n\nbut the job looks liek its using a very old version \n\nhttps://zuul.opendev.org/t/openstack/build/f090761b56e14b07b1a3b230c5d94d26/log/job-output.txt#31130-31159\n4.2.0\n\nwhy is that?","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"65954606581be0574c886c753eee1119253f4dbe","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9d1420a9_a7d6b0c7","line":214,"in_reply_to":"3338eb89_1ca604c8","updated":"2026-02-16 16:33:03.000000000","message":"btw, the same (version 0.0.0) happens if I try to install Gnocchi with `--no-build-isolation` from master","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b3e22f335be5a00c968469ee1fc511e7e97de443","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"89a37c83_5a11de3f","line":214,"in_reply_to":"3a024471_14730640","updated":"2026-02-16 19:30:07.000000000","message":"yep but that shoudl not chagne so that a packaging bug in gnochii\nwe that said we can work aroudn it for now\n\nas noted above disabling build isolation is a temproy meesure until all our deps are update to not break with setuptool 82 or the pgk resouces issue is genreally fixed\n\ngnochii 4.6 still has a hard depency on pkg_resouces \n\nhttps://github.com/gnocchixyz/gnocchi/blob/stable/4.6/gnocchi/__init__.py#L18\nso it is not installbale with setuptools 82.\n\nso it woudl fail to install and build on stable if it was not being pinned\n\n\nbuild isolation unpins it but they seam to have goten lucky as its only a runtime dep not a built time dep or itfaile with build isolation enabled\n\nmost project either dpened on it at both runtime and build tiem or not at all.","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"4e748c3d9c3245a9a0027c16be944151219e7609","unresolved":false,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3a61d762_6a43c11a","line":214,"in_reply_to":"89a37c83_5a11de3f","updated":"2026-02-16 22:10:28.000000000","message":"Thank you for taking a look at this so fast @smooney@redhat.com","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"465fc7562a736eb2841cdf66a4e792e5de3d5f7d","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3a024471_14730640","line":214,"in_reply_to":"9992276b_16a97b02","updated":"2026-02-16 19:08:41.000000000","message":"Ceilometer is hitting Gnocchi\u0027s REST API to get the server version. The value fgor the server version returned by `gnocchi_client.build.get()` is based on the package version, which in this case is `0.0.0` due to `--no-build-isolation` being used.","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"a2d1eb99cf5065cab59c1c24cfd6e0d56570af6f","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"d4847d25_81c3043f","line":214,"in_reply_to":"9aea2ee9_26228f75","updated":"2026-02-16 15:44:54.000000000","message":"I think you\u0027re looking at the wrong repo. Gnocchi is here https://github.com/gnocchixyz/gnocchi and the newest I see is 4.7.0\n\nstable/2025.1 should use gnocchi stable/4.6 https://github.com/openstack/telemetry-tempest-plugin/blob/master/.zuul.yaml#L82","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3831edcda26e9d0e67c570a5e4e898550bc25ffe","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"a04d7ad0_6e9d6d59","line":214,"in_reply_to":"9aea2ee9_26228f75","updated":"2026-02-16 15:44:23.000000000","message":"note \n2025.1 allowed 7.1.0\n2025.2 allows 7.2.0\n\nhttps://github.com/openstack/requirements/blob/stable/2025.1/upper-constraints.txt#L143\nhttps://github.com/openstack/requirements/blob/stable/2025.2/upper-constraints.txt#L139\n\nthe 4.2.0 is proably coming form here\n\nhttps://github.com/openstack/ceilometer/blob/master/ceilometer/gnocchi_client.py#L281\n\n\nbut i suspect the real fix woudl be to update \n\nhttps://github.com/openstack/ceilometer/blob/master/ceilometer/gnocchi_client.py#L287\n\nto use importlib.metadata.version(\u0027gnocchiclient\u0027) and backport that instead\nor \nhttps://github.com/gnocchixyz/python-gnocchiclient/blob/master/gnocchiclient/v1/client.py#L67\nhttps://github.com/gnocchixyz/python-gnocchiclient/blob/03a0e6ea052d8717b444e41dbbeca5e82b013fa3/gnocchiclient/v1/build.py#L19","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"24288011ce707e71457179bf8393b53a68c17ccc","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9992276b_16a97b02","line":214,"in_reply_to":"9d1420a9_a7d6b0c7","updated":"2026-02-16 17:02:16.000000000","message":"this honestly sound like a ceilometer or gnocci bug to me but if other are ok with it we can proably proceed witht eh workaorund\n\n\nthe ceilometer code shoudl really be hittign ghooics rest api to get the version which shoudl not depend on the way the package is installed","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6d22b1cd1eaeb3141993177191ba2ae26a6a55f9","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"d897c2b2_60d6c82d","line":214,"in_reply_to":"d4847d25_81c3043f","updated":"2026-02-16 15:51:43.000000000","message":"ceilometn depens on gnochi client rather then gnochi\n\ngnocchi its self was fixed in \n\nhttps://github.com/gnocchixyz/gnocchi/commit/7c90f348d7aba9155e50da4886f93c45c8d9f381\n\n\nwhich you could check the version of with \nimportlib.metadata.version(\u0027gnocchi\u0027)\nhowever that is not correct for cepimter to do as it cant assume gnoich is installed on teh same host\n\nwgucg us why i was suggesting you check only the clinet version since that is the only thing you can directly depend on.\n\nhttps://github.com/gnocchixyz/gnocchi/releases/tag/4.7.0 fixed setup tools comaptiably by removing the pkg_resouces dependcy \n\nthat was relest in august prior to the 2025.2 release but close enough that it make ssens that you may not have swapped testing ot usign it.","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"49147212fca99555e5e2ecbe4a23312216ff7081","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    # build-isolation, so disable it only if we aren\u0027t installing Gnocchi"},{"line_number":212,"context_line":"    if [[ ! \"$@\" \u003d~ \"gnocchi\" ]]; then"},{"line_number":213,"context_line":"        cmd_pip\u003d\"$cmd_pip --no-build-isolation\""},{"line_number":214,"context_line":"    fi"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    $xtrace"},{"line_number":217,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3338eb89_1ca604c8","line":214,"in_reply_to":"d897c2b2_60d6c82d","updated":"2026-02-16 16:14:26.000000000","message":"The problem is not with pkg_resources.\n\nOur tempest tests are relying on gnocchi actually being deployed and working. So just satisfying a gnocchiclient dependency isn\u0027t enough for CI to pass.\n\nThe issue here is, as far as I see, if `-no-build-isolation` is used when installing gnocchi, it installs as version 0.0.0 for some reason (I honestly have no clue why). And this version is then visible in the gnocchi API responses.\n\nCeilometer actually checks the gnocchi version in its code and requires at least 4.2.0 (0.0.0 obviously doesn\u0027t pass that check).\n\nYou can see that ceilometer\u0027s grenade job was failing ever since the setuptools pin was merged https://zuul.opendev.org/t/openstack/builds?job_name\u003dgrenade-ceilometer\u0026project\u003dopenstack/ceilometer\n\nBut it passes when using Depends-On on this patch here: https://zuul.opendev.org/t/openstack/build/3c66b4240d3b467685e4fca6446d4b08","commit_id":"58e156c2cd144283e8aea6432a9468cdaefc3972"}]}