RBD存储模式为RWX时kubernetes多次挂载该PV不报错

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
		})
。。。
}
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值