)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"bdb659d9bf31111ebf760c0019f67925dde507c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2ca9f261_9ad0c766","updated":"2023-08-21 07:39:34.000000000","message":"hacking is pinned by -c; -r just adds extra installations","commit_id":"39e05b67bbb2166b446d6cec12c74db14e688966"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"a8eb2fd3400c0449ab8fdbe63806e59456ea0d09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"833549cc_c33546dc","updated":"2023-08-31 19:25:48.000000000","message":"thanks for review Radoslaw","commit_id":"39e05b67bbb2166b446d6cec12c74db14e688966"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"a8eb2fd3400c0449ab8fdbe63806e59456ea0d09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"578c4188_589e79a2","in_reply_to":"2ca9f261_9ad0c766","updated":"2023-08-31 19:25:48.000000000","message":"1. Hacking is test requirement which is not present in upper constraints https://github.com/openstack/requirements/blob/master/upper-constraints.txt\n2. requirements file which is used with -r option allow to set caps/add excludes https://github.com/openstack/masakari-monitors/blob/master/test-requirements.txt#L7","commit_id":"39e05b67bbb2166b446d6cec12c74db14e688966"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"bbb5005b1bf049770fc651e58a8bf6fb9990d276","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fca7ec6d_1b28d144","in_reply_to":"578c4188_589e79a2","updated":"2023-09-01 05:29:09.000000000","message":"Thanks for the explanations. I see the issue now.\n\nI made a mistake when porting this to tox.ini, I should have preserved the constraint. You should change -r to -c (though I don\u0027t remember if two -c are allowed; if not, you can simply copy the constraint to tox.ini directly).","commit_id":"39e05b67bbb2166b446d6cec12c74db14e688966"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"8f7eea6a9e4ec870dd28e1096f4f3696eba1d052","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"de44db72_c9e3ef6e","in_reply_to":"8007f2a3_2420fad2","updated":"2023-09-22 11:26:37.000000000","message":"Yes, but we don\u0027t need all these requirements in the pep8 job. Your example is comparing apples to oranges - in fact, in L15 here we also use test-requirements.txt in the same way.","commit_id":"39e05b67bbb2166b446d6cec12c74db14e688966"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"040241b47810f6d1aa7800066b801b68e8ca94a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9105664c_211eec89","in_reply_to":"de44db72_c9e3ef6e","updated":"2023-09-26 08:41:35.000000000","message":"I took the liberty to edit this patch so that it does not rot. I will merge once the jobs succeed.","commit_id":"39e05b67bbb2166b446d6cec12c74db14e688966"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"c15866a337e89a87d91c1f1f7d4daba1b396ed06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8007f2a3_2420fad2","in_reply_to":"fca7ec6d_1b28d144","updated":"2023-09-22 11:22:36.000000000","message":"test-requirements.txt defines requirements not constraints, requirements should be installed with -r flag, the constraints file is general for all components and is TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master\n\nThe same is done in other projects, for example neutron https://github.com/openstack/neutron/blob/master/tox.ini#L28","commit_id":"39e05b67bbb2166b446d6cec12c74db14e688966"},{"author":{"_account_id":35819,"name":"zhuyawei","display_name":"zhuyawei","email":"zhuyawei@inspur.com","username":"zhuyawei"},"change_message_id":"7d64fcec372ee585322797860a173ddca2dab643","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"413ca55c_f230291d","updated":"2023-09-26 08:45:06.000000000","message":"Look good","commit_id":"58cbb346effc0aa5484815823939241fb5658905"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"e0651457a665e9858130ae61402a1b7a7acc3193","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3cca7e68_fbd6861e","updated":"2023-09-26 09:50:24.000000000","message":"worked like a charm, merging","commit_id":"58cbb346effc0aa5484815823939241fb5658905"}]}