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.

Sign up for GitHub

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

Jump to bottom

Add support for insecure registries in HelmRepositories of Type oci #807

Closed
souleb opened this issue Jun 30, 2022 · 8 comments
Closed

Add support for insecure registries in HelmRepositories of Type oci #807

souleb opened this issue Jun 30, 2022 · 8 comments
Assignees
@makkes @souleb
Labels
area/helm Helm related issues and pull requests area/oci OCI related issues and pull requests enhancement New feature or request

Comments

@souleb
Copy link
Member

souleb commented Jun 30, 2022

The idea is to provide the ability to declare a registry as insecure and to be able to communicate with the http scheme.
@souleb souleb added enhancement New feature or request area/helm Helm related issues and pull requests area/oci OCI related issues and pull requests labels Jun 30, 2022
@souleb souleb mentioned this issue Jun 30, 2022
Open
2 tasks
@gfichtenholt
Copy link

gfichtenholt commented Jun 30, 2022 •
edited
Loading

Could you please implement this for both OCI and regular (non-OCI) HTTP repositories? Thanks in advance

@souleb souleb self-assigned this Jun 30, 2022
@gfichtenholt gfichtenholt mentioned this issue Jul 1, 2022
Closed
@makkes makkes self-assigned this Jul 5, 2022
@makkes makkes moved this to In Progress in Maintainers' Focus Jul 12, 2022
This was referenced Jul 12, 2022
Closed
Closed
@makkes
Copy link
Member

makkes commented Jul 19, 2022

I just noticed that the EKS-A folks are maintaining a Helm fork with some patches: aws/eks-anywhere#2536

There's also a bunch of upstream Helm issues and PRs around this so I'll try to find a way for Helm to settle on a quick way forward.

@gfichtenholt gfichtenholt mentioned this issue Jul 22, 2022
Merged
@runyontr runyontr mentioned this issue Dec 8, 2022
Closed
7 tasks
@runyontr
Copy link
Contributor

runyontr commented Dec 9, 2022

Submitted this MR to Helm to get the options needed for this.

helm/helm#11623

@bburky bburky mentioned this issue Aug 17, 2023
Closed
@Noxsios
Copy link

Noxsios commented Sep 26, 2023

Helm will support plain HTTP registries in its next release due to the merging of helm/helm#12128

@Racer159 Racer159 mentioned this issue Oct 31, 2023
Closed
@shanewhite97
Copy link

Looks like t has been merged now, 3.13.0 has been released. I was wondering if this means that this issue/feature can now progress? Thanks!

@Cavaler
Copy link

Cavaler commented Dec 14, 2023

Seems to be resolved in #1288 with the ticket #957

@github-project-automation github-project-automation bot moved this from In Progress to Since Last Dev Meeting in Maintainers' Focus Dec 14, 2023
@souleb souleb moved this from Since Last Dev Meeting to Done in Maintainers' Focus Feb 22, 2024
@stdnt-xiao stdnt-xiao mentioned this issue Jun 3, 2024
Closed
@uqix
Copy link

uqix commented Jun 6, 2024

insecure: true is specified in HelmRepository, but still got error:

failed to login to OCI registry, server gave HTTP response to HTTPS client

@makkes
Copy link
Member

makkes commented Jun 6, 2024

insecure: true is specified in HelmRepository, but still got error:

failed to login to OCI registry, server gave HTTP response to HTTPS client

Please don't comment on closed issues but create a new one and provide more details (Flux version, HelmRepository spec etc.).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@makkes makkes

@souleb souleb

Labels
area/helm Helm related issues and pull requests area/oci OCI related issues and pull requests enhancement New feature or request
Projects
Maintainers' Focus
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

support plain HTTP OCI registries in HelmRepositories fluxcd/source-controller
9 participants
@makkes @stefanprodan @runyontr @uqix @Cavaler @souleb @Noxsios @gfichtenholt @shanewhite97