)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"34ede4c50351653a3b1bf14805c0e141dd2618f5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"85134d90_a1d38fd7","updated":"2023-12-20 21:16:33.000000000","message":"Looks like swift will have to adopt the mocking they do in their tests. Running some more integration tests with this might also be helpful.","commit_id":"d5166bebafb78f43ffec1e98671b60335a880dbd"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"26a6315e91add1846f7b8cc2cc2869a584c12bde","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b0bd153b_6e16d6b2","in_reply_to":"1dad75f2_42a88bf8","updated":"2024-01-03 21:22:46.000000000","message":"No worries! And FWIW, I found I *could* dodge the bug by adjusting the mocking, so it turned out you were right: https://review.opendev.org/c/openstack/swift/+/904459\n\nI think it might still be best to wait for a new eventlet, though.","commit_id":"d5166bebafb78f43ffec1e98671b60335a880dbd"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"99394f1c93e5d89efcaac6600e7d9810842ba6ef","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1dad75f2_42a88bf8","in_reply_to":"70f2af4e_a3d4f785","updated":"2024-01-02 17:16:57.000000000","message":"I only took a rough look at the job failure, sorry for the false blame, but at least it seems to have gotten things moving ... ;-)\n\nhttps://github.com/eventlet/eventlet/pull/866 proposed now, likely there\u0027ll be another version soon","commit_id":"d5166bebafb78f43ffec1e98671b60335a880dbd"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ae3d565a3a3287ef73fef6ebb2603c381dd97092","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"70f2af4e_a3d4f785","in_reply_to":"85134d90_a1d38fd7","updated":"2023-12-29 09:00:50.000000000","message":"I\u0027m skeptical that\n```\nFatal Python error: Segmentation fault\n```\nwill be resolved by simply adjusting mocking. I wrote up https://github.com/eventlet/eventlet/issues/864 -- I\u0027ll see if I can dig into it more, but it seems like eventlet may need to consciously _not_ attempt to green existing locks upon monkey-patching 😕","commit_id":"d5166bebafb78f43ffec1e98671b60335a880dbd"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"af17c67f5786e4eac36eb0ed7d48b8384f3d7223","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e5ac626a_9a940225","in_reply_to":"b0bd153b_6e16d6b2","updated":"2024-01-10 17:03:50.000000000","message":"As the root cause of the segfault is a pure python bug [1][2], with no reference to eventlet, I don\u0027t think that a new evenlet version would help much here.\n\nI think that until this bug is solved in the CPython side, we need the mock adjustment.\n\nI\u0027d suggest:\n\n1. to release a new swift version (not sure if cross tests are based on master branches or on specific pulled versions of services)\n2. recheck this patch\n3. if tests are ok with the mock adjustments, then, merge this patch.\n\nThoughts?\n\n[1] https://github.com/python/cpython/issues/113631\n[2] https://github.com/faster-cpython/ideas/issues/643","commit_id":"d5166bebafb78f43ffec1e98671b60335a880dbd"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"0e6943aee5e9e656e4f7019f4e282198c835065f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"efda8e9d_53c54fe8","updated":"2024-01-12 08:46:37.000000000","message":"Gates are now happy","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"12f1afc0d47334d9d3009b93f59b8703c4809954","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"71a93641_6a85019a","updated":"2024-01-11 15:43:53.000000000","message":"This eventlet should help to manage the previous segfaults:\nhttps://github.com/eventlet/eventlet/commit/dec2ed5de1f170fad9539f0638278dbe36a3672f","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"06f4674cf1b78d7c1e89eb3d0f7763fd74e4f26c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"47b0043b_32c1be6c","updated":"2024-01-16 11:03:05.000000000","message":"actually I mixed this up, it was ceilometer once failing in https://review.opendev.org/c/openstack/requirements/+/905606, so I think we can attribute both to general gate instability and go on with this and see how it fares \"in the wild\"","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"6f2be51d40b6cfa04794bedd8d3e71a1f4e43cdf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4d2baee3_f2bae890","updated":"2024-01-12 16:43:58.000000000","message":"recheck","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"287320094115668148baa162436051d5831f610f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fd80c6b6_bcfe0155","updated":"2024-01-11 19:49:53.000000000","message":"recheck py38-uc job ran too early?","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"077dbeec9bc729409db1b2658e1c913731f75058","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3d13a303_1e22a4dc","updated":"2024-01-16 09:14:24.000000000","message":"recheck see more cinder results\n\nactually cinder tests were unstable in another patch, too, so maybe not related after all","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"0b9cd990a1c1ac1205d78486cb60ffe19c4e1a30","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e23cf29f_64e16d8b","updated":"2024-01-12 16:43:46.000000000","message":"the failure in the cinder unit test looks related to eventlet, so even if it is only an instability instead of a 100% failure, I think this warrants closer inspection","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"603c6d37c07f380d095191006bdedddc0b8e3ada","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"da678984_00959dee","in_reply_to":"3d13a303_1e22a4dc","updated":"2024-01-16 09:35:35.000000000","message":"Do you remember which patch also failed?\nCould be worth to compare and to keep an eye on.","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"8543395afed6e751865dba9aa7836263609acbff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fa09f59f_081927c5","in_reply_to":"47b0043b_32c1be6c","updated":"2024-01-16 13:21:27.000000000","message":"WFM","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"0b597105aaf31dda4c899d47dc3aa823c757ce18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f001b2bd_cf15cef8","in_reply_to":"71a93641_6a85019a","updated":"2024-01-11 15:44:25.000000000","message":"*patch","commit_id":"3728e28a91f25f7c1023c07f563dc0e0ce688312"}]}