-
Notifications
You must be signed in to change notification settings - Fork 365
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add tools_path variable to kubectl-helm-minikube feature #981
base: main
Are you sure you want to change the base?
Conversation
@microsoft-github-policy-service agree |
@microsoft-github-policy-service agree
… On May 25, 2024, at 2:27 PM, microsoft-github-policy-service[bot] ***@***.***> wrote:
@robinmordasiewicz <https://github.com/robinmordasiewicz> please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]
Options:
(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"
Contributor License Agreement
<http://www.opensource.org/>
—
Reply to this email directly, view it on GitHub <#981 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG2O2ODYCPYSK653TEYCCDTZEDJXZAVCNFSM6AAAAABIJCZZRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZRGM4DKMJRGI>.
You are receiving this because you were mentioned.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Creating a new Feature option makes sense to me, thanks for taking the time in contributing this PR.
Left some comments!
src/kubectl-helm-minikube/install.sh
Outdated
@@ -15,6 +15,7 @@ rm -rf /var/lib/apt/lists/* | |||
KUBECTL_VERSION="${VERSION:-"latest"}" | |||
HELM_VERSION="${HELM:-"latest"}" | |||
MINIKUBE_VERSION="${MINIKUBE:-"latest"}" # latest is also valid | |||
TOOLS_PATH="${TOOLS_PATH:-"/usr/local/bin"}" # latest is also valid |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we create a test scenario to validate this new Feature option? thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add some tests to validate this change?
Co-authored-by: Samruddhi Khandale <[email protected]>
Co-authored-by: Samruddhi Khandale <[email protected]>
Co-authored-by: Samruddhi Khandale <[email protected]>
Co-authored-by: Samruddhi Khandale <[email protected]>
I do not know how to add tests yet.
… On Jun 4, 2024, at 2:37 PM, Samruddhi Khandale ***@***.***> wrote:
@samruddhikhandale commented on this pull request.
In src/kubectl-helm-minikube/install.sh <#981 (comment)>:
> @@ -15,6 +15,7 @@ rm -rf /var/lib/apt/lists/*
KUBECTL_VERSION="${VERSION:-"latest"}"
HELM_VERSION="${HELM:-"latest"}"
MINIKUBE_VERSION="${MINIKUBE:-"latest"}" # latest is also valid
+TOOLS_PATH="${TOOLS_PATH:-"/usr/local/bin"}" # latest is also valid
Can we add some tests to validate this change?
—
Reply to this email directly, view it on GitHub <#981 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG2O2OCTGQY7LEBHLH4MYT3ZFYCPFAVCNFSM6AAAAABIJCZZRKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAOJXGE2DIMZUG4>.
You are receiving this because you were mentioned.
|
Some tools and extensions expect to find the tools in a particular PATH that is not /usr/local/bin