Skip to content

Commit

Permalink
Add stub Cluster ID support to CCM
Browse files Browse the repository at this point in the history
Resolves warning 2 from k3s-io#2471.

As per kubernetes/cloud-provider#12 the
ClusterID requirement was never really followed through on, so the
flag is probably going to be removed in the future along with the
warning it currently triggers.

Signed-off-by: Brad Davidson <[email protected]>
  • Loading branch information
brandond committed Nov 4, 2020
1 parent 35c44fd commit 9559011
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion pkg/cloudprovider/cloudprovider.go
Original file line number Diff line number Diff line change
Expand Up @@ -58,5 +58,5 @@ func (k *k3s) ProviderName() string {
}

func (k *k3s) HasClusterID() bool {
return false
return true
}
1 change: 0 additions & 1 deletion pkg/daemons/control/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -920,7 +920,6 @@ func cloudControllerManager(ctx context.Context, cfg *config.Control, runtime *c
"bind-address": localhostIP.String(),
"secure-port": "0",
"cloud-provider": version.Program,
"allow-untagged-cloud": "true",
"node-status-update-frequency": "1m",
"profiling": "false",
}
Expand Down

0 comments on commit 9559011

Please sign in to comment.