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

docs: Describe how to configure insecure registries #580

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

cgwalters
Copy link
Collaborator

@cgwalters cgwalters commented Jun 3, 2024

We don't have a --tls-verify=false flag because:

  • It's a bad idea to make it too easy to do
  • When you do do it, you want it to be persistent/global anyways so a global config file is right

Closes: #461

We don't have a `--tls-verify=false` flag because:

- It's a bad idea to make it too easy to do
- When you *do* do it, you want it to be persistent/global anyways
  so a global config file is righ

Closes: containers#461

Signed-off-by: Colin Walters <walters@verbum.org>
Copy link
Contributor

@prestist prestist left a comment

Choose a reason for hiding this comment

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

Lgtm, maybe a nit on the PR? Im not sure but it looks like the 't' was dropped on your comment for 'righ'

@cgwalters
Copy link
Collaborator Author

Lgtm, maybe a nit on the PR? Im not sure but it looks like the 't' was dropped on your comment for 'righ'

Yeah you're righ, I edited the PR description but didn't force push because it'd rerun CI just for that. Let's just live with my typo for ETERNITY.

@cgwalters cgwalters merged commit 7cdb8de into containers:main Jun 4, 2024
16 checks passed
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.

support insecure registries
2 participants