Kevin Carter
Kevin Carter
@gemini-code-assist review
@gemini-code-assist review
@oilbeater I think this should now be good to go. I added tests and a migration function, which should detect the existing kube-ovn resources and add the vendor tags accordingly....
Thank you @ycy1766 for reporting this, we're observing this same behavior. ```shell # kubectl-ko nbctl list acl | grep external | grep neutron external_ids : {"neutron:security_group_rule_id"="676924c9-a454-465e-88b0-86700fcc4a97"} external_ids : {"neutron:security_group_rule_id"="dc08c72d-1dcf-448a-8238-bba4b93f0094"} external_ids...
@dosu how might you suggest resolving this issue when OVN is a shared fabric?
> Thanks for the analysis. I think the reasonable approach is to set the `external_id:vendor=kube-ovn` for every ACL that Kube-OVN generates. Garbage collection or migration processes can then use this...
@gemini-code-assist review