)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9d379c14e67af0c9d84834f49a91f74326de527d","unresolved":true,"context_lines":[{"line_number":10,"context_line":"Support for python3.6 in jsonschema was dropped in v4."},{"line_number":11,"context_line":"Since older versions of OpenStack still require py36 support, we need to add a constraint for py36."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Currently, this is blocking telemetry-dvsm-integration-centos-8s jobs on"},{"line_number":14,"context_line":"yoga [1] [2]"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[1] https://review.opendev.org/c/openstack/ceilometer/+/852878"},{"line_number":17,"context_line":"(json_schema)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"795e0688_b902c396","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":12},"updated":"2022-08-23 16:50:50.000000000","message":"its also causing failures on several other project branches\n\nhttps://opensearch.logs.openstack.org/_dashboards/app/discover/?security_tenant\u003dglobal#/?_g\u003d(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-30d,to:now))\u0026_a\u003d(columns:!(project,build_branch,build_name,message),filters:!(),index:\u002794869730-aea8-11ec-9e6a-83741af3fdcd\u0027,interval:auto,query:(language:kuery,query:\u0027%22The%20user%20requested%20(constraint)%20cliff%22\u0027),sort:!())\n\n134 hits in the last 30 days\n\nabout 40% of the failures are on voting jobs.","commit_id":"94f10e2fd9859c7adfba0d795e9abf50fd505068"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2a0cbbf09b44cf9788c7a49131645e6e226eef06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"016b0f1f_60e3a1a7","updated":"2022-08-25 14:14:48.000000000","message":"-1 for the \u003e\u003d3.6","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"036a78bdb952e6e29a9ba481f7a2110fda87dd2b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0f5a34fd_e9b8960b","updated":"2022-08-25 19:48:14.000000000","message":"TL;DR Rats this is going to be messy.\n\nThere are a number of issues with this patch many of which have been pointed out.  \n\nI\u0027m -1 on this as the patch wont actually fix the issue that ceilometer is seeing, that we introduced unwittingly by removing python3.6 spport on master.\n\nYou can trivially reproduce this issue with tempest@master:\n\n [tony@thor ~]$ cd src/opendev.org/openstack/tempest\n [tony@thor tempest]$ tox -repy36 --notest\n \nI can\u0027t actually think of a good way to fix this issue and the only un-good option I can think of is to use some form, probably unpackaged, of python-3.8 on Centos-8s explitly and only for running tempest.\n\nOr maybe we can add back the py3.6 markers to the master upper-constraints file.  Testing that can\u0027t really be done in the gate though so we end up with a nasty \"merge this and hope\" situation\n\n#sorry","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"45498bc937cc61d73d92f547bcdceb3b0a099b5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f72a9603_0531fd3e","updated":"2022-08-25 12:50:20.000000000","message":"cool zuul is happy.\n\nwe will need the same on xena.\n\nas i noted in the opensech query there are 146 failure across multipel project on master, yoga and xena  and i think victoria althoguh that has rotated out of that search window.\n\nso +1 to backporting this.\n\nit does not break nova as we dont have any voting jobs that hit this but it will affect other projects that do.\n","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"56a50c736b8da377b0a3d5dbaa3e0e96c21309fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ea4ccd8b_c1fa21e9","updated":"2022-08-26 14:30:43.000000000","message":"looks good to me, can add some comment inline as Jen suggested those.","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"9dfded88d9e17619aec5e1a4d32c81349719d1bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"3ac4e9d9_d69a46e3","in_reply_to":"0f5a34fd_e9b8960b","updated":"2022-08-26 12:58:14.000000000","message":"Add the py36 markers into upper-constraints? You mean like in [1]: \n\n```\nsomepackage\u003d\u003d\u003d4.0.0;python_version\u003d\u003d\u00273.8\u0027\nsomepackage\u003d\u003d\u003d3.9.0;python_version\u003d\u003d\u00273.6\u0027\n```\nSince the effected jobs are already broken, could we try merging, seeing if it works and reverting if it doesn\u0027t? it\u0027s unlikely that we\u0027ll break anything else.\n\n[1] https://review.opendev.org/c/openstack/requirements/+/852935","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"95be176a0df5bd4fc54a09ae23ad575ab729dc0d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8f16f599_7787044f","in_reply_to":"29ed3790_3d94382f","updated":"2022-08-29 09:31:26.000000000","message":"I\u0027m unclear as to who is doing each of these. I can propose the patch to requirements:master, but I don\u0027t have the knowledge for devstack.","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"f3567aec8a297a301e239ae0c16cd625202fde36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"29ed3790_3d94382f","in_reply_to":"32e6c2a6_4cacee80","updated":"2022-08-26 19:00:53.000000000","message":"No ammount of patching stable/yoga will alter the behavior of tempest from master.\n\nI agree we have found a real problem but this isn\u0027t the correct fix.  and due to the way the stable branches and tempest operate we can\u0027t really test this speculativly.\n\nI think we\u0027re going to need several patches\n\n1) Update the *master* branch of requirements so that we include constraints with markers with python_version 3.6 [1]\n\n2) chnages to the generic devstack jobs such that we pin to a version of tempest that supports py3.6 on OSs that only have python3.6 available.  This feels hugely risky to me as it implies that yoga on these OSs (Centos 8) will never need to be patched.\n\n[1] This is actually a really \"big\" change to do properly as upper-constraints is a generated file but I think we\u0027re happy to accept that tech debt to unblock everyone","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"56a50c736b8da377b0a3d5dbaa3e0e96c21309fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"322c27f1_66dff20c","in_reply_to":"3ac4e9d9_d69a46e3","updated":"2022-08-26 14:30:43.000000000","message":"I agree with Emma Foley solution here. Tempest master does not support py3.6 or any other old version of python and to make it work on py3.6 env (where no other higher python version is available to be used in tempest virutal env), we need to adjust few constraints in requirement.","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"56a50c736b8da377b0a3d5dbaa3e0e96c21309fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"32e6c2a6_4cacee80","in_reply_to":"3ac4e9d9_d69a46e3","updated":"2022-08-26 14:30:43.000000000","message":"I agree with Emma Foley solution here. Tempest master does not support py3.6 or any other old version of python and to make it work on py3.6 env (where no other higher python version is available to be used in tempest virutal env), we need to adjust few constraints in requirement.\n\nCannot we do these directly in stable branch constraints instead of master  requirements constraints which does not need to support py3.6?","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"d19839afd68b87d2a9d194f0f0d181a8d22b1ba5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"354a4ba4_83ae86f7","updated":"2022-08-29 10:17:34.000000000","message":"I\u0027ve restored and updated the review on master: https://review.opendev.org/c/openstack/requirements/+/854038\n","commit_id":"c2bbb86f2b133dfa83a50e42426d1c87b5839874"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"d5e9b00197425179504944eb7bbc644e2d4d282f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"66ffb861_65530a4c","updated":"2022-08-26 18:24:53.000000000","message":"recheck\n\ncross-swift-py38 timed out","commit_id":"c2bbb86f2b133dfa83a50e42426d1c87b5839874"}],"global-requirements.txt":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2a0cbbf09b44cf9788c7a49131645e6e226eef06","unresolved":true,"context_lines":[{"line_number":23,"context_line":"ceilometermiddleware  # Apache-2.0"},{"line_number":24,"context_line":"cachetools  # MIT License"},{"line_number":25,"context_line":"cffi  # MIT"},{"line_number":26,"context_line":"cliff!\u003d2.9.0,!\u003d2.17.0;python_version\u003e\u003d\u00273.6\u0027  # Apache-2.0"},{"line_number":27,"context_line":"cliff\u003c\u003d3.10.1;python_version\u003d\u003d\u00273.6\u0027  # Apache-2.0"},{"line_number":28,"context_line":"cmd2!\u003d0.8.3  # MIT"},{"line_number":29,"context_line":"confluent-kafka!\u003d1.4.0  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":7,"id":"d4a17a82_51b6567c","line":26,"updated":"2022-08-25 14:14:48.000000000","message":"This should be either \"\u003e\u003d\u00273.8\u0027\" or at least \"\u003e3.6\".","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"dbc4aef346111e018965bb2b1a23bcc9df3758f2","unresolved":true,"context_lines":[{"line_number":23,"context_line":"ceilometermiddleware  # Apache-2.0"},{"line_number":24,"context_line":"cachetools  # MIT License"},{"line_number":25,"context_line":"cffi  # MIT"},{"line_number":26,"context_line":"cliff!\u003d2.9.0,!\u003d2.17.0;python_version\u003e\u003d\u00273.6\u0027  # Apache-2.0"},{"line_number":27,"context_line":"cliff\u003c\u003d3.10.1;python_version\u003d\u003d\u00273.6\u0027  # Apache-2.0"},{"line_number":28,"context_line":"cmd2!\u003d0.8.3  # MIT"},{"line_number":29,"context_line":"confluent-kafka!\u003d1.4.0  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":7,"id":"e90dd027_15222d1a","line":26,"in_reply_to":"d4a17a82_51b6567c","updated":"2022-08-25 14:59:42.000000000","message":"I believe that these requirements also hold in 3.6, so it should be \u003e\u003d3.6, unless I duplicate the \u0027!\u003d2.9.0,!\u003d2.17.0\u0027 on line 27","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2a0cbbf09b44cf9788c7a49131645e6e226eef06","unresolved":true,"context_lines":[{"line_number":24,"context_line":"cachetools  # MIT License"},{"line_number":25,"context_line":"cffi  # MIT"},{"line_number":26,"context_line":"cliff!\u003d2.9.0,!\u003d2.17.0;python_version\u003e\u003d\u00273.6\u0027  # Apache-2.0"},{"line_number":27,"context_line":"cliff\u003c\u003d3.10.1;python_version\u003d\u003d\u00273.6\u0027  # Apache-2.0"},{"line_number":28,"context_line":"cmd2!\u003d0.8.3  # MIT"},{"line_number":29,"context_line":"confluent-kafka!\u003d1.4.0  # Apache-2.0"},{"line_number":30,"context_line":"confspirator  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":7,"id":"b167e285_409dcf6a","line":27,"updated":"2022-08-25 14:14:48.000000000","message":"Maybe add a comment about why this is needed? Also maybe better cap \"\u003c4.0.0\"? Not sure whether another stable release for cliff might happen, but it that would be the case, I don\u0027t think we would want to block it.","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2a0cbbf09b44cf9788c7a49131645e6e226eef06","unresolved":true,"context_lines":[{"line_number":93,"context_line":"jsonpatch!\u003d1.20  # BSD"},{"line_number":94,"context_line":"jsonpath-rw  # Apache-2.0"},{"line_number":95,"context_line":"jsonpath-rw-ext  # Apache-2.0"},{"line_number":96,"context_line":"jsonschema\u003c4  # MIT"},{"line_number":97,"context_line":"kazoo  # Apache-2.0"},{"line_number":98,"context_line":"keystoneauth1  # Apache-2.0"},{"line_number":99,"context_line":"keystonemiddleware  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":7,"id":"fa55755b_7ab12ca6","line":96,"updated":"2022-08-25 14:14:48.000000000","message":"Looking at the commit message, this would only be needed for python\u003d\u003d3.6. Otoh we seem to have this capped already by some transitive requirement, so one could argue whether this cap is needed at all. But I don\u0027t object to making it explicit like this, just amend the commit message if you do.","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"89bd25a56d75e1f0e51aaa83266a697a82e38483","unresolved":true,"context_lines":[{"line_number":93,"context_line":"jsonpatch!\u003d1.20  # BSD"},{"line_number":94,"context_line":"jsonpath-rw  # Apache-2.0"},{"line_number":95,"context_line":"jsonpath-rw-ext  # Apache-2.0"},{"line_number":96,"context_line":"jsonschema\u003c4  # MIT"},{"line_number":97,"context_line":"kazoo  # Apache-2.0"},{"line_number":98,"context_line":"keystoneauth1  # Apache-2.0"},{"line_number":99,"context_line":"keystonemiddleware  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":7,"id":"c0569aa6_8c374c5e","line":96,"in_reply_to":"8a717c1d_20500600","updated":"2022-08-25 15:01:17.000000000","message":"Please see the builds from PS6: https://zuul.opendev.org/t/openstack/buildset/da0434bca14d4c21934ca13e68cd8598","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"dbc4aef346111e018965bb2b1a23bcc9df3758f2","unresolved":true,"context_lines":[{"line_number":93,"context_line":"jsonpatch!\u003d1.20  # BSD"},{"line_number":94,"context_line":"jsonpath-rw  # Apache-2.0"},{"line_number":95,"context_line":"jsonpath-rw-ext  # Apache-2.0"},{"line_number":96,"context_line":"jsonschema\u003c4  # MIT"},{"line_number":97,"context_line":"kazoo  # Apache-2.0"},{"line_number":98,"context_line":"keystoneauth1  # Apache-2.0"},{"line_number":99,"context_line":"keystonemiddleware  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":7,"id":"8a717c1d_20500600","line":96,"in_reply_to":"fa55755b_7ab12ca6","updated":"2022-08-25 14:59:42.000000000","message":"Warlock still requires jsonschema\u003c4 on 38","commit_id":"3149940054a8fa2d765b053b62fe3c8f0b52c117"}]}