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

Enhancement proposal : Helm repo add to support --insecure-skip-verify #3113

Closed
pramod0310 opened this issue Feb 12, 2020 · 5 comments · Fixed by #3514
Closed

Enhancement proposal : Helm repo add to support --insecure-skip-verify #3113

pramod0310 opened this issue Feb 12, 2020 · 5 comments · Fixed by #3514
Assignees
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@pramod0310
Copy link

Summary

Adding helm repo does not support insecure options, as supported while adding git repository.

Motivation

Many of the self-hosted helm repositories like harbor etc would be using self-signed certs. It would be a helpful option.

Proposal

The addition of helm repo should also support the insecure-skip-verify option to skip cert verification.

@alexmt
Copy link
Collaborator

alexmt commented Mar 4, 2020

This is already supported in CLI: argocd ... --insecure-skip-server-verification

The checkbox is missing in UI.

@alexmt alexmt added good first issue Good for newcomers help wanted Extra attention is needed ui labels Mar 4, 2020
@pramod0310
Copy link
Author

@alexmt I tried using --insecure-skip-server-verification for helm charts but does not have any effect only for git it works fine.

@thirdeyenick
Copy link

This is because the insecure option is not passed to the Helm client here. It is only passed if the type of the repo is 'git'.

@mayzhang2000 mayzhang2000 self-assigned this Apr 14, 2020
@alexmt
Copy link
Collaborator

alexmt commented Apr 15, 2020

PR #3420 partially addresses this issue. However, helm CLI itself does not have --insecure flag. So above PR will allow adding repository but it still will be impossible to use charts from the repo (helm repo add will fail) .

The --insecure-skip-tls-verify flag was added to helm last month but still not released (checked both helm v2.16.6 and v3.1.2).

Lets keep ticket open and wait for helm release.

@alexmt
Copy link
Collaborator

alexmt commented Apr 27, 2020

@mayzhang2000 lets keep it open until we upgrade to the helm 3.20 which adds --insecure-skip-tls-verify flag support. The helm release was published last week so we can work on it now: https://github.com/helm/helm/releases/tag/v3.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants