https://blog.csdn.net/pencc/article/details/84333315
之前在研究上面这个问题时,发现有一个bug,会导致kubernetes多次以RW模式挂载同一个PV。原生代码基于1.11.2如下,这里的主要问题是如果只判断if v != v1.ReadWriteOnce会导致accessModes为RWX时候跳过验证,也就是ReadWriteMany进入这里会忽略util.rbdStatus的判断值,也就是说即便该PV已经被挂载了,也会被k8s再次挂载到别的Pod上去,RDB原生不支持被多次读写挂载,这就可能导致RDB数据同步等出现问题。至于为什么RWX会进入这里?还是看上面给出的这篇文章吧。
// AttachDisk attaches the disk on the node.
func (util *RBDUtil) AttachDisk(b rbdMounter) (string, error) {
。。。
needValidUsed := true
// If accessModes contain ReadOnlyMany, we don't need check rbd status of being used.
if b.accessModes != nil {
for _, v := range b.accessModes {
if v != v1.ReadWriteOnce {
needValidUsed = false
break
}
}
}
err := wait.ExponentialBackoff(backoff, func() (bool, error) {
used, rbdOutput, err := util.rbdStatus(&b)
if err != nil {
return false, fmt.Errorf("fail to check rbd image status with: (%v), rbd output: (%s)", err, rbdOutput)
}
return !needValidUsed || !used, nil
})
。。。
}
发现了这个问题,兴高采烈的准备提PR,一进master代码一看,别人已经修复掉这个问题了,无奈之下只能backport到我们的分支上来了。其实实现也很简单,如下,除开挂载单个ROX PV的情况,其它时候都让它的状态验证由util.rbdStatus来决定。
// AttachDisk attaches the disk on the node.
func (util *RBDUtil) AttachDisk(b rbdMounter) (string, error) {
。。。
needValidUsed := true
if b.accessModes != nil {
// If accessModes only contain ReadOnlyMany, we don't need check rbd status of being used.
if len(b.accessModes) == 1 && b.accessModes[0] == v1.ReadOnlyMany {
needValidUsed = false
}
}
err := wait.ExponentialBackoff(backoff, func() (bool, error) {
used, rbdOutput, err := util.rbdStatus(&b)
if err != nil {
return false, fmt.Errorf("fail to check rbd image status with: (%v), rbd output: (%s)", err, rbdOutput)
}
return !needValidUsed || !used, nil
})
。。。
}