fix(kubernetes): modify cluster description (#257)

villevsv-upcloud · web-flow · commit ffc584267674 · 2023-08-17T11:01:07.000+03:00
diff --git a/upcloud/service/kubernetes.go b/upcloud/service/kubernetes.go
@@ -58,7 +58,7 @@ func (s *Service) CreateKubernetesCluster(ctx context.Context, r *request.Create
 	return &cluster, s.create(ctx, r, &cluster)
 }
 
-// ModifyKubernetesCluster modifies an existing node group.
+// ModifyKubernetesCluster modifies an existing Kubernetes cluster.
 func (s *Service) ModifyKubernetesCluster(ctx context.Context, r *request.ModifyKubernetesClusterRequest) (*upcloud.KubernetesCluster, error) {
 	cluster := upcloud.KubernetesCluster{}
 	return &cluster, s.modify(ctx, r, &cluster)
diff --git a/upcloud/service/kubernetes_test.go b/upcloud/service/kubernetes_test.go
@@ -17,12 +17,10 @@ import (
 
 const exampleClusterResponse = `
 {
+	"control_plane_ip_filter":null,
 	"name":"test-name",
 	"network":"03e4970d-7791-4b80-a892-682ae0faf46b",
 	"network_cidr":"176.16.1.0/24",
-	"state":"running",
-	"uuid":"0dca7a18-98e1-4e2d-aea5-ef5dd5fa450e",
-	"zone":"de-fra1",
 	"node_groups":[
 		{
 			"name":"my-group1",
@@ -65,7 +63,10 @@ const exampleClusterResponse = `
 			],
 			"utility_network_access": false
 		}
-	]
+	],
+	"state":"running",
+	"uuid":"0dca7a18-98e1-4e2d-aea5-ef5dd5fa450e",
+	"zone":"de-fra1"
 }
 `
 

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ func (s Service) CreateKubernetesCluster(ctx context.Context, r request.Create`
`58`	`58`	`return &cluster, s.create(ctx, r, &cluster)`
`59`	`59`	`}`
`60`	`60`
`61`		`-// ModifyKubernetesCluster modifies an existing node group.`
	`61`	`+// ModifyKubernetesCluster modifies an existing Kubernetes cluster.`
`62`	`62`	`func (s Service) ModifyKubernetesCluster(ctx context.Context, r request.ModifyKubernetesClusterRequest) (*upcloud.KubernetesCluster, error) {`
`63`	`63`	`cluster := upcloud.KubernetesCluster{}`
`64`	`64`	`return &cluster, s.modify(ctx, r, &cluster)`
Original file line number	Diff line number	Diff line change
`@@ -17,12 +17,10 @@ import (`
`17`	`17`
`18`	`18`	const exampleClusterResponse = `
`19`	`19`	`{`
	`20`	`+ "control_plane_ip_filter":null,`
`20`	`21`	`"name":"test-name",`
`21`	`22`	`"network":"03e4970d-7791-4b80-a892-682ae0faf46b",`
`22`	`23`	`"network_cidr":"176.16.1.0/24",`
`23`		`- "state":"running",`
`24`		`- "uuid":"0dca7a18-98e1-4e2d-aea5-ef5dd5fa450e",`
`25`		`- "zone":"de-fra1",`
`26`	`24`	`"node_groups":[`
`27`	`25`	`{`
`28`	`26`	`"name":"my-group1",`
@@ -65,7 +63,10 @@ const exampleClusterResponse = `
`65`	`63`	`],`
`66`	`64`	`"utility_network_access": false`
`67`	`65`	`}`
`68`		`- ]`
	`66`	`+ ],`
	`67`	`+ "state":"running",`
	`68`	`+ "uuid":"0dca7a18-98e1-4e2d-aea5-ef5dd5fa450e",`
	`69`	`+ "zone":"de-fra1"`
`69`	`70`	`}`
`70`	`71`	`
`71`	`72`