)]}'
{"tox.ini":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"1eb426c7c2384cc378435103b5766f52b8c533b1","unresolved":true,"context_lines":[{"line_number":8,"context_line":"install_command \u003d pip install {opts} {packages}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"[testenv:bashate]"},{"line_number":11,"context_line":"deps \u003d bashate\u003e\u003d2.1.0,\u003c2.2.0"},{"line_number":12,"context_line":"allowlist_externals \u003d bash"},{"line_number":13,"context_line":"commands \u003d bash -c \"find {toxinidir}          \\"},{"line_number":14,"context_line":"         -not \\( -type d -name .?\\* -prune \\) \\"}],"source_content_type":"text/x-properties","patch_set":2,"id":"ee9d7725_f569281a","line":11,"range":{"start_line":11,"start_character":7,"end_line":11,"end_character":28},"updated":"2024-01-29 02:28:04.000000000","message":"do we need to cap here? I mean let latest version installed as it is currently. \n\nIf any specific version is used then we can bump that like you did in devstack otherwise we should not cap it.","commit_id":"b7e5a2dc1d0e2072484b135b05448ae6e071f319"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fa7d57a9ec266603e28cf16114e25d50513f9dee","unresolved":true,"context_lines":[{"line_number":8,"context_line":"install_command \u003d pip install {opts} {packages}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"[testenv:bashate]"},{"line_number":11,"context_line":"deps \u003d bashate\u003e\u003d2.1.0,\u003c2.2.0"},{"line_number":12,"context_line":"allowlist_externals \u003d bash"},{"line_number":13,"context_line":"commands \u003d bash -c \"find {toxinidir}          \\"},{"line_number":14,"context_line":"         -not \\( -type d -name .?\\* -prune \\) \\"}],"source_content_type":"text/x-properties","patch_set":2,"id":"b4d4e66e_8eb63d63","line":11,"range":{"start_line":11,"start_character":7,"end_line":11,"end_character":28},"in_reply_to":"14e5d74e_b777c35d","updated":"2024-01-29 02:36:48.000000000","message":"I see, I missed the depends-on. I agree if we are doing it in devstack we should keep it same but I am rethinking if in devstack also we can just install the latest. commenting there.","commit_id":"b7e5a2dc1d0e2072484b135b05448ae6e071f319"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b96194332bdb2c8257af1fb02b5ebf536a885e7b","unresolved":true,"context_lines":[{"line_number":8,"context_line":"install_command \u003d pip install {opts} {packages}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"[testenv:bashate]"},{"line_number":11,"context_line":"deps \u003d bashate\u003e\u003d2.1.0,\u003c2.2.0"},{"line_number":12,"context_line":"allowlist_externals \u003d bash"},{"line_number":13,"context_line":"commands \u003d bash -c \"find {toxinidir}          \\"},{"line_number":14,"context_line":"         -not \\( -type d -name .?\\* -prune \\) \\"}],"source_content_type":"text/x-properties","patch_set":2,"id":"14e5d74e_b777c35d","line":11,"range":{"start_line":11,"start_character":7,"end_line":11,"end_character":28},"in_reply_to":"ee9d7725_f569281a","updated":"2024-01-29 02:34:16.000000000","message":"IMO if we cap bashate in devstack then we should follow it in all plugins to avoid inconsistencies between devstack and plugins. This is why I made this change dependent on the devstack change here.","commit_id":"b7e5a2dc1d0e2072484b135b05448ae6e071f319"}]}