nyl1001

Results 14 comments of nyl1001

> > > 加了一个单元测试,实现应该是没问题的:https://github.com/yunionio/cloudpods/pull/19312/files > > > > > > 你这用例不够,无法验证出问题,你用实际的keystone的configmap配置作为用例试试吧,深拷贝没有你这么使用的 > > 我试过你的改动,没有起到作用,改完configmap后,重启服务配置依然是20,本身设计就是需要用climc去改配置,不然删除configmap后会重置配置,没办法持久化到数据库中 没起作用的原因就是copyOptions不是深拷贝,明白吗?修改配置用climc没有问题,没有反对你这个。你也别谈你的设计就是故意让这个copyOptions不能完成深拷贝,故意让他不起作用的哈,那我真就呵呵了

你用我这个改动后的函数,就可以正确完成copyOptions,这个我是在我们这边的生产环境验证了的,不然也不会在这里跟你们费劲的解释了

不再使用cgo libvirt的原因,一来cgo libvirt不方便调试且难以维护,二来c代码申请的内存和变量需要手动释放,容易产生内存泄露。

本pr是给下述四个敏感信息接口加auth鉴权: proto.AdminListVols: proto.MsgMasterVolListReq, proto.UserList: proto.MsgMasterUserListReq, proto.UserGetInfo: proto.MsgMasterUserInfoGetReq, proto.UsersOfVol: proto.MsgMasterUsersOfVolReq,