ref(cmd/osm-controller): rm unused Azure vars
+ clean up azure references
+ remove TODO line around cobra - don't think we need to
switch to using cobra as this isn't a user facing cli
tool that needs all the functionality cobra provides
@@ -45,7 +45,6 @@ const (
var (
verbosity string
meshName string // An ID that uniquely identifies an OSM instance
- azureAuthFile string
kubeConfigFile string
osmNamespace string
webhookName string
@@ -62,14 +61,12 @@ var (
var (
flags = pflag.NewFlagSet(`osm-controller`, pflag.ExitOnError)
- azureSubscriptionID = flags.String("azure-subscription-id", "", "Azure Subscription ID")
port = flags.Int("port", constants.OSMControllerPort, "Aggregated Discovery Service port number.")
log = logger.New("osm-controller/main")
// What is the Certification Authority to be used
osmCertificateManagerKind = flags.String("certificate-manager", "tresor", "Certificate manager")
- // TODO(draychev): convert all these flags to spf13/cobra: https://github.com/openservicemesh/osm/issues/576
// When certmanager == "vault"
vaultProtocol = flags.String("vault-protocol", "http", "Host name of the Hashi Vault")
vaultHost = flags.String("vault-host", "vault.default.svc.cluster.local", "Host name of the Hashi Vault")
@@ -172,8 +169,6 @@ func main() {
endpointsProviders := []endpoint.Provider{provider}
- // TODO (#88): Add Azure Endpoint provider to list of providers when supported
-
ingressClient, err := ingress.NewIngressClient(kubeClient, namespaceController, stop, cfg)
if err != nil {
log.Fatal().Err(err).Msg("Failed to initialize ingress client")
