LockExclusive crash about go-ceph HOT 7 CLOSED

This issue only affect ceph after infernalis, which should be caused by librbd code change.
rbd_open didn't initialize image_wacher for ImageCtx. Later ImageCtx::notify_update uses the pointer and cause segfault.

from go-ceph.

I'm trying to get caught up on go-ceph issues, sorry about the delay. Do I understand correctly that the code works ok before infernalis and is broken after infernalis?

from go-ceph.

how hard would it be to provide a code snipper that reproduces the problem?

from go-ceph.

oh, i'm just using rdb-docker-plugin. didn't actually write the code.
and yes, it works with older version of cepth

from go-ceph.

@noahdesu you are right, this issue won't happen on hammer at least. I also met this issue when using rbd-docker-plugin on recent jewel release. I just made some comments on ceph/ceph#9660 and hope to fix this issue soon.

If you would like to reproduce it, here is a golang code snippet, but pls ensure rbd image is existed before you run this code.

package main

import (
        "github.com/ceph/go-ceph/rbd"
        "github.com/ceph/go-ceph/rados"
        "fmt"
)

func main() {
        fmt.Printf("test_librbd\n")

        conn, _ := rados.NewConn()
        conn.ReadDefaultConfigFile()
        conn.Connect()

        poolname := "rbd"
        ioctx, err := conn.OpenIOContext(poolname)
        if err != nil {
                fmt.Printf("open pool fails: %s\n", err)
                return
        }

        // ensure rbd image is existed already
        imagename := "rbd_image_1"
        img := rbd.GetImage(ioctx, imagename)

        err = img.Open(true)
        if err != nil {
                fmt.Printf("fail to open image: %s\n", err)
                return
        }
        defer img.Close()

        locker := "test_lock"
        err = img.LockExclusive(locker)
        if err != nil {
                fmt.Printf("fail to lock image: %s\n", err)
                return
        }
        fmt.Printf("lock done\n")

        err = img.Unlock(locker)
        if err != nil {
                fmt.Printf("fail to unlock image: %s\n", err)
                return
        }
        fmt.Printf("unlock done\n")
}

from go-ceph.

Perfect, thanks. That will make things much easier!

from go-ceph.

Hello, I'm automatically closing all issues filed prior to 2018 that have not been commented on recently. If this issue is still relevant to you please reopen the issue and restart the discussion. Thank you for your understanding.

from go-ceph.