search

confidential-containers / operator

Operator to deploy confidential containers runtime
Apache License 2.0
102 stars 56 forks source link

Include peerpod specific crd deployments via operator #349

Closed bpradipt closed 4 months ago

bpradipt commented 4 months ago

This includes peerpod and peerpodconfig controllers This is based on @caavery work. I just rebased and adapted the code to align with latest code base

bpradipt commented 4 months ago

LGTM but one question about which version to use. We could use 0.8.0, right?

We can use 0.8.0.

bpradipt commented 4 months ago

@mythi looks like you'll need to explicitly approve the PR for the approval to register

mythi commented 4 months ago

@mythi looks like you'll need to explicitly approve the PR for the approval to register

another approval is needed still. perhaps @fidencio can take a look and judge if the failing tests are acceptable

stevenhorsman commented 4 months ago

The tests passed in the last nightly https://github.com/confidential-containers/operator/actions/runs/7838705386, so I'll re-queue in case that helps