treewide: replace bool-to-boolptr helpers with k8s.io/utils/ptr.To
Change-Id: I90419ddfe087291f41f7f2f3589263e56c15470a
Reviewed-on: https://review.monogon.dev/c/monogon/+/3675
Tested-by: Jenkins CI
Reviewed-by: Lorenz Brun <lorenz@monogon.tech>
diff --git a/metropolis/cli/metroctl/cmd_node_set.go b/metropolis/cli/metroctl/cmd_node_set.go
index bdba8a9..4211aca 100644
--- a/metropolis/cli/metroctl/cmd_node_set.go
+++ b/metropolis/cli/metroctl/cmd_node_set.go
@@ -9,6 +9,7 @@
"strings"
"github.com/spf13/cobra"
+ "k8s.io/utils/ptr"
"source.monogon.dev/metropolis/proto/api"
)
@@ -39,7 +40,6 @@
role := strings.ToLower(args[0])
nodes := args[1:]
- opt := func(v bool) *bool { return &v }
for _, node := range nodes {
req := &api.UpdateNodeRolesRequest{
Node: &api.UpdateNodeRolesRequest_Id{
@@ -48,11 +48,11 @@
}
switch role {
case "kubernetescontroller", "kc":
- req.KubernetesController = opt(true)
+ req.KubernetesController = ptr.To(true)
case "kubernetesworker", "kw":
- req.KubernetesWorker = opt(true)
+ req.KubernetesWorker = ptr.To(true)
case "consensusmember", "cm":
- req.ConsensusMember = opt(true)
+ req.ConsensusMember = ptr.To(true)
default:
return fmt.Errorf("unknown role: %s", role)
}
@@ -84,7 +84,6 @@
role := strings.ToLower(args[0])
nodes := args[1:]
- opt := func(v bool) *bool { return &v }
for _, node := range nodes {
req := &api.UpdateNodeRolesRequest{
Node: &api.UpdateNodeRolesRequest_Id{
@@ -94,11 +93,11 @@
switch role {
case "kubernetescontroller", "kc":
- req.KubernetesController = opt(false)
+ req.KubernetesController = ptr.To(false)
case "kubernetesworker", "kw":
- req.KubernetesWorker = opt(false)
+ req.KubernetesWorker = ptr.To(false)
case "consensusmember", "cm":
- req.ConsensusMember = opt(false)
+ req.ConsensusMember = ptr.To(false)
default:
return fmt.Errorf("unknown role: %s. Must be one of: KubernetesController, KubernetesWorker, ConsensusMember", role)
}