diff --git a/pkg/controllers/pod/pod_controller.go b/pkg/controllers/pod/pod_controller.go index ab0ad0122..bc27dbb13 100644 --- a/pkg/controllers/pod/pod_controller.go +++ b/pkg/controllers/pod/pod_controller.go @@ -88,7 +88,7 @@ func (r *PodReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.R return common.ResultRequeue, err } contextID := *node.Id - nsxSubnetPortState, err := r.Service.CreateOrUpdateSubnetPort(pod, nsxSubnetPath, contextID, &pod.ObjectMeta.Labels) + nsxSubnetPortState, err := r.Service.CreateOrUpdateSubnetPort(pod, nsxSubnetPath, contextID, &pod.ObjectMeta.Labels, false) if err != nil { log.Error(err, "failed to create or update NSX subnet port, would retry exponentially", "pod.Name", req.NamespacedName, "pod.UID", pod.UID) updateFail(r, &ctx, pod, &err) diff --git a/pkg/controllers/subnetport/subnetport_controller.go b/pkg/controllers/subnetport/subnetport_controller.go index 03b1da994..ba254a9ec 100644 --- a/pkg/controllers/subnetport/subnetport_controller.go +++ b/pkg/controllers/subnetport/subnetport_controller.go @@ -94,7 +94,19 @@ func (r *SubnetPortReconciler) Reconcile(ctx context.Context, req ctrl.Request) return common.ResultRequeue, err } log.Info("got labels from virtualmachine for subnetport", "subnetPort.UID", subnetPort.UID, "virtualmachine name", subnetPort.Spec.AttachmentRef.Name, "labels", labels) - nsxSubnetPortState, err := r.Service.CreateOrUpdateSubnetPort(subnetPort, nsxSubnetPath, "", labels) + + isVmSubnetPort := true + if subnetPort.Labels[servicecommon.LabelImageFetcher] == "true" { + isVmSubnetPort = false + if labels == nil { + labels = &map[string]string{} + } + (*labels)[servicecommon.LabelImageFetcher] = "true" + } + // specified by user, or default to the node id of the VM the pod runs on + contextID := subnetPort.Annotations[servicecommon.AnnotationNodeID] + + nsxSubnetPortState, err := r.Service.CreateOrUpdateSubnetPort(subnetPort, nsxSubnetPath, contextID, labels, isVmSubnetPort) if err != nil { log.Error(err, "failed to create or update NSX subnet port, would retry exponentially", "subnetport", req.NamespacedName) updateFail(r, &ctx, subnetPort, &err) @@ -157,8 +169,8 @@ func (r *SubnetPortReconciler) SetupWithManager(mgr ctrl.Manager) error { MaxConcurrentReconciles: runtime.NumCPU(), }). Watches(&vmv1alpha1.VirtualMachine{}, - handler.EnqueueRequestsFromMapFunc(r.vmMapFunc), - builder.WithPredicates(predicate.LabelChangedPredicate{})). + handler.EnqueueRequestsFromMapFunc(r.vmMapFunc), + builder.WithPredicates(predicate.LabelChangedPredicate{})). Complete(r) // TODO: watch the virtualmachine event and update the labels on NSX subnet port. } diff --git a/pkg/nsx/services/common/types.go b/pkg/nsx/services/common/types.go index 163209778..16274bc48 100644 --- a/pkg/nsx/services/common/types.go +++ b/pkg/nsx/services/common/types.go @@ -56,6 +56,8 @@ const ( TagScopeVMNamespaceUID string = "nsx-op/vm_namespace_uid" TagScopeVMNamespace string = "nsx-op/vm_namespace" LabelDefaultSubnetSet string = "nsxoperator.vmware.com/default-subnetset-for" + LabelImageFetcher string = "wcp.vmware.com/image-fetcher" + AnnotationNodeID string = "nsx.vmware.com/node-id" LabelDefaultVMSubnetSet string = "VirtualMachine" LabelDefaultPodSubnetSet string = "Pod" DefaultPodSubnetSet string = "pod-default" diff --git a/pkg/nsx/services/subnetport/builder.go b/pkg/nsx/services/subnetport/builder.go index 0372b1ae3..bdb0250ea 100644 --- a/pkg/nsx/services/subnetport/builder.go +++ b/pkg/nsx/services/subnetport/builder.go @@ -20,7 +20,7 @@ var ( String = common.String ) -func (service *SubnetPortService) buildSubnetPort(obj interface{}, nsxSubnetPath string, contextID string, labelTags *map[string]string) (*model.VpcSubnetPort, error) { +func (service *SubnetPortService) buildSubnetPort(obj interface{}, nsxSubnetPath string, contextID string, labelTags *map[string]string, isVmSubnetPort bool) (*model.VpcSubnetPort, error) { var objName, objNamespace, uid, appId string switch o := obj.(type) { case *v1alpha1.SubnetPort: @@ -54,9 +54,20 @@ func (service *SubnetPortService) buildSubnetPort(obj interface{}, nsxSubnetPath } namespace_uid := namespace.UID tags := util.BuildBasicTags(getCluster(service), obj, namespace_uid) + var tagsFiltered []model.Tag + for _, tag := range tags { + if isVmSubnetPort && *tag.Scope == common.TagScopeNamespaceUID { + continue + } + if !isVmSubnetPort && *tag.Scope == common.TagScopeVMNamespaceUID { + continue + } + tagsFiltered = append(tagsFiltered, tag) + } + if labelTags != nil { for k, v := range *labelTags { - tags = append(tags, model.Tag{Scope: String(k), Tag: String(v)}) + tagsFiltered = append(tagsFiltered, model.Tag{Scope: String(k), Tag: String(v)}) } } nsxSubnetPort := &model.VpcSubnetPort{ @@ -68,14 +79,14 @@ func (service *SubnetPortService) buildSubnetPort(obj interface{}, nsxSubnetPath TrafficTag: common.Int64(0), Type_: String("STATIC"), }, - Tags: tags, + Tags: tagsFiltered, Path: &nsxSubnetPortPath, ParentPath: &nsxSubnetPath, } if appId != "" { nsxSubnetPort.Attachment.AppId = &appId - nsxSubnetPort.Attachment.ContextId = &contextID } + nsxSubnetPort.Attachment.ContextId = &contextID return nsxSubnetPort, nil } diff --git a/pkg/nsx/services/subnetport/subnetport.go b/pkg/nsx/services/subnetport/subnetport.go index d437dfcc6..3ddce4477 100644 --- a/pkg/nsx/services/subnetport/subnetport.go +++ b/pkg/nsx/services/subnetport/subnetport.go @@ -73,7 +73,7 @@ func InitializeSubnetPort(service servicecommon.Service) (*SubnetPortService, er return subnetPortService, nil } -func (service *SubnetPortService) CreateOrUpdateSubnetPort(obj interface{}, nsxSubnetPath string, contextID string, tags *map[string]string) (*model.SegmentPortState, error) { +func (service *SubnetPortService) CreateOrUpdateSubnetPort(obj interface{}, nsxSubnetPath string, contextID string, tags *map[string]string, isVmSubnetPort bool) (*model.SegmentPortState, error) { var uid string switch o := obj.(type) { case *v1alpha1.SubnetPort: @@ -82,7 +82,7 @@ func (service *SubnetPortService) CreateOrUpdateSubnetPort(obj interface{}, nsxS uid = string(o.UID) } log.Info("creating or updating subnetport", "nsxSubnetPort.Id", uid, "nsxSubnetPath", nsxSubnetPath) - nsxSubnetPort, err := service.buildSubnetPort(obj, nsxSubnetPath, contextID, tags) + nsxSubnetPort, err := service.buildSubnetPort(obj, nsxSubnetPath, contextID, tags, isVmSubnetPort) if err != nil { log.Error(err, "failed to build NSX subnet port", "nsxSubnetPort.Id", uid, "nsxSubnetPath", nsxSubnetPath, "contextID", contextID) return nil, err @@ -271,7 +271,7 @@ func (service *SubnetPortService) Cleanup() error { subnetPorts := service.SubnetPortStore.List() log.Info("cleanup subnetports", "count", len(subnetPorts)) for _, subnetPort := range subnetPorts { - subnetPortID := types.UID(*subnetPort.(model.VpcSubnetPort).Id) + subnetPortID := types.UID(*subnetPort.(*model.VpcSubnetPort).Id) err := service.DeleteSubnetPort(subnetPortID) if err != nil { log.Error(err, "cleanup subnetport failed", "subnetPortID", subnetPortID) diff --git a/pkg/util/utils.go b/pkg/util/utils.go index e1f07dad2..5f9288cb2 100644 --- a/pkg/util/utils.go +++ b/pkg/util/utils.go @@ -447,7 +447,6 @@ func BuildBasicTags(cluster string, obj interface{}, namespaceID types.UID) []mo Tag: String(strings.Join(common.TagValueVersion, ".")), }, } - isVmSubnetPort := false switch i := obj.(type) { case *v1alpha1.StaticRoute: tags = append(tags, model.Tag{Scope: String(common.TagScopeNamespace), Tag: String(i.ObjectMeta.Namespace)}) @@ -469,9 +468,12 @@ func BuildBasicTags(cluster string, obj interface{}, namespaceID types.UID) []mo tags = append(tags, model.Tag{Scope: String(common.TagScopeSubnetSetCRUID), Tag: String(string(i.UID))}) case *v1alpha1.SubnetPort: tags = append(tags, model.Tag{Scope: String(common.TagScopeVMNamespace), Tag: String(i.ObjectMeta.Namespace)}) - isVmSubnetPort = true tags = append(tags, model.Tag{Scope: String(common.TagScopeSubnetPortCRName), Tag: String(i.ObjectMeta.Name)}) tags = append(tags, model.Tag{Scope: String(common.TagScopeSubnetPortCRUID), Tag: String(string(i.UID))}) + if len(namespaceID) > 0 { + tags = append(tags, model.Tag{Scope: String(common.TagScopeVMNamespaceUID), Tag: String(string(namespaceID))}) + tags = append(tags, model.Tag{Scope: String(common.TagScopeNamespaceUID), Tag: String(string(namespaceID))}) + } case *v1.Pod: tags = append(tags, model.Tag{Scope: String(common.TagScopeNamespace), Tag: String(i.ObjectMeta.Namespace)}) tags = append(tags, model.Tag{Scope: String(common.TagScopePodName), Tag: String(i.ObjectMeta.Name)}) @@ -488,14 +490,6 @@ func BuildBasicTags(cluster string, obj interface{}, namespaceID types.UID) []mo log.Info("unknown obj type", "obj", obj) } - if len(namespaceID) > 0 { - if isVmSubnetPort == true { - // In the NSX subnet port created for VM, the namespace uid tag is TagScopeVMNamespaceUID instead of TagScopeNamespaceUID. - tags = append(tags, model.Tag{Scope: String(common.TagScopeVMNamespaceUID), Tag: String(string(namespaceID))}) - } else { - tags = append(tags, model.Tag{Scope: String(common.TagScopeNamespaceUID), Tag: String(string(namespaceID))}) - } - } return tags }