Skip to content
This repository has been archived by the owner on Jun 23, 2020. It is now read-only.

Added override mechanism for kubeconfig. This allows the full path to… #154

Merged
merged 3 commits into from
Oct 15, 2018

Conversation

templecloud
Copy link
Contributor

Allows the kubeconfig to be overridden via the 'OCI_FLEXD_KUBECONFIG_PATH' environment variable. Otherwise the default behaviour is unchanged.

… the kubeconfig to be overidden via an environment varaible. Otherwise the default behaviour is unchanged.
@owainlewis owainlewis self-assigned this Oct 12, 2018
@owainlewis owainlewis self-requested a review October 12, 2018 12:41
Copy link
Member

@owainlewis owainlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -96,6 +96,18 @@ func GetConfigPath() string {
return filepath.Join(path, "config.yaml")
}

// GetKubeconfigPath gets the override path of the 'kubeconfig'. This override
// can be uses to explicitly set the name and location of the kubeconfig file
// via the OCI_FLEXD_KUBECONFIG environment variable. If this value is not
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OCI_FLEXD_KUBECONFIG => OCI_FLEXD_KUBECONFIG_PATH

@owainlewis owainlewis merged commit 14700ca into master Oct 15, 2018
@owainlewis owainlewis deleted the trjl/OKE-2409 branch October 15, 2018 09:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants