feat: Certificate Authorities to wasmcloud host #72

lxfontes · 2024-07-11T20:57:34Z

what

Fixes #69

Allow passing one or more certificate authorities into wasmcloud via configmaps & secrets.

Example authorities:

apiVersion: v1
kind: ConfigMap
metadata:
  name: org-authorities
data:
  dontmountme: |
    -----BEGIN CERTIFICATE-----
    Custom CA certificate bundle.
    -----END CERTIFICATE-----
  root.crt: |
    -----BEGIN CERTIFICATE-----
    Custom CA certificate bundle.
    -----END CERTIFICATE-----

---
apiVersion: v1
kind: Secret
metadata:
  name: email-authorities
stringData:
  email.crt: |
    -----BEGIN CERTIFICATE-----
    Custom CA certificate bundle.
    -----END CERTIFICATE-----
  notifications.crt: |
    -----BEGIN CERTIFICATE-----
    Custom CA certificate bundle.
    -----END CERTIFICATE-----

how

We pass configmaps as part of the wasmcloud host definition:

apiVersion: k8s.wasmcloud.dev/v1alpha1
kind: WasmCloudHostConfig
metadata:
  name: wasmcloud-host
spec:
  lattice: default
  version: "1.0.4"
  natsAddress: nats://nats-cluster.default.svc.cluster.local
  certificates:
    authorities:
      - name: org-wide-authorities
         configmap:
           name: org-authorities
      - name: partner-authorities
         secret:
           secretName: email-authorities

The authorities list follows the Volume convention as defined in https://pkg.go.dev/k8s.io/api/core/v1#VolumeSource

The operator will mount the org-authorities configmap into the wasmcloud host container under /wasmcloud/certificates/ca-org-wide-authorities. Notice we prefix the authority name with ca-.
The operator then scans the configmap for items that end in .crt ( certificate ) and append them to the arguments passed to wasmcloud host. Ex: if the configmap has the keys dontmountme and root.crt, only the root.crt will be passed into wasmcloud as --tls-ca-path /wasmcloud/certificates/ca-org-wide-authorities/root.crt.

The same applies to Secrets.

We also raise a reconciliation error in case the desired object is not defined or not found.

Signed-off-by: Lucas Fontes <[email protected]>

crates/types/src/v1alpha1/wasmcloud_host_config.rs

src/controller.rs

crates/types/src/v1alpha1/wasmcloud_host_config.rs

src/controller.rs

Signed-off-by: Lucas Fontes <[email protected]>

src/controller.rs

joonas · 2024-07-15T13:48:36Z

I'm not entirely sure I understand the purpose of optional, would you be able to elaborate what it's intended for?

lxfontes · 2024-07-15T14:06:07Z

I'm not entirely sure I understand the purpose of optional, would you be able to elaborate what it's intended for?

Similar to optional in env mapping & volume mounting: It blocks reconciliation in case the desired object doesnt exist. The default behaviour is to yield a blank volume.

