)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"568d0f7f22951a97cdad79ac3f7f3054c825ccc8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e250522a_3a803107","updated":"2026-10-01 03:57:49.000000000","message":"@erkin.mussurmankulov@ps.kz thanks for looking into this. \n\ncryptography does provide the same functionality (it also links against openssl) and it\u0027s good to have only a single module dealing with crypto. \n\nThe change you merged is unrelated to this one. The reason I\u0027m posting this is really to reduce dependencies and having only cryptography dealing with crypto.","commit_id":"64702cfc84c340d1c1191714313634e995b2e2a9"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"2c929d0c559eb86bafa9d83f3673c5fff6f58a5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5e3a6e5e_2643f887","updated":"2026-09-28 12:27:18.000000000","message":"Hello! Good job, thank you for the contribution. I will do a review asap.\n\nWe got an error in our CI:\n```\nFile \"/opt/guest-agent-venv/lib/python3.12/site-packages/trove/common/ssl.py\", line 86, in certificate_details\n    ext \u003d cert.get_extension(i)\n          ^^^^^^^^^^^^^^^^^^\n          AttributeError: \u0027X509\u0027 object has no attribute \u0027get_extension\u0027. Did you mean: \u0027get_version\u0027?\n```\n\nIs this MR related to this error? Did you also hit this in your cloud recently?\n\nDo you also have some opinions about the pyOpenSSL library in Python? Should we really remove it in favor of cryptography?","commit_id":"64702cfc84c340d1c1191714313634e995b2e2a9"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"f8311d0987f2e4ac1da87b6cd5b846e082ee6fb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20d1482d_8846ef04","updated":"2026-09-30 05:45:46.000000000","message":"We decided to implement another patch quickly, based on this one:\nhttps://review.opendev.org/c/openstack/trove/+/1007836\n\nIt provides a simpler, more focused fix, avoids unnecessary compatibility code, and safely handles unsupported SAN types without introducing additional failure cases.\n\n____________\nBut anyway, thank you for the contribution!","commit_id":"64702cfc84c340d1c1191714313634e995b2e2a9"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"13cb73b0b4f0abd79b877509af4d8ff205ad3ad8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4730f052_81b49489","updated":"2026-09-29 07:12:54.000000000","message":"recheck","commit_id":"64702cfc84c340d1c1191714313634e995b2e2a9"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"0538ca48bd72192cd064bb231d35bc85401d08df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4afb55d8_467e927e","in_reply_to":"7735c606_80714961","updated":"2026-10-05 09:28:43.000000000","message":"you are right. sorry for the noise and thanks for removing pyopenssl!","commit_id":"64702cfc84c340d1c1191714313634e995b2e2a9"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"c70d0c60bca22dec6c838125639eff60b586d85f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7735c606_80714961","in_reply_to":"e250522a_3a803107","updated":"2026-10-01 08:49:37.000000000","message":"The change that we merged is related. It does the same. Please take a closer look:\nhttps://review.opendev.org/c/openstack/trove/+/1007836\n\nIt removes the `pyOpenSSL` extension from `requirements.txt` but without implementing some low-level ASN checks.\n\nMaybe there is something that I don\u0027t see or don\u0027t understand. Please provide detailed info, then.\n\nAlso, please resolve the conflict in this MR if it\u0027s still relevant.","commit_id":"64702cfc84c340d1c1191714313634e995b2e2a9"}]}