)]}'
{".zuul.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"8e3d89e736f884fc924087386ad72342cdf96092","unresolved":true,"context_lines":[{"line_number":41,"context_line":"      # in stable/ussuri."},{"line_number":42,"context_line":"      # details in https://storyboard.openstack.org/#!/story/2010057"},{"line_number":43,"context_line":"      - name: openstack/openstacksdk"},{"line_number":44,"context_line":"        override-branch: stable/ussuri"},{"line_number":45,"context_line":"      - name: openstack/os-client-config"},{"line_number":46,"context_line":"        override-branch: stable/ussuri"},{"line_number":47,"context_line":"    timeout: 9000"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"c7c6ccb8_b9f9a3c5","line":44,"updated":"2022-05-31 15:16:09.000000000","message":"If I\u0027m not mistaken override-branch can be omitted as by default stable/ussuri is checked out.","commit_id":"566a7fb9fd935ce9af657b436d1faf70ec0cc012"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4f065cd21e048aabea73eddb14ef8a01163a336f","unresolved":true,"context_lines":[{"line_number":41,"context_line":"      # in stable/ussuri."},{"line_number":42,"context_line":"      # details in https://storyboard.openstack.org/#!/story/2010057"},{"line_number":43,"context_line":"      - name: openstack/openstacksdk"},{"line_number":44,"context_line":"        override-branch: stable/ussuri"},{"line_number":45,"context_line":"      - name: openstack/os-client-config"},{"line_number":46,"context_line":"        override-branch: stable/ussuri"},{"line_number":47,"context_line":"    timeout: 9000"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ca708e9e_6fecee21","line":44,"in_reply_to":"c7c6ccb8_b9f9a3c5","updated":"2022-05-31 16:57:49.000000000","message":"ah yes, we are already in ussuri branch. done","commit_id":"566a7fb9fd935ce9af657b436d1faf70ec0cc012"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"67739a03010d34c9b475d96447b471fda4fbe8c1","unresolved":false,"context_lines":[{"line_number":41,"context_line":"      # in stable/ussuri."},{"line_number":42,"context_line":"      # details in https://storyboard.openstack.org/#!/story/2010057"},{"line_number":43,"context_line":"      - name: openstack/openstacksdk"},{"line_number":44,"context_line":"        override-branch: stable/ussuri"},{"line_number":45,"context_line":"      - name: openstack/os-client-config"},{"line_number":46,"context_line":"        override-branch: stable/ussuri"},{"line_number":47,"context_line":"    timeout: 9000"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f50f3901_6230ed8d","line":44,"in_reply_to":"ca708e9e_6fecee21","updated":"2022-06-01 11:57:08.000000000","message":"Done","commit_id":"566a7fb9fd935ce9af657b436d1faf70ec0cc012"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"8e3d89e736f884fc924087386ad72342cdf96092","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b50c7e41_cba1401e","updated":"2022-05-31 15:16:09.000000000","message":"lower constraints job is still present :S I\u0027ve backported the l-c dropping patch [1] though i guess that needs to be squashed into this to unblock the gate.\n\n[1] https://review.opendev.org/q/Ib0ef1fadf4fe44f1d6af2f7d542b8af919652501","commit_id":"566a7fb9fd935ce9af657b436d1faf70ec0cc012"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4f065cd21e048aabea73eddb14ef8a01163a336f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"611ca987_47fb871d","in_reply_to":"b50c7e41_cba1401e","updated":"2022-05-31 16:57:49.000000000","message":"+1, sure","commit_id":"566a7fb9fd935ce9af657b436d1faf70ec0cc012"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e30392bd3b23f88ec9ac6a88b225572a60b253b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f7722b28_baddb69d","updated":"2022-06-01 11:54:07.000000000","message":"We need this soon as stable/ussuri is blocked by this (100% failure rate in nova [2]) since master upper-constraints bumped coverage version in [1]. \n\n[1] https://review.opendev.org/c/openstack/requirements/+/843722/3/upper-constraints.txt#273\n[2] https://zuul.opendev.org/t/openstack/builds?job_name\u003dopenstacksdk-functional-devstack\u0026project\u003dopenstack%2Fnova\u0026branch\u003dstable%2Fussuri\u0026skip\u003d0","commit_id":"4cc272f276405dfc2df22496a0abb8c8757651ea"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5f6ba2adf6c2517904301fdacd863ef828278b16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e4b9fcf1_aa5ded15","updated":"2022-06-01 15:23:24.000000000","message":"honestly saying this is not actually giving any benefit then making it non voting. If we really want to test master sdk on EM we should have two things\n\n1. test like what tempest does, create a venv with stable branch constraints and test master sdk.\n\n2. sdk core team is able to support the EM branch support. for example, Tempest team does not have much bandwidth to support all EM with tempest master so we pin it.\n\nIf pinning sdk in EM like we do for Tempest is ok then we can proceed with the this patch otherwise I will say let\u0027s make the job non voting on stable/ussuri and consider the above approach.  ","commit_id":"7f79f6a02461e1d4b03b88d750d2cf45c62f5f15"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"84b89690be58b2dd593254cb42df0363dcd40491","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"90ef7d36_619a25ef","updated":"2023-04-21 16:27:58.000000000","message":"recheck","commit_id":"7f79f6a02461e1d4b03b88d750d2cf45c62f5f15"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"806f26b9e2a8f8a78aa69cb14b3e7b15699a74df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b7ffb299_61c44029","updated":"2023-04-25 10:37:21.000000000","message":"Assuming CI passes","commit_id":"bef964cab2c44aec84daef83cb6296e9751db0e9"}],"tox.ini":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"8a7000ea0397085e3a89fc4a663b941549f0a637","unresolved":true,"context_lines":[{"line_number":126,"context_line":""},{"line_number":127,"context_line":"[testenv:py35]"},{"line_number":128,"context_line":"basepython \u003d python3.5"},{"line_number":129,"context_line":"deps \u003d {[testenv:lower-constraints]deps}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"d782daf3_020e279e","line":129,"updated":"2022-06-01 03:09:51.000000000","message":"Looks like this needs to be changed but I\u0027m not sure what it should be. Maybe(?):\n\n deps \u003d\n   -r{toxinidir}/test-requirements.txt\n   -r{toxinidir}/requirements.txt","commit_id":"4cc272f276405dfc2df22496a0abb8c8757651ea"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a8cb1df33931b5f273ea28856a4e04cdeb8dedb5","unresolved":true,"context_lines":[{"line_number":126,"context_line":""},{"line_number":127,"context_line":"[testenv:py35]"},{"line_number":128,"context_line":"basepython \u003d python3.5"},{"line_number":129,"context_line":"deps \u003d {[testenv:lower-constraints]deps}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"df185335_f20198f3","line":129,"in_reply_to":"691aecd8_a98a96ab","updated":"2022-06-01 12:00:37.000000000","message":"the upper constratis are ment to work across all support python version so while lower shoudl also work aroucss all of them there was no reall reason to use lower constraitn here.\n\nits not valid to asume that just becasue you have python interperter form the lower bound that the rest off you requiremnt will be the lower bound.\n\nyou might use python form the distro and isntall all your deps form pypi\n\nif you are using the disto package for the sdk then it will ignore upper and lower constraints anyway so i see no reaon to not just use\n{[testenv]deps} here","commit_id":"4cc272f276405dfc2df22496a0abb8c8757651ea"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"67739a03010d34c9b475d96447b471fda4fbe8c1","unresolved":true,"context_lines":[{"line_number":126,"context_line":""},{"line_number":127,"context_line":"[testenv:py35]"},{"line_number":128,"context_line":"basepython \u003d python3.5"},{"line_number":129,"context_line":"deps \u003d {[testenv:lower-constraints]deps}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"691aecd8_a98a96ab","line":129,"in_reply_to":"d782daf3_020e279e","updated":"2022-06-01 11:57:08.000000000","message":"it should just be \ndeps \u003d\n    {[testenv]deps}\n\nhttps://review.opendev.org/c/openstack/openstacksdk/+/843978/2..3/tox.ini#78","commit_id":"4cc272f276405dfc2df22496a0abb8c8757651ea"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"1ba6757df1757030cca93be88f5f485e003a974c","unresolved":true,"context_lines":[{"line_number":126,"context_line":""},{"line_number":127,"context_line":"[testenv:py35]"},{"line_number":128,"context_line":"basepython \u003d python3.5"},{"line_number":129,"context_line":"deps \u003d {[testenv:lower-constraints]deps}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"a2798777_c1c260c5","line":129,"in_reply_to":"df185335_f20198f3","updated":"2022-06-01 12:14:12.000000000","message":"yes, sorry i did not notice that l-c is used in a non-l-c tox target :S\n@Sean: yes, we can use simply [testenv]deps (or omit it as it is the same), thanks for the comment, i\u0027ve updated the patch accordingly.","commit_id":"4cc272f276405dfc2df22496a0abb8c8757651ea"}]}