Revert "De duplication logic to NF Deploy Fn Param Ref"

[radoslawc]

Reverts #494

[electrocucaracha]

I discovered during a manual End-to-End testing on this change that the operator was having some issues

$ kubectl logs -n free5gc free5gc-operator-6887fb54f8-xbxs8 --kubeconfig /tmp/tmp.B1MiU5THUn_kubeconfig-edge02 -c operator
2024-02-01T23:37:09Z    INFO    controller-runtime.metrics      Metrics server is starting to listen    {"addr": "127.0.0.1:8080"}
2024-02-01T23:37:09Z    INFO    setup   starting manager
2024-02-01T23:37:09Z    INFO    Starting server {"kind": "health probe", "addr": "[::]:8081"}
I0201 23:37:09.167201       1 leaderelection.go:245] attempting to acquire leader lease free5gc/5089c67f.nephio.org...
2024-02-01T23:37:09Z    INFO    starting server {"path": "/metrics", "kind": "metrics", "addr": "127.0.0.1:8080"}
I0201 23:37:09.222363       1 leaderelection.go:255] successfully acquired lease free5gc/5089c67f.nephio.org
2024-02-01T23:37:09Z    DEBUG   events  free5gc-operator-6887fb54f8-xbxs8_29d2d95d-1272-4538-b49a-534e5f96d024 became leader    {"type": "Normal", "object": {"kind":"Lease","namespace":"free5gc","name":"5089c67f.nephio.org","uid":"c498354e-c502-4496-854f-44551ca87b95","apiVersion":"coordination.k8s.io/v1","resourceVersion":"2011"}, "reason": "LeaderElection"}
2024-02-01T23:37:09Z    INFO    Starting EventSource    {"controller": "nfdeployment", "controllerGroup": "workload.nephio.org", "controllerKind": "NFDeployment", "source": "kind source: *v1alpha1.NFDeployment"}
2024-02-01T23:37:09Z    INFO    Starting EventSource    {"controller": "nfdeployment", "controllerGroup": "workload.nephio.org", "controllerKind": "NFDeployment", "source": "kind source: *v1.Deployment"}
2024-02-01T23:37:09Z    INFO    Starting EventSource    {"controller": "nfdeployment", "controllerGroup": "workload.nephio.org", "controllerKind": "NFDeployment", "source": "kind source: *v1.ConfigMap"}
2024-02-01T23:37:09Z    INFO    Starting Controller     {"controller": "nfdeployment", "controllerGroup": "workload.nephio.org", "controllerKind": "NFDeployment"}
2024-02-01T23:37:09Z    INFO    Starting workers        {"controller": "nfdeployment", "controllerGroup": "workload.nephio.org", "controllerKind": "NFDeployment", "worker count": 1}
2024-02-01T23:51:19Z    INFO    Creating ConfigMap      {"controller": "nfdeployment", "controllerGroup": "workload.nephio.org", "controllerKind": "NFDeployment", "NFDeployment": {"name":"upf-edge02","namespace":"free5gc-upf"}, "namespace": "free5gc-upf", "name": "upf-edge02", "reconcileID": "63574471-2aec-40d5-b602-e8ee0a23a740", "NFDeployment": {"name":"upf-edge02","namespace":"free5gc-upf"}, "NF": "UPF", "ConfigMap.namespace": "free5gc-upf", "ConfigMap.name": "upf-edge02"}
2024-02-01T23:51:19Z    INFO    Creating Deployment     {"controller": "nfdeployment", "controllerGroup": "workload.nephio.org", "controllerKind": "NFDeployment", "NFDeployment": {"name":"upf-edge02","namespace":"free5gc-upf"}, "namespace": "free5gc-upf", "name": "upf-edge02", "reconcileID": "63574471-2aec-40d5-b602-e8ee0a23a740", "NFDeployment": {"name":"upf-edge02","namespace":"free5gc-upf"}, "NF": "UPF", "Deployment.namespace": "free5gc-upf", "Deployment.name": "upf-edge02"}
2024-02-01T23:51:19Z    INFO    Creating Deployment     {"controller": "nfdeployment", "controllerGroup": "workload.nephio.org", "controllerKind": "NFDeployment", "NFDeployment": {"name":"upf-edge02","namespace":"free5gc-upf"}, "namespace": "free5gc-upf", "name": "upf-edge02", "reconcileID": "4204b900-cdd5-48ad-93e7-7b3794e804d3", "NFDeployment": {"name":"upf-edge02","namespace":"free5gc-upf"}, "NF": "UPF", "Deployment.namespace": "free5gc-upf", "Deployment.name": "upf-edge02"}
2024-02-01T23:51:19Z    ERROR   Failed to create new Deployment {"controller": "nfdeployment", "controllerGroup": "workload.nephio.org", "controllerKind": "NFDeployment", "NFDeployment": {"name":"upf-edge02","namespace":"free5gc-upf"}, "namespace": "free5gc-upf", "name": "upf-edge02", "reconcileID": "4204b900-cdd5-48ad-93e7-7b3794e804d3", "NFDeployment": {"name":"upf-edge02","namespace":"free5gc-upf"}, "NF": "UPF", "Deployment.namespace": "free5gc-upf", "Deployment.name": "upf-edge02", "error": "deployments.apps \"upf-edge02\" already exists"}
github.com/nephio-project/free5gc/controllers/nf/upf.(*UPFDeploymentReconciler).Reconcile
        /workspace/controllers/nf/upf/reconciler.go:140
github.com/nephio-project/free5gc/controllers/nf.(*NFDeploymentReconciler).Reconcile
        /workspace/controllers/nf/reconciler.go:99
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:118
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:314
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:265
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:226

[radoslawc]

/override build-push-image-vlan-fn-commit-conf

[nephio-prow]

@radoslawc: radoslawc unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file.

In response to this:

/override build-push-image-vlan-fn-commit-conf

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[tliron]

/override build-push-image-vlan-fn-commit-conf

[nephio-prow]

@tliron: Overrode contexts on behalf of tliron: build-push-image-vlan-fn-commit-conf

In response to this:

/override build-push-image-vlan-fn-commit-conf

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[tliron]

/lgtm
/approve

[nephio-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tliron

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [tliron]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment