)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"a295d3c78801d7c5f3ecd06f790fa250cd3ce3d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fb93eb4d_f7f45448","updated":"2022-04-04 23:34:43.000000000","message":"Using the string instead of the type that we evaluated just now seems a little weird, but whatever. It\u0027s for something that presumably is on its way out anyway.","commit_id":"5110c5b1a01eb646c5d7c5e4f80da7293152bc6f"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"f69f7a966275a1ac0ebd474424907c330cf4c37f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"13a92a13_82a6f63b","updated":"2022-04-05 02:33:28.000000000","message":"Wow\n\n+ git clone git://github.com/intel/isa-l.git -b v2.30.0 /home/zuul/src/opendev.org/openstack/pyeclib/.tox/py39-compilelibs/isa-l\nCloning into \u0027/home/zuul/src/opendev.org/openstack/pyeclib/.tox/py39-compilelibs/isa-l\u0027...\nfatal: remote error:\n  The unauthenticated git protocol on port 9418 is no longer supported.\nPlease see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.\nERROR: invocation failed (exit code 128)","commit_id":"5110c5b1a01eb646c5d7c5e4f80da7293152bc6f"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"59cc6bc23cda0d9eeb070a7966d7829faa0dab94","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2a08652a_4f2c9cbf","updated":"2022-04-05 20:19:27.000000000","message":"Even better.\n\nThe download error cleared itself too.","commit_id":"4c5dc672d9ad6cbff320bdd3f5a0ae72fe295e87"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"857227dbe1d4a86ea93492cae2b5f57e55bf7b70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a60b301f_75eaa6dc","updated":"2022-04-06 13:42:23.000000000","message":"FWIW, download error was fixed by the parent patch I inserted: https://review.opendev.org/c/openstack/pyeclib/+/836691","commit_id":"4c5dc672d9ad6cbff320bdd3f5a0ae72fe295e87"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ed287598b5dfb04310778d46777f116614c52298","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"646616f3_d0611d75","updated":"2022-04-06 13:12:33.000000000","message":"recheck\n\nHopefully the py310 troubles were just transient.","commit_id":"4c5dc672d9ad6cbff320bdd3f5a0ae72fe295e87"}],"pyeclib/ec_iface.py":[{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"da68929cec7b41cbf04ef26b6c4c0fd56f504f39","unresolved":true,"context_lines":[{"line_number":207,"context_line":"                        warnings.warn(\u0027Jerasure support is deprecated and \u0027"},{"line_number":208,"context_line":"                                      \u0027may be removed in a future release\u0027,"},{"line_number":209,"context_line":"                                      DeprecationWarning, stacklevel\u003d2)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"                else:"},{"line_number":212,"context_line":"                    raise ECBackendNotSupported("},{"line_number":213,"context_line":"                        \"%s is not a valid EC type for PyECLib!\" % value)"}],"source_content_type":"text/x-python","patch_set":1,"id":"8ad8476e_6026f635","line":210,"updated":"2022-04-04 15:44:26.000000000","message":"I confirmed the warning is showed up to my console when I init the jerasure_rs_vand ECDriver in main. Probably as you know, the warning is showed only after python 3.7, I don\u0027t see the message before python3.6 version. And the usage of stacklevel looks fine to me.\n\nhttps://docs.python.org/3/library/warnings.html#default-warning-filter","commit_id":"5110c5b1a01eb646c5d7c5e4f80da7293152bc6f"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"4d2d5c371c661b2c6fe8e351f5d6217428eca035","unresolved":true,"context_lines":[{"line_number":207,"context_line":"                        warnings.warn(\u0027Jerasure support is deprecated and \u0027"},{"line_number":208,"context_line":"                                      \u0027may be removed in a future release\u0027,"},{"line_number":209,"context_line":"                                      DeprecationWarning, stacklevel\u003d2)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"                else:"},{"line_number":212,"context_line":"                    raise ECBackendNotSupported("},{"line_number":213,"context_line":"                        \"%s is not a valid EC type for PyECLib!\" % value)"}],"source_content_type":"text/x-python","patch_set":1,"id":"04325490_25369210","line":210,"in_reply_to":"8ad8476e_6026f635","updated":"2022-04-04 18:01:28.000000000","message":"Oh, that\u0027s a good point -- maybe it\u0027d be better if I used FutureWarning? From the docs, that\u0027s\n\n\u003e for warnings about deprecated features when those warnings are intended for end users of applications that are written in Python\n\nas opposed to DeprecationWarning which is\n\n\u003e for warnings about deprecated features when those warnings are intended for other Python developers\n\nSince EC scheme is typically user-configurable, it seems like this ought to be more of a user-facing warning, rather than developer-facing.\n\nFurther reading:\n- https://docs.python.org/3/library/warnings.html#warning-categories\n- https://peps.python.org/pep-0565/","commit_id":"5110c5b1a01eb646c5d7c5e4f80da7293152bc6f"}]}