[edit]
thinking a bit more about it and can't think of a situation where we would want it to be optional=true.
lets chat cause this relates to relying on upstream structs or not ( https://pkg.go.dev/k8s.io/api/core/v1#ConfigMapVolumeSource ) as they always come with optional and defaultMode

Signed-off-by: Lucas Fontes <[email protected]>

lxfontes · 2024-07-15T19:46:00Z

src/controller.rs

+    {
+        for authority in authorities.iter() {
+            let authority_name = authority.name.clone();
+            let volume_name = format!("ca-{authority_name}");


the resulting volume name

lxfontes · 2024-07-15T19:46:49Z

src/controller.rs

+                let secret_name = match &secret_ref.secret_name {
+                    Some(s) => s,
+                    None => {
+                        return Err(Error::CertificateError(format!(


when secret.secretName is not present. similar check for configmap.

lxfontes · 2024-07-15T19:47:35Z

src/lib.rs

@@ -31,6 +31,9 @@ pub enum Error {
    #[error("Error retrieving secrets: {0}")]
    SecretError(String),

+    #[error("Certificate error: {0}")]


specific error

lxfontes · 2024-07-15T19:47:48Z

crates/types/src/v1alpha1/wasmcloud_host_config.rs

@@ -187,6 +189,11 @@ fn default_nats_leafnode_port() -> u16 {
    7422
 }

+#[derive(Deserialize, Serialize, Clone, Debug, JsonSchema)]
+pub struct WasmCloudHostCertificates {
+    pub authorities: Option<Vec<Volume>>,


using upstream structs.

src/controller.rs

Signed-off-by: Lucas Fontes <[email protected]>

feat: Customize Certificate Authorities

0b036e2

Signed-off-by: Lucas Fontes <[email protected]>

lxfontes force-pushed the lxfontes/onboarding-certificates branch from ec19510 to 0b036e2 Compare July 11, 2024 20:58

lxfontes commented Jul 11, 2024

View reviewed changes

crates/types/src/v1alpha1/wasmcloud_host_config.rs Outdated Show resolved Hide resolved

lxfontes commented Jul 11, 2024

View reviewed changes

crates/types/src/v1alpha1/wasmcloud_host_config.rs Outdated Show resolved Hide resolved

lxfontes commented Jul 11, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

joonas reviewed Jul 11, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

protochron reviewed Jul 12, 2024

View reviewed changes

joonas reviewed Jul 12, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

joonas reviewed Jul 12, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

joonas reviewed Jul 12, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

joonas reviewed Jul 12, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

joonas reviewed Jul 12, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

joonas reviewed Jul 12, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

joonas reviewed Jul 12, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

lxfontes added 4 commits July 12, 2024 17:15

handling 'optional'

61f3554

Signed-off-by: Lucas Fontes <[email protected]>

falling back to default if needed

ffec6b8

Signed-off-by: Lucas Fontes <[email protected]>

changing function signature

cbbfa29

Signed-off-by: Lucas Fontes <[email protected]>

better 'format'

6cbb314

Signed-off-by: Lucas Fontes <[email protected]>

lxfontes marked this pull request as ready for review July 12, 2024 20:31

protochron approved these changes Jul 15, 2024

View reviewed changes

joonas reviewed Jul 15, 2024

View reviewed changes

src/controller.rs Outdated Show resolved Hide resolved

lxfontes added 2 commits July 15, 2024 16:35

Using upstream structs

e275b4f

Signed-off-by: Lucas Fontes <[email protected]>

addressing nits

f6af671

Signed-off-by: Lucas Fontes <[email protected]>

lxfontes commented Jul 15, 2024

View reviewed changes

joonas reviewed Jul 15, 2024

View reviewed changes

src/controller.rs Show resolved Hide resolved

checking for valid types

65bbf5f

Signed-off-by: Lucas Fontes <[email protected]>

joonas merged commit 97f0c12 into wasmCloud:main Jul 17, 2024
5 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Certificate Authorities to wasmcloud host #72

feat: Certificate Authorities to wasmcloud host #72

lxfontes commented Jul 11, 2024 •

edited

Loading

joonas commented Jul 15, 2024 •

edited

Loading

lxfontes commented Jul 15, 2024 •

edited

Loading

lxfontes Jul 15, 2024

lxfontes Jul 15, 2024 •

edited

Loading

lxfontes Jul 15, 2024

lxfontes Jul 15, 2024

feat: Certificate Authorities to wasmcloud host #72

feat: Certificate Authorities to wasmcloud host #72

Conversation

lxfontes commented Jul 11, 2024 • edited Loading

what

how

joonas commented Jul 15, 2024 • edited Loading

lxfontes commented Jul 15, 2024 • edited Loading

lxfontes Jul 15, 2024

Choose a reason for hiding this comment

lxfontes Jul 15, 2024 • edited Loading

Choose a reason for hiding this comment

lxfontes Jul 15, 2024

Choose a reason for hiding this comment

lxfontes Jul 15, 2024

Choose a reason for hiding this comment

lxfontes commented Jul 11, 2024 •

edited

Loading

joonas commented Jul 15, 2024 •

edited

Loading

lxfontes commented Jul 15, 2024 •

edited

Loading

lxfontes Jul 15, 2024 •

edited

Loading