Skip to content

Commit 8b3bb2a

Browse files
pranavsriram8l-technicore
authored andcommitted
fix UT to add instance id to instanceCache
1 parent 786617b commit 8b3bb2a

File tree

2 files changed

+3
-70
lines changed

2 files changed

+3
-70
lines changed

pkg/cloudprovider/providers/oci/instances_test.go

Lines changed: 2 additions & 69 deletions
Original file line numberDiff line numberDiff line change
@@ -129,42 +129,7 @@ var (
129129
HostnameLabel: common.String("no-vcn-dns-label"),
130130
SubnetId: common.String("subnetwithnovcndnslabel"),
131131
},
132-
"ipv6-instance": {
133-
HostnameLabel: common.String("no-vcn-dns-label"),
134-
SubnetId: common.String("IPv6-subnet"),
135-
Ipv6Addresses: []string{"2001:0db8:85a3:0000:0000:8a2e:0370:7334"},
136-
},
137-
"ipv6-instance-ula": {
138-
HostnameLabel: common.String("no-vcn-dns-label"),
139-
SubnetId: common.String("IPv6-subnet"),
140-
Ipv6Addresses: []string{"fc00:0000:0000:0000:0000:0000:0000:0000"},
141-
},
142-
"ipv6-instance-2": {
143-
PrivateIp: common.String("10.0.0.1"),
144-
PublicIp: common.String("0.0.0.1"),
145-
HostnameLabel: common.String("no-vcn-dns-label"),
146-
SubnetId: common.String("IPv6-subnet"),
147-
Ipv6Addresses: []string{"2001:0db8:85a3:0000:0000:8a2e:0370:idfe"},
148-
},
149-
"instance-id-ipv4-ipv6": {
150-
PrivateIp: common.String("10.0.0.1"),
151-
PublicIp: common.String("0.0.0.1"),
152-
HostnameLabel: common.String("no-vcn-dns-label"),
153-
SubnetId: common.String("IPv4-IPv6-subnet"),
154-
Ipv6Addresses: []string{"2001:0db8:85a3:0000:0000:8a2e:0370:7334"},
155-
},
156-
"instance-id-ipv6": {
157-
HostnameLabel: common.String("no-vcn-dns-label"),
158-
SubnetId: common.String("ipv6-instance"),
159-
Ipv6Addresses: []string{"2001:0db8:85a3:0000:0000:8a2e:0370:7334"},
160-
},
161-
"instance-id-ipv4": {
162-
PrivateIp: common.String("10.0.0.1"),
163-
PublicIp: common.String("0.0.0.1"),
164-
HostnameLabel: common.String("no-vcn-dns-label"),
165-
SubnetId: common.String("subnetwithnovcndnslabel"),
166-
},
167-
"ipv6-gua-ipv4-instance": {
132+
"ocid1.ipv6-gua-ipv4-instance": {
168133
PrivateIp: common.String("10.0.0.1"),
169134
HostnameLabel: common.String("no-vcn-dns-label"),
170135
SubnetId: common.String("ipv6-gua-ipv4-instance"),
@@ -236,6 +201,7 @@ var (
236201
CompartmentId: common.String("instance-id-ipv6"),
237202
},
238203
"ipv6-gua-ipv4-instance": {
204+
Id: common.String("ocid1.ipv6-gua-ipv4-instance"),
239205
CompartmentId: common.String("ipv6-gua-ipv4-instance"),
240206
},
241207
}
@@ -411,39 +377,6 @@ var (
411377
},
412378
},
413379
},
414-
"virtualNodeDefault": {
415-
ObjectMeta: metav1.ObjectMeta{
416-
Annotations: map[string]string{
417-
VirtualNodePoolIdAnnotation: "vnpId",
418-
},
419-
Name: "virtualNodeDefault",
420-
},
421-
Spec: v1.NodeSpec{
422-
ProviderID: "ocid1.virtualnode.oc1.iad.default",
423-
},
424-
},
425-
"virtualNodeZoneTest": {
426-
ObjectMeta: metav1.ObjectMeta{
427-
Annotations: map[string]string{
428-
VirtualNodePoolIdAnnotation: "vnpId",
429-
},
430-
Name: "virtualNodeZoneTest",
431-
},
432-
Spec: v1.NodeSpec{
433-
ProviderID: "ocid1.virtualnode.oc1.iad.zonetest",
434-
},
435-
},
436-
"virtualNodeNonCache": {
437-
ObjectMeta: metav1.ObjectMeta{
438-
Annotations: map[string]string{
439-
VirtualNodePoolIdAnnotation: "vnpId",
440-
},
441-
Name: "virtualNodeNonCache",
442-
},
443-
Spec: v1.NodeSpec{
444-
ProviderID: "ocid1.virtualnode.oc1.iad.noncache",
445-
},
446-
},
447380
"ipv6-node": {
448381
ObjectMeta: metav1.ObjectMeta{
449382
Annotations: map[string]string{

pkg/cloudprovider/providers/oci/load_balancer_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -675,7 +675,7 @@ func TestGetSubnetsForNodes(t *testing.T) {
675675
nodes: []*v1.Node{
676676
{
677677
Spec: v1.NodeSpec{
678-
ProviderID: "ipv6-gua-ipv4-instance",
678+
ProviderID: "ocid1.ipv6-gua-ipv4-instance",
679679
},
680680
ObjectMeta: metav1.ObjectMeta{
681681
Annotations: map[string]string{

0 commit comments

Comments
 (0)