Skip to content
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

Separate helm install and helm test extra-args flags. #540

Closed
benjdaun opened this issue Mar 27, 2023 · 3 comments
Closed

Separate helm install and helm test extra-args flags. #540

benjdaun opened this issue Mar 27, 2023 · 3 comments
Labels

Comments

@benjdaun
Copy link

benjdaun commented Mar 27, 2023

Is this a request for help?:
Yes

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE REQUEST

Version of Helm and Kubernetes:
Helm: version.BuildInfo{Version:"v3.9.0", GitCommit:"7ceeda6c585217a19a1131663d8cd1f7d641b2a7", GitTreeState:"clean", GoVersion:"go1.17.5"}

Kubectl: Client Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.3", GitCommit:"9e644106593f3f4aa98f8a84b23db5fa378900bd", GitTreeState:"clean", BuildDate:"2023-03-15T13:33:11Z", GoVersion:"go1.19.7", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.3", GitCommit:"434bfd82814af038ad94d62ebe59b133fcb50506", GitTreeState:"clean", BuildDate:"2022-10-25T19:35:11Z", GoVersion:"go1.19.2", Compiler:"gc", Platform:"linux/amd64"}

ct version:
Version: 3.7.1
Git commit: f261a28
Date: 2022-09-27
License: Apache 2.0

What happened:
My command:

ct install <Release-name> --helm-extra-args "--logs" \ 
--helm-dependency-extra-args "--skip-refresh" --charts <chart-path> --namespace <release-namespace>

fails with the error message:

Error: unknown flag: --logs
Error deleting Helm release: failed waiting for process: exit status 1

What you expected to happen:

The ct command uses the --logs flag with the helm test command to print logs from the test pods.

How to reproduce it (as minimally and precisely as possible):

Select a chart to install, and include a --helm-extra-args "--logs" in the ct install command.

Anything else we need to know:

This raises the question of whether the helm-extra-args flag would need to be handled differently, and whether a helm-global-extra-args flag should be considered.

Consideration should also be given to how the logs from the test pods should be split from the default logs that are produced by this command.

@petewall
Copy link

petewall commented Oct 4, 2023

I'd love to see this improvement.

Copy link

github-actions bot commented Nov 4, 2023

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Nov 4, 2023
Copy link

github-actions bot commented Nov 9, 2023

This issue was closed because it has been stalled for 5 days with no activity.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants