Skip to content

Conversation

@verdel
Copy link
Contributor

@verdel verdel commented Feb 19, 2021

No description provided.

@verdel verdel requested a review from a team as a code owner February 19, 2021 13:39
"--repo",
"https://repo.com/helm/charts",
"--version",
"0.0.1",
Copy link
Contributor

@Nitive Nitive Feb 19, 2021

Choose a reason for hiding this comment

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

There is more flags that should be passed to helm fetch

--ca-file string             verify certificates of HTTPS-enabled servers using this CA bundle
--cert-file string           identify HTTPS client using this SSL certificate file
--devel                      use development versions, too. Equivalent to version '>0.0.0-0'. If --version is set, this is ignored.
--insecure-skip-tls-verify   skip tls certificate checks for the chart download
--key-file string            identify HTTPS client using this SSL key file
--keyring string             location of public keys used for verification (default "/Users/nitive/.gnupg/pubring.gpg")
--password string            chart repository password where to locate the requested chart
--repo string                chart repository url where to locate the requested chart
--username string            chart repository username where to locate the requested chart
--verify                     verify the package before using it
--version string             specify the exact chart version to use. If this is not specified, the latest version is used
--repository-cache string    path to the file containing cached repository indexes (default "/Users/nitive/Library/Caches/helm/repository")
--registry-config string     path to the registry config file (default "/Users/nitive/Library/Preferences/helm/registry.json")

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Two last options are global not only for fetch sub-command

--repository-cache string    path to the file containing cached repository indexes (default "/Users/nitive/Library/Caches/helm/repository")
--registry-config string     path to the registry config file (default "/Users/nitive/Library/Preferences/helm/registry.json")

Copy link
Contributor

@Nitive Nitive Feb 19, 2021

Choose a reason for hiding this comment

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

That's right but they must be passed to fetch command otherwise it won't work as expected

$ helm repo add my-repo https://charts.helm.sh/stable --registry-config ./config
"my-repo" has been added to your repositories

$ helm repo update
...Successfully got an update from the "my-repo" chart repository
Update Complete. ⎈Happy Helming!# Error here
$ helm deno template my-app my-repo/mysql --registry-config ./config
Could not find my-repo/mysql
Error: plugin "deno" exited with error

# No error without plugin
$ helm template my-app my-repo/mysql --registry-config ./config
---
# Source: mysql/templates/secrets.yaml
apiVersion: v1
kind: Secret
...

@verdel verdel merged commit dfce91b into main Feb 19, 2021
@verdel verdel deleted the feature-helm-fetch-args branch February 19, 2021 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants