)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"4e504c6ff7823f4a4f9129c7e6fa0ec5c3524ece","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2024-08-02 15:50:01 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"update requirements for VAST manila driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I43b1fd35a0c31db8664a7c4f8be55cf4ea6dfa3e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b5052d8d_5ed81361","line":8,"updated":"2024-08-02 19:34:18.000000000","message":"could you please explain why you\u0027re doing this?\n\nPerhaps:\n\n```\nThe requirements added correspond to the latest versions of the\nlibraries or the current upper constraints. The VAST driver doesn\u0027t\nneed the latest versions of these libraries. Hence, we can keep\nthe lower requirements old enough so distributors have some wiggle\nroom.\n```","commit_id":"7dc3a39df6e278d08a853bae111d35c5821c6b60"},{"author":{"_account_id":36962,"name":"Volodymyr Boiko","display_name":"Volodymyr","email":"volodymyr.boiko@vastdata.com","username":"fnn45"},"change_message_id":"d2d6443b4d5c04ffc9fe92dbf14d242ef36b3072","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2024-08-02 15:50:01 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"update requirements for VAST manila driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I43b1fd35a0c31db8664a7c4f8be55cf4ea6dfa3e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d061e825_03b5ad87","line":8,"in_reply_to":"7a3bceab_3e5a3fa5","updated":"2024-08-02 22:40:46.000000000","message":"Done","commit_id":"7dc3a39df6e278d08a853bae111d35c5821c6b60"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"657766dda23028f93f689077514821d3a9b9f896","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2024-08-02 15:50:01 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"update requirements for VAST manila driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I43b1fd35a0c31db8664a7c4f8be55cf4ea6dfa3e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a3bceab_3e5a3fa5","line":8,"in_reply_to":"b5052d8d_5ed81361","updated":"2024-08-02 19:39:55.000000000","message":"++ on this suggestion. @volodymyr.boiko@vastdata.com could you please update the commit message with this suggestion?","commit_id":"7dc3a39df6e278d08a853bae111d35c5821c6b60"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"41fb64a80cd15af7fb1911138a9271f167d5a5a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2a68f937_b31556c0","updated":"2024-08-02 19:39:07.000000000","message":"Hey, Volodymyr, change LGTM. However, I believe we could have more context in the commit message, as Goutham suggested. This is important in case someone is looking back on the commit history and trying to understand why the requirements are changing.","commit_id":"7dc3a39df6e278d08a853bae111d35c5821c6b60"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"4e504c6ff7823f4a4f9129c7e6fa0ec5c3524ece","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"004c676e_3746b5da","updated":"2024-08-02 19:34:18.000000000","message":"LGTM; please add some more details to the commit message if you make another patch","commit_id":"7dc3a39df6e278d08a853bae111d35c5821c6b60"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"83b46b3454d3ad2b7fce3addb84a0dc6f80ffe91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dfb232c2_00f6c290","updated":"2024-08-02 23:16:07.000000000","message":"Perfect; thank you Volodymr!","commit_id":"14ef89161130a41bdc6e916de4fab9b08ae439f6"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"6a093f6d2ce9d9aee60af23da53e0d49b5b64d5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6dae93c5_836132b5","updated":"2024-08-02 23:00:54.000000000","message":"lgtm, thanks for the update!","commit_id":"14ef89161130a41bdc6e916de4fab9b08ae439f6"}]}
