)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"bf2cf5811ddcd54354a1a41c4ad417af535bf277","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f80acec1_b1c918f5","updated":"2022-03-24 21:27:57.000000000","message":"Oh, also once it\u0027s removed from the requirements list it probably will need to be in test-requirements.txt so we can run the unittests.","commit_id":"019896b7c9736a7d7aa284f6bf2f268e605b1b5f"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"eec7d12457bd5ddf1e02be6c810c457430df1e73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cd8941a2_7d6ecf38","updated":"2022-03-24 21:26:55.000000000","message":"Overall LGTM, besides the one nit inline on the release note I think the only other thing is I don\u0027t think we need stestr in the requirements list anymore without the ostestr command. I\u0027m pretty sure the only thing using stestr was the ostestr command.","commit_id":"019896b7c9736a7d7aa284f6bf2f268e605b1b5f"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"42aa0fdf923ce4e14df71a95af922e8a74aa9219","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"78c0e986_96a4f1ea","in_reply_to":"cd8941a2_7d6ecf38","updated":"2022-03-24 21:46:30.000000000","message":"good point, I will move that to test-requirements file as we use that in tox unit test env.","commit_id":"019896b7c9736a7d7aa284f6bf2f268e605b1b5f"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"723449129d46787a57b8b30b63d4284a5eea61f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f4552a65_650855c0","updated":"2022-03-24 22:00:02.000000000","message":"LGTM","commit_id":"6f10535042ac3220a5dd9f21d25dd20bb14ba523"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"b77afa34a3454d843f8e1b3e31bf272e853098a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dc5778d1_a3082d38","updated":"2022-03-24 22:16:16.000000000","message":"Thanks! This is a good cleanup.","commit_id":"6f10535042ac3220a5dd9f21d25dd20bb14ba523"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"7c7c5c1d3e151ee67ee91e5f2ca9241e6e6bbe1e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8a4f32fc_553fc3d3","updated":"2022-03-25 09:39:46.000000000","message":"looks good, thank you","commit_id":"6f10535042ac3220a5dd9f21d25dd20bb14ba523"}],"releasenotes/notes/remove-ostestr-command-2cbe2a3254ba4adb.yaml":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"eec7d12457bd5ddf1e02be6c810c457430df1e73","unresolved":true,"context_lines":[{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The deprecated ``ostestr`` command is now removed. This has been"},{"line_number":5,"context_line":"    deprecated since more than a year and most of the users have been"},{"line_number":6,"context_line":"    switched to ``stestr`` command. For more information about stestr,"},{"line_number":7,"context_line":"    see https://stestr.readthedocs.io/en/latest/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6e618c55_8026f287","line":5,"updated":"2022-03-24 21:26:55.000000000","message":"I\u0027d probably replace \"since\" here with \"for\", just reads a bit better to me that way.","commit_id":"019896b7c9736a7d7aa284f6bf2f268e605b1b5f"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"42aa0fdf923ce4e14df71a95af922e8a74aa9219","unresolved":false,"context_lines":[{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The deprecated ``ostestr`` command is now removed. This has been"},{"line_number":5,"context_line":"    deprecated since more than a year and most of the users have been"},{"line_number":6,"context_line":"    switched to ``stestr`` command. For more information about stestr,"},{"line_number":7,"context_line":"    see https://stestr.readthedocs.io/en/latest/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2b280604_0604d43f","line":5,"in_reply_to":"6e618c55_8026f287","updated":"2022-03-24 21:46:30.000000000","message":"Done","commit_id":"019896b7c9736a7d7aa284f6bf2f268e605b1b5f"}]}
