)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"cdf94294c7620269857802a88b96019dd1645eab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3cce5d6e_20c267ce","updated":"2022-06-08 12:42:27.000000000","message":"We\u0027ll risk running very close to the total job timeout (3h) so i\u0027m not a fan of this change. However, this bump is unavoidable because of vanilla Jammy\u0027s footprint. Maybe its time we split up the generic driver job again (unfortunately); \n\nCurrently stable/victoria is broken because tempest is pinned there and manila-tempest-plugin isn\u0027t. I noticed a change to drop testing of that branch. I\u0027m okay with it, since its happening across all tempest plugins: https://review.opendev.org/c/openstack/manila-tempest-plugin/+/843307/\nyou could add a Depends-On on that change to get this change to pass CI","commit_id":"7ad08effc37a4f9047a0585ffd900264aa523df3"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"e999c02701a855e95a77120787d68fce4f3e917d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"53c64e03_8b9a73da","in_reply_to":"3cce5d6e_20c267ce","updated":"2022-06-08 20:14:44.000000000","message":"You are right. I have first splitted the job into two (api and scenario), but we can also have the tests having a long list of irrelevant files and skipping a run if nothing in the API has been touched. Let\u0027s bring some more attention into this.","commit_id":"7ad08effc37a4f9047a0585ffd900264aa523df3"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b038d4f4f354b66e4c03b2924f9605f20d672120","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7df91c78_caa4a9a0","updated":"2022-06-09 16:07:55.000000000","message":"Thanks, LGTM. This should pass once we fix the LVM victoria job.","commit_id":"bcb16bb8bfbe1c45042eafb504222ac10161a761"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"610a6a21d31147a222141f04bf8edd11de2a65d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"04581205_534dc14a","updated":"2022-06-09 18:08:36.000000000","message":"recheck\nVictoria CI job just merged","commit_id":"bcb16bb8bfbe1c45042eafb504222ac10161a761"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"4ea8c9cf1023bfe0d12838dedae37b6a7b2bf36f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"904e6180_a065b39b","updated":"2022-06-10 15:29:05.000000000","message":"Thanks for the description of the issue in the commit message, seems a sane approach to me, thanks!","commit_id":"cc8a651faeac27731a981eb54765da72a162d93a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"f7d6f25d471745a9bb9f9281d386875eb42e704d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"48ba3794_891d0f8e","updated":"2022-06-10 13:39:28.000000000","message":"Thanks for your explanation in commit message, I think that this is the correct approach here. LGTM","commit_id":"cc8a651faeac27731a981eb54765da72a162d93a"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"2df6cfb0c2bd98ba66059db8fb9286463c55a0f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"27237f96_4723a0ad","updated":"2022-06-10 14:16:50.000000000","message":"recheck","commit_id":"cc8a651faeac27731a981eb54765da72a162d93a"}]}
