)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"f7c963eb25a30b6354380256cbb545d9e23c8db9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9f3d9265_32b4cb62","updated":"2021-12-20 13:24:25.000000000","message":"recheck","commit_id":"336c41c1234cd143b5f03cc19913d33f760a13e5"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"8d98af23faad757e3d769f93918857d004312b71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"50bb24f8_8e4f0933","updated":"2022-03-08 19:26:08.000000000","message":"Carlos, \n\nI\u0027m nervous about the implications of this monkey patching; paramiko can change underneath us and break stuff. paramiko is being used in other projects here including nova, cinder, sahara, mistral, etc: https://codesearch.opendev.org/?q\u003dimport%20paramiko\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003d\n\nIs this approach being pursued in any other project?\n\nAlso, the commit message suggests a link is being added, what does that mean?","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"24d71a57316c3b12e07fe05b17636928d4490093","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6d8785e8_7d072925","updated":"2022-03-03 18:14:14.000000000","message":"LGTM thanks Carlos","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"0cddacef3f8de5121c25288d85e11ff8b42bff9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"76995058_cd01bbb4","updated":"2022-03-08 12:49:14.000000000","message":"Looks good, thanks Carlos!","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"54ee65b795a27d8ea609ca6c032df76742cb6168","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a58e4724_d0cf4bdf","updated":"2022-03-03 18:13:28.000000000","message":"Looks good, thanks.","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2d161847ff61aa5a005da544e342344ac9f10893","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"adcf7867_45fcb4fb","updated":"2022-06-21 12:55:30.000000000","message":"Seeing as there is poor upstream support for this; i agree there isn\u0027t a better choice. Lets get this in and hope that this monkeypatching doesn\u0027t break on us. Thanks all!","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"180596f11397711b7ff398760494e75b44924acf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6f75f6cc_4222aece","updated":"2022-02-17 18:13:07.000000000","message":"recheck","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"a5a15c47e68045e1c0609a699231ee0687293251","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"bebf6b82_c048bb15","updated":"2022-06-14 12:44:45.000000000","message":"recheck\nLogs are gone","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"4976ba5876d64e425b4ddef5ac8acffa415e347f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"85cdf9ab_1626c691","updated":"2022-06-20 13:49:07.000000000","message":"run Pure Storage CI","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"c946d32e5cb4a5d32a4a7eaa96fbc69c6a10be69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"dcd75d9f_843ae1a8","updated":"2022-06-17 14:44:15.000000000","message":"run Pure Storage CI","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"74c5fc5acac9bc3ca7a58e953c51c5b4a1447f9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e254668b_1f480d4f","updated":"2022-06-17 18:25:35.000000000","message":"run Pure Storage CI","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"9acf64f2f235b0a91222b39b52103b51817c2bf2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"daff50e6_524e6126","in_reply_to":"50bb24f8_8e4f0933","updated":"2022-03-08 19:31:43.000000000","message":"\u003e Also, the commit message suggests a link is being added, what does that mean?\nJust bad wording :) - The idea of this change is only to monkey patch paramiko\n\n\u003e Is this approach being pursued in any other project?\nYes, it is:\nhttps://review.opendev.org/c/openstack/tempest/+/822560\nBut it is only until we can get the pull request Ade proposed into paramiko.","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"e18c3673d52fbdfa5de86060788f9be63bc43eac","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"81d51c5a_188e3388","in_reply_to":"58e7b634_03346d83","updated":"2022-03-09 04:29:36.000000000","message":"Goutham,\n\nThe problem is that paramiko is incredibly slow moving.  There are currently 226 open PRs, some up to five years old.  If we have to wait for paramiko, we could be waiting for a very long time indeed.\n\nGiven that paramiko changes so slowly, its unlikely this will change anytime soon - or that the change will be much different than what is here.  The change is actually quite small.\n\nIf it does change in a breaking way, we\u0027ll know right away because of broken FIPS enabled jobs, and we\u0027ll be able to fix it.  For stable branches, we can always constrain the version of paramiko.\n\nFinally, as we move towards FIPS compliance (rather than  just compatibility), we\u0027re likely to either replace paramiko, or provide the option to use a different FIPS compatible library.  At that point, we could remove this monkey-patch.","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"c248c465a4e93d91d4b5bd914544885129f6b003","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a91b7fe5_8ce7ffa9","in_reply_to":"daff50e6_524e6126","updated":"2022-03-08 19:32:18.000000000","message":"Sorry for the misleading commit message :/","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"5a3a2858998adb18e29c78aa044035dbee1933a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"58e7b634_03346d83","in_reply_to":"daff50e6_524e6126","updated":"2022-03-08 19:33:36.000000000","message":"tempest is a branchless project - it worries me less if something breaks with tempest; if we decide to do this, we\u0027d be leaving this in code potentially forever in a released stable branch...","commit_id":"72a0a1b9d8ef082d0970192a6ae1446e4f346d6f"}]}