common icon indicating copy to clipboard operation
common copied to clipboard

Add fields for TLS material to destination config

Open meln5674 opened this issue 1 year ago • 5 comments

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 .

meln5674 avatar Nov 18 '24 23:11 meln5674

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.

rhatdan avatar Nov 19 '24 11:11 rhatdan

@Luap99 PTAL /approve

rhatdan avatar Dec 05 '24 19:12 rhatdan

[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

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci[bot] avatar Dec 05 '24 19:12 openshift-ci[bot]

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.

meln5674 avatar Dec 28 '24 20:12 meln5674

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

Luap99 avatar Jan 07 '25 16:01 Luap99

Needs a rebase as well. Before we merge, let's test everything in the podman PR too.

ashley-cui avatar Jul 31 '25 15:07 ashley-cui

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!

jankaluza avatar Aug 26 '25 14:08 jankaluza