2024-09-26T05:08:31.492Z | <Venky Shankar> @Kotresh H R re: <https://tracker.ceph.com/issues/68146>, do you still have the setup where the client crash was reproduced? |
2024-09-26T05:11:05.779Z | <Kotresh H R> Yeah it's on local vstart cluster |
2024-09-26T05:12:19.395Z | <Venky Shankar> kk, I'm pushing a possible fix once I run it through the basic tests locally. Possible to do a quick verification to see if the crash gets fixed? |
2024-09-26T05:12:57.832Z | <Kotresh H R> I wrote a possible fix too and it worked fine. Please pass on, I will run it through. |
2024-09-26T05:13:40.828Z | <Venky Shankar> oh |
2024-09-26T05:13:47.620Z | <Venky Shankar> In that case, post an PR please. |
2024-09-26T05:14:23.547Z | <Venky Shankar> (didn't know you had a fix and I quickly spinned up one based on the downstream slack update) |
2024-09-26T05:15:55.221Z | <Kotresh H R> Sure, I will do that. |
2024-09-26T05:16:12.338Z | <Kotresh H R> Yeah but gave a shot and it worked!!! |
2024-09-26T05:38:19.288Z | <Kotresh H R> @Venky Shankar Pushed a PR <https://github.com/ceph/ceph/pull/59987> PTAL Need to add test though |
2024-09-26T12:52:39.136Z | <Dhairya Parmar> most of the async io test cases are yet to be merged <https://github.com/ceph/ceph/pull/54435> @Venky Shankar @Kotresh H R maybe these can be run with the setup and see it they reproduce it |
2024-09-26T12:59:20.140Z | <Mark Nelson (nhm)> yeah, I'd be super curious what that would show. |