)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"92690a87beb219eae23b86b1aff33e8d009516ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a4a2a375_1d1f0e28","updated":"2022-11-01 05:23:51.000000000","message":"Regarding the need for pytest-html, it is just for convenience.\nIt is not picked up in our zuul job results, so I am not sure we need to keep pytest-html.\nIf it is only for local environments, I don\u0027t think we need to keep it.","commit_id":"3bd51118e6a1e268cd5ea234861f0c81ad89c8e4"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"48fa3d7c1e836f28e0d2079fcae3cc6672d75350","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a0912db7_864a106d","updated":"2022-11-01 04:40:42.000000000","message":"Thanks for the patch, please find some comments inline.\n\n\u0027requirements-chek\u0027 job is failing as \u0027py\u0027 is not added in openstack/requirements.\n\n[1] https://zuul.opendev.org/t/openstack/build/237cc61263f14063ba64a2a497e02433/log/job-output.txt#602","commit_id":"3bd51118e6a1e268cd5ea234861f0c81ad89c8e4"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"5abc6ea7bd68690758672a6a10c34d6a69042851","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a66fde34_323b250e","updated":"2022-11-01 05:22:18.000000000","message":"py","commit_id":"3bd51118e6a1e268cd5ea234861f0c81ad89c8e4"},{"author":{"_account_id":32927,"name":"mitya-eremeev-2","display_name":"Mitya Eremeev","email":"mitossvyaz@mail.ru","username":"mitya-eremeev-2"},"change_message_id":"8ef673f283046ee88e70ec3639727340fb3b87ac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"88ef7461_65dc7b14","in_reply_to":"a4a2a375_1d1f0e28","updated":"2022-11-02 11:39:20.000000000","message":"So I may remove pytest-html altogether with py...","commit_id":"3bd51118e6a1e268cd5ea234861f0c81ad89c8e4"}],"test-requirements.txt":[{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"48fa3d7c1e836f28e0d2079fcae3cc6672d75350","unresolved":true,"context_lines":[{"line_number":9,"context_line":"coverage!\u003d4.4,\u003e\u003d4.0 # Apache-2.0"},{"line_number":10,"context_line":"freezegun\u003e\u003d0.3.15 # Apache-2.0"},{"line_number":11,"context_line":"nodeenv\u003e\u003d0.9.4 # BSD"},{"line_number":12,"context_line":"pytest\u003e\u003d5.3.5 # MIT"},{"line_number":13,"context_line":"pytest-django\u003e\u003d3.8.0 # BSD (3 clause)"},{"line_number":14,"context_line":"py\u003e\u003d1.11.0 # MIT"},{"line_number":15,"context_line":"pytest-html\u003e\u003d2.0.1 #MPL-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"ba688480_4f92f7a1","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":6},"updated":"2022-11-01 04:40:42.000000000","message":"you should update pytest version here?","commit_id":"3bd51118e6a1e268cd5ea234861f0c81ad89c8e4"},{"author":{"_account_id":32927,"name":"mitya-eremeev-2","display_name":"Mitya Eremeev","email":"mitossvyaz@mail.ru","username":"mitya-eremeev-2"},"change_message_id":"8ef673f283046ee88e70ec3639727340fb3b87ac","unresolved":true,"context_lines":[{"line_number":9,"context_line":"coverage!\u003d4.4,\u003e\u003d4.0 # Apache-2.0"},{"line_number":10,"context_line":"freezegun\u003e\u003d0.3.15 # Apache-2.0"},{"line_number":11,"context_line":"nodeenv\u003e\u003d0.9.4 # BSD"},{"line_number":12,"context_line":"pytest\u003e\u003d5.3.5 # MIT"},{"line_number":13,"context_line":"pytest-django\u003e\u003d3.8.0 # BSD (3 clause)"},{"line_number":14,"context_line":"py\u003e\u003d1.11.0 # MIT"},{"line_number":15,"context_line":"pytest-html\u003e\u003d2.0.1 #MPL-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"7da43cbf_ae4e8b1e","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":6},"in_reply_to":"ba688480_4f92f7a1","updated":"2022-11-02 11:39:20.000000000","message":"No, it\u0027s already updated in upper-constraints:\n863083: [CVE-2022-42969] Do not use package py | https://review.opendev.org/c/openstack/requirements/+/863083","commit_id":"3bd51118e6a1e268cd5ea234861f0c81ad89c8e4"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"48fa3d7c1e836f28e0d2079fcae3cc6672d75350","unresolved":true,"context_lines":[{"line_number":11,"context_line":"nodeenv\u003e\u003d0.9.4 # BSD"},{"line_number":12,"context_line":"pytest\u003e\u003d5.3.5 # MIT"},{"line_number":13,"context_line":"pytest-django\u003e\u003d3.8.0 # BSD (3 clause)"},{"line_number":14,"context_line":"py\u003e\u003d1.11.0 # MIT"},{"line_number":15,"context_line":"pytest-html\u003e\u003d2.0.1 #MPL-2.0"},{"line_number":16,"context_line":"python-memcached\u003e\u003d1.59 # PSF"},{"line_number":17,"context_line":"selenium\u003e\u003d2.50.1 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"2fa3950c_fd088d94","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":2},"updated":"2022-11-01 04:40:42.000000000","message":"if we bump pytest version to 7.2.0 then still \u0027py\u0027 will be used by pytest-html?","commit_id":"3bd51118e6a1e268cd5ea234861f0c81ad89c8e4"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"5abc6ea7bd68690758672a6a10c34d6a69042851","unresolved":true,"context_lines":[{"line_number":11,"context_line":"nodeenv\u003e\u003d0.9.4 # BSD"},{"line_number":12,"context_line":"pytest\u003e\u003d5.3.5 # MIT"},{"line_number":13,"context_line":"pytest-django\u003e\u003d3.8.0 # BSD (3 clause)"},{"line_number":14,"context_line":"py\u003e\u003d1.11.0 # MIT"},{"line_number":15,"context_line":"pytest-html\u003e\u003d2.0.1 #MPL-2.0"},{"line_number":16,"context_line":"python-memcached\u003e\u003d1.59 # PSF"},{"line_number":17,"context_line":"selenium\u003e\u003d2.50.1 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"c9e9f482_d0197999","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":2},"in_reply_to":"2fa3950c_fd088d94","updated":"2022-11-01 05:22:18.000000000","message":"Even in the pytest-html master, py.xml is still used [1].\n\nHowever, pytest-html explicitly requires \"py\", so I cannot understand why we need to have \"py\" in requirements.txt directly.\n\n[1] https://github.com/pytest-dev/pytest-html/blob/a867776a403d007dfca7e22650fd8241bfabe0d1/src/pytest_html/html_report.py#L11-L12","commit_id":"3bd51118e6a1e268cd5ea234861f0c81ad89c8e4"},{"author":{"_account_id":32927,"name":"mitya-eremeev-2","display_name":"Mitya Eremeev","email":"mitossvyaz@mail.ru","username":"mitya-eremeev-2"},"change_message_id":"8ef673f283046ee88e70ec3639727340fb3b87ac","unresolved":true,"context_lines":[{"line_number":11,"context_line":"nodeenv\u003e\u003d0.9.4 # BSD"},{"line_number":12,"context_line":"pytest\u003e\u003d5.3.5 # MIT"},{"line_number":13,"context_line":"pytest-django\u003e\u003d3.8.0 # BSD (3 clause)"},{"line_number":14,"context_line":"py\u003e\u003d1.11.0 # MIT"},{"line_number":15,"context_line":"pytest-html\u003e\u003d2.0.1 #MPL-2.0"},{"line_number":16,"context_line":"python-memcached\u003e\u003d1.59 # PSF"},{"line_number":17,"context_line":"selenium\u003e\u003d2.50.1 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"7c819861_da0e3258","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":2},"in_reply_to":"c9e9f482_d0197999","updated":"2022-11-02 11:39:20.000000000","message":"Before pytest 7.2.0 package \"py\" was required by pytest and was installed.\nAfter pytest upgrade \"py\"is not installed, but pytest-html requires \"py\".\nUnfortunately py is not installed if it\u0027s not explicitly specified.\nPlease see log of failed CI job:\nFile \"/home/zuul/src/opendev.org/openstack/horizon/.tox/py310/lib/python3.10/site-packages/pytest_html/plugin.py\", line 22, in \u003cmodule\u003e\n    from py.xml import html\nModuleNotFoundError: No module named \u0027py.xml\u0027; \u0027py\u0027 is not a package\nhttps://zuul.opendev.org/t/openstack/build/8cb65edb8ecf4241a6fe614c2dd17358\nhttps://review.opendev.org/c/openstack/requirements/+/863083\n\nMaintainers advice to specify explicitly \"py\":\nhttps://github.com/pytest-dev/pytest-html/pull/555","commit_id":"3bd51118e6a1e268cd5ea234861f0c81ad89c8e4"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"f1f8a343b15f7da4195e9600c513c994d5d13186","unresolved":true,"context_lines":[{"line_number":9,"context_line":"coverage!\u003d4.4,\u003e\u003d4.0 # Apache-2.0"},{"line_number":10,"context_line":"freezegun\u003e\u003d0.3.15 # Apache-2.0"},{"line_number":11,"context_line":"nodeenv\u003e\u003d0.9.4 # BSD"},{"line_number":12,"context_line":"pytest-html\u003e\u003d2.0.1 #MPL-2.0"},{"line_number":13,"context_line":"pytest\u003e\u003d5.3.5 # MIT"},{"line_number":14,"context_line":"pytest-django\u003e\u003d3.8.0 # BSD (3 clause)"},{"line_number":15,"context_line":"python-memcached\u003e\u003d1.59 # PSF"}],"source_content_type":"text/plain","patch_set":5,"id":"a5e6d216_addb1cd6","line":12,"range":{"start_line":12,"start_character":1,"end_line":12,"end_character":27},"updated":"2022-11-07 15:50:04.000000000","message":"I didn\u0027t get what you did in this patch. You just moved the pytest-html package\nfrom L.14 to L.12. why?","commit_id":"4d003f8d92f63cc0fb2d0b367cd7056e96447b06"},{"author":{"_account_id":32927,"name":"mitya-eremeev-2","display_name":"Mitya Eremeev","email":"mitossvyaz@mail.ru","username":"mitya-eremeev-2"},"change_message_id":"1909f6694bbb752d084d62e4403307842a69fafe","unresolved":true,"context_lines":[{"line_number":9,"context_line":"coverage!\u003d4.4,\u003e\u003d4.0 # Apache-2.0"},{"line_number":10,"context_line":"freezegun\u003e\u003d0.3.15 # Apache-2.0"},{"line_number":11,"context_line":"nodeenv\u003e\u003d0.9.4 # BSD"},{"line_number":12,"context_line":"pytest-html\u003e\u003d2.0.1 #MPL-2.0"},{"line_number":13,"context_line":"pytest\u003e\u003d5.3.5 # MIT"},{"line_number":14,"context_line":"pytest-django\u003e\u003d3.8.0 # BSD (3 clause)"},{"line_number":15,"context_line":"python-memcached\u003e\u003d1.59 # PSF"}],"source_content_type":"text/plain","patch_set":5,"id":"8bba27b6_f8d7a320","line":12,"range":{"start_line":12,"start_character":1,"end_line":12,"end_character":27},"in_reply_to":"a5e6d216_addb1cd6","updated":"2022-11-07 16:18:55.000000000","message":"Already fixed.","commit_id":"4d003f8d92f63cc0fb2d0b367cd7056e96447b06"}]}