common
common copied to clipboard
Add fields for TLS material to destination config
Adds optional fields tls_cert_file, tls_key_file, and tls_cafile to the configuration TOML to support connecting to TLS and mTLS podman API sockets.
This is in support of https://github.com/containers/podman/pull/24601 to fix https://github.com/containers/podman/issues/24583 .
If you are going to add more fields, you need to document them in the containers.conf file and the containers.conf.5.md file.
@Luap99 PTAL /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: meln5674, rhatdan
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [rhatdan]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
Also please add some basic test to pkg/config/connections_test.go to ensure the config files can be parsed correctly and the serialization/deserialization works/looks as expected.
Done.
As an aside, it seems pkg/config/modules_test.go is failing due to a sync.Once capturing the wrong XDG_CONFIG_HOME value, but this failure doesn't appear to be reflected in the CI because it doesn't test rootless.
As an aside, it seems pkg/config/modules_test.go is failing due to a sync.Once capturing the wrong XDG_CONFIG_HOME value, but this failure doesn't appear to be reflected in the CI because it doesn't test rootless.
Ah yes you are right. I hit this a while back but then forget about it/ignored it. Maybe good to file an issue for that, these unit tests should work rootless
Needs a rebase as well. Before we merge, let's test everything in the podman PR too.
Ephemeral COPR build failed. @containers/packit-build please check.
Hi, and thank you for your contribution!
We’ve recently migrated this repository into a new monorepo: containers/container-libs along with other repositories
As part of this migration, this repository is no longer accepting new Pull-Requests and therefore this Pull-Request is being closed.
Thank you very much for your contribution. We would appreciate your continued help in migrating this PR to the new container-libs repository. Please let us know if you are facing any issues.
You can read more about the migration and the reasoning behind it in our blog post: Upcoming migration of three containers repositories to monorepo.
Thanks again for your work and for supporting the containers ecosystem!