Skip to content

Commit 0e3ef75

Browse files
authored
move singletenantcontroller to nested kubecontroller package and rename (Azure#1493)
move singletenantcontroller to kubecontroller package and rename Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
1 parent b58dc70 commit 0e3ef75

File tree

7 files changed

+11
-11
lines changed

7 files changed

+11
-11
lines changed

cns/singletenantcontroller/conversion.go renamed to cns/kubecontroller/nodenetworkconfig/conversion.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package kubecontroller
1+
package nodenetworkconfig
22

33
import (
44
"net"

cns/singletenantcontroller/conversion_test.go renamed to cns/kubecontroller/nodenetworkconfig/conversion_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package kubecontroller
1+
package nodenetworkconfig
22

33
import (
44
"strconv"

cns/singletenantcontroller/metrics.go renamed to cns/kubecontroller/nodenetworkconfig/metrics.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package kubecontroller
1+
package nodenetworkconfig
22

33
import (
44
"github.com/prometheus/client_golang/prometheus"

cns/singletenantcontroller/reconciler.go renamed to cns/kubecontroller/nodenetworkconfig/reconciler.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package kubecontroller
1+
package nodenetworkconfig
22

33
import (
44
"context"

cns/singletenantcontroller/reconciler_test.go renamed to cns/kubecontroller/nodenetworkconfig/reconciler_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package kubecontroller
1+
package nodenetworkconfig
22

33
import (
44
"context"

cns/singletenantcontroller/scopedclient.go renamed to cns/kubecontroller/nodenetworkconfig/scopedclient.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package kubecontroller
1+
package nodenetworkconfig
22

33
import (
44
"context"

cns/service/main.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,12 @@ import (
2929
"github.com/Azure/azure-container-networking/cns/healthserver"
3030
"github.com/Azure/azure-container-networking/cns/hnsclient"
3131
"github.com/Azure/azure-container-networking/cns/ipampool"
32+
nncctrl "github.com/Azure/azure-container-networking/cns/kubecontroller/nodenetworkconfig"
3233
"github.com/Azure/azure-container-networking/cns/logger"
3334
"github.com/Azure/azure-container-networking/cns/multitenantcontroller"
3435
"github.com/Azure/azure-container-networking/cns/multitenantcontroller/multitenantoperator"
3536
"github.com/Azure/azure-container-networking/cns/nmagent"
3637
"github.com/Azure/azure-container-networking/cns/restserver"
37-
kubecontroller "github.com/Azure/azure-container-networking/cns/singletenantcontroller"
3838
cnstypes "github.com/Azure/azure-container-networking/cns/types"
3939
"github.com/Azure/azure-container-networking/cns/wireserver"
4040
acn "github.com/Azure/azure-container-networking/common"
@@ -910,9 +910,9 @@ func reconcileInitialCNSState(ctx context.Context, cli nodeNetworkConfigGetter,
910910

911911
switch nnc.Status.NetworkContainers[i].AssignmentMode { //nolint:exhaustive // skipping dynamic case
912912
case v1alpha.Static:
913-
ncRequest, err = kubecontroller.CreateNCRequestFromStaticNC(nnc.Status.NetworkContainers[i])
913+
ncRequest, err = nncctrl.CreateNCRequestFromStaticNC(nnc.Status.NetworkContainers[i])
914914
default: // For backward compatibility, default will be treated as Dynamic too.
915-
ncRequest, err = kubecontroller.CreateNCRequestFromDynamicNC(nnc.Status.NetworkContainers[i])
915+
ncRequest, err = nncctrl.CreateNCRequestFromDynamicNC(nnc.Status.NetworkContainers[i])
916916
}
917917

918918
if err != nil {
@@ -999,7 +999,7 @@ func InitializeCRDState(ctx context.Context, httpRestService cns.HTTPService, cn
999999
return errors.Wrap(err, "failed to create NNC client")
10001000
}
10011001
// TODO(rbtr): nodename and namespace should be in the cns config
1002-
scopedcli := kubecontroller.NewScopedClient(nnccli, types.NamespacedName{Namespace: "kube-system", Name: nodeName})
1002+
scopedcli := nncctrl.NewScopedClient(nnccli, types.NamespacedName{Namespace: "kube-system", Name: nodeName})
10031003

10041004
// initialize the ipam pool monitor
10051005
poolOpts := ipampool.Options{
@@ -1076,7 +1076,7 @@ func InitializeCRDState(ctx context.Context, httpRestService cns.HTTPService, cn
10761076
return errors.Wrapf(err, "failed to get node %s", nodeName)
10771077
}
10781078

1079-
reconciler := kubecontroller.NewReconciler(httpRestServiceImplementation, nnccli, poolMonitor)
1079+
reconciler := nncctrl.NewReconciler(httpRestServiceImplementation, nnccli, poolMonitor)
10801080
// pass Node to the Reconciler for Controller xref
10811081
if err := reconciler.SetupWithManager(manager, node); err != nil {
10821082
return errors.Wrapf(err, "failed to setup reconciler with manager")

0 commit comments

Comments
 (0)