)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9546ac11794b2d6a04dc1809fbe7082aa37d730b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a41642c9_933a54e6","updated":"2022-07-13 09:46:25.000000000","message":"see my comment + question inline.","commit_id":"af68a51bcfef8ea4ce9ce0bd92b640172e0aaeab"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"07976aa8fd6cb9332e30b057f4b54051589f0a39","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"baa7c00e_481ba12c","updated":"2022-07-14 11:02:48.000000000","message":"Adding w-1 for now only to wait for Goutham\u0027s take on holding this for a couple more of fixes or not :)","commit_id":"76024898c9560ea0d813fb27e71d9ac5a1f9d996"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"81be2ada055c4d85464f031382461a26d36025c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a7242bca_09a1181a","updated":"2022-07-14 06:06:10.000000000","message":"Thanks, LGTM!","commit_id":"76024898c9560ea0d813fb27e71d9ac5a1f9d996"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c27722d21a804068a1e4b53ac7f22be66be92bce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8fe215d7_9fe20e0e","in_reply_to":"baa7c00e_481ba12c","updated":"2022-07-15 12:09:40.000000000","message":"Thanks for checking Carlos! I missed something else in the past reviews:\n\n  d62b5dc Use _parse_resp method from tempest\n\nThis commit needs a newer version of tempest than we\u0027re currently requiring; we should update the requirements.txt or revert the change and spin up a different release when we\u0027re ready to... I\u0027m exploring the update [2].. \n\n[1] https://github.com/openstack/tempest/commit/c0a15ba7a06a77c694a64f75a9c7829c4f603a68\n[2] https://review.opendev.org/c/openstack/manila-tempest-plugin/+/849995","commit_id":"76024898c9560ea0d813fb27e71d9ac5a1f9d996"},{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"8d5470cb6a105785d03234496262245e7d405d51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"88c3737c_ef413fde","updated":"2022-07-18 12:39:26.000000000","message":"LGTM","commit_id":"972e9c56795710b6e4d65f5dc1f32c411f451141"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"13281e73b498543977867adbaed51b531bdba226","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2bf0b91c_119b72a5","updated":"2022-07-18 13:04:24.000000000","message":"Still looks good to me! Thanks! Let\u0027s release this.","commit_id":"972e9c56795710b6e4d65f5dc1f32c411f451141"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"b1f44efea76ffe89936c27c0c348e964eff1df9f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"460d3c54_c897efec","updated":"2022-07-15 20:33:46.000000000","message":"Thank you for the reviews.\n@Goutham, the change you pointed out has merged, I have updated the hash with on the latest PS.\nPlease check the latest PS","commit_id":"972e9c56795710b6e4d65f5dc1f32c411f451141"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"6302d07d495f236ab219841fcb75a2716b13cff2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1257901b_a2546b3e","updated":"2022-07-18 12:18:06.000000000","message":"Thanks Carlos, LGTM","commit_id":"972e9c56795710b6e4d65f5dc1f32c411f451141"}],"deliverables/zed/manila-tempest-plugin.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9546ac11794b2d6a04dc1809fbe7082aa37d730b","unresolved":true,"context_lines":[{"line_number":14,"context_line":"    projects:"},{"line_number":15,"context_line":"      - repo: openstack/manila-tempest-plugin"},{"line_number":16,"context_line":"        hash: ee15891b5d5e02ff355f50ad7647ead9d8ec6841"},{"line_number":17,"context_line":"  - version: 1.8.1"},{"line_number":18,"context_line":"    projects:"},{"line_number":19,"context_line":"      - repo: openstack/manila-tempest-plugin"},{"line_number":20,"context_line":"        hash: 684ab8a44a1209366841fffd1bad973bb2365797"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7adb6eb4_f8b551fb","line":17,"range":{"start_line":17,"start_character":13,"end_line":17,"end_character":18},"updated":"2022-07-13 09:46:25.000000000","message":"tempest\u0027s minimum version is bumped, so according to semver this requires a MINOR version bump. Furthermore, i see there a lot of \u0027removal\u0027 patch: is there amongst them maybe something that causes \u0027backward incompatibility\u0027 thus requiring MAJOR version bump?","commit_id":"af68a51bcfef8ea4ce9ce0bd92b640172e0aaeab"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0b4032312fb456583391d043eb0a6b9d46151325","unresolved":false,"context_lines":[{"line_number":14,"context_line":"    projects:"},{"line_number":15,"context_line":"      - repo: openstack/manila-tempest-plugin"},{"line_number":16,"context_line":"        hash: ee15891b5d5e02ff355f50ad7647ead9d8ec6841"},{"line_number":17,"context_line":"  - version: 1.8.1"},{"line_number":18,"context_line":"    projects:"},{"line_number":19,"context_line":"      - repo: openstack/manila-tempest-plugin"},{"line_number":20,"context_line":"        hash: 684ab8a44a1209366841fffd1bad973bb2365797"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"14d070e2_577ff3ab","line":17,"range":{"start_line":17,"start_character":13,"end_line":17,"end_character":18},"in_reply_to":"28abe717_7964b14f","updated":"2022-07-14 01:58:35.000000000","message":"Minor sounds nice. All changes mentioned have merged today, submitted an updated PS.\nGoutham, I know we may have a couple of fixes incoming that will address issues for older branches. Should we wait for those as well?","commit_id":"af68a51bcfef8ea4ce9ce0bd92b640172e0aaeab"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ba3af7fda0a37f082aea715422ddbad2b0f9e42b","unresolved":true,"context_lines":[{"line_number":14,"context_line":"    projects:"},{"line_number":15,"context_line":"      - repo: openstack/manila-tempest-plugin"},{"line_number":16,"context_line":"        hash: ee15891b5d5e02ff355f50ad7647ead9d8ec6841"},{"line_number":17,"context_line":"  - version: 1.8.1"},{"line_number":18,"context_line":"    projects:"},{"line_number":19,"context_line":"      - repo: openstack/manila-tempest-plugin"},{"line_number":20,"context_line":"        hash: 684ab8a44a1209366841fffd1bad973bb2365797"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"28abe717_7964b14f","line":17,"range":{"start_line":17,"start_character":13,"end_line":17,"end_character":18},"in_reply_to":"7adb6eb4_f8b551fb","updated":"2022-07-13 11:48:40.000000000","message":"the removals are a refactoring of the code; We\u0027re starting to consume the same code paths from tempest.lib instead of having it in tree. \n\nI agree this ought to be 1.9.0 to call out the change in the tempest version required. Carlos: Do we want to wait for a day or two to get these in as well: https://review.opendev.org/q/topic:tempest-scenario-manager-cleanup+status:open+project:openstack/manila-tempest-plugin","commit_id":"af68a51bcfef8ea4ce9ce0bd92b640172e0aaeab"}]}