John Mulligan

Results 271 comments of John Mulligan
trafficstars

Hi all, it's been a while since this PR saw any activity. Based on previous comments I assume @mattbenjamin is not interested in driving this to completion. @leseb are you?...

Thanks all, sorry if I misunderstood the original discussion as well. We'll be patient with this PR. PRs in go-ceph don't go stale/expire but I do try to make sure...

@nixpanic , @ansiwen I think this is the same crash I was getting the other day too ... but I have not managed to clean up my experiments to get...

Hmm. I can't seem to at the moment. @ansiwen seems to be missing from the list. I could assign it to you tho so there's got to be some sort...

> @nixpanic @phlogistonjohn So, apparently this only crashes, when the error returned by `image.Remove()` is ignored, which many of the tests in `rbd_test.go` do (like [here](https://github.com/ceph/go-ceph/blob/master/rbd/rbd_test.go#L393)), and which is why...

There's a lot to unpack in this so please bear with me if I miss anything: > In the meantime I was able to reproduce it with a pure C...

> go-ceph version: v0.8.0 > This version is a bit old at this point (8 months ago), but let's assume that it's not an issue for the moment. Can you...

The pleasant coincidence of a ceph mailing list post indicating that nfs-ganesha once encountered similar problems and a bit more searching shows that there is now api hooks intended for...

> Yeah -- we probably need to add some documentation comments to `struct ceph_client_callback_args`. If you wouldn't mind opening a bug at tracker.ceph.com, we can add some comments in the...

@qiankunli based on this conversation it appears that the issue is not a bug per-se, but something more architectural. As such, we're interested in improving go-ceph to handle this case...