Replies: 5 comments
-
I am not sure this can work with The option to use is a combo of destroy/deploy or introduce a Another option worth exploring is to create a config snippet that contains cert information and is mounted to srl nodes. That way srl should use the config with an extra tls profile that it will use for gnmi and salt nodes would have access to key material. But maybe the first question is why salt can't work with |
Beta Was this translation helpful? Give feedback.
-
It's a Salt Nornir proxy, I'm trying to get it to make gNMI calls from a Napalm driver that doesn't support --skip-verify |
Beta Was this translation helpful? Give feedback.
-
I haven't looked in detail, but I suspect the issue is a timing issue: Certificates get generated during pre-deploy, for each container separately (in parallel): https://github.com/srl-labs/containerlab/blob/main/clab/clab.go#L219 If we would change that to a 2-step approach, i.e. first generate certificates for all nodes and then create them, it would work |
Beta Was this translation helpful? Give feedback.
-
It might still be reasonable to add Secondly, you don't really need keys to make TLS work, you only need a ca.cert file to verify the server keys. Unless you must use mutual verification, of course. So I think there might be an easy way out of this by following the same approach we did for ansible inventory file. So the potential flow is:
|
Beta Was this translation helpful? Give feedback.
-
I'm just thinking that there are cases where it can be useful to have access to generated certificate files, by bind-mounting them into the containers. The easiest, most generic solution is to first generate all certificates, and then start the containers (instead of generating them as part of the container startup process) |
Beta Was this translation helpful? Give feedback.
-
Use case: A setup with a Salt Master/Proxy container and 2 SRL nodes. The proxy needs access to the certificates generated for the SRL containers (gNMI)
It works when doing 'destroy' followed by 'deploy', but '--reconfigure' messes things up
Beta Was this translation helpful? Give feedback.
All reactions