Skip to content

Allow to pull/push artifacts to insecure registries without TLS #5299

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

Merged
merged 1 commit into from
Apr 11, 2025

Conversation

mottetm
Copy link

@mottetm mottetm commented Apr 11, 2025

This PR relates to the following discussion.

If merged, this PR will introduce a new --insecure-registry flag to the following commands: push artifacts, pull artifact, diff artifact and list artifacts. When used the flag will lead to the option crane.Insecure being passed to the crane client allowing the use of insecure repositories.

If applied, this commit will introduce a new `--insecure-repository`
flag to the following commands: `push artifacts`, `pull artifact`,
`diff artifact` and `list artifacts`. When used the flag will lead to
the option `crane.Insecure` being passed to the `crane` client allowing
the use of insecure repositories.

Signed-off-by: Matthieu Mottet <m.mottet@outlook.com>
@stefanprodan
Copy link
Member

Can you please update the PR description and correct the flag name it’s insecure-registry

@mottetm
Copy link
Author

mottetm commented Apr 11, 2025

fixed 🤦

@stefanprodan stefanprodan added the area/oci OCI related issues and pull requests label Apr 11, 2025
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @mottetm

@stefanprodan stefanprodan changed the title Allow to pull/push artifacts without TLS Allow to pull/push artifacts to insecure registries without TLS Apr 11, 2025
@stefanprodan stefanprodan merged commit c436708 into fluxcd:main Apr 11, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/oci OCI related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants