Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include peerpod specific crd deployments via operator #349

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

bpradipt
Copy link
Member

@bpradipt bpradipt commented Feb 9, 2024

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 bpradipt requested review from mythi, jensfr and stevenhorsman and removed request for mythi February 9, 2024 05:58
go.mod Outdated Show resolved Hide resolved
Copy link
Contributor

@mythi mythi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@bpradipt
Copy link
Member Author

bpradipt commented Feb 9, 2024

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

We can use 0.8.0.

This includes peerpod and peerpodconfig controllers

Signed-off-by: Cathy Avery <[email protected]>
Signed-off-by: Pradipta Banerjee <[email protected]>
@bpradipt
Copy link
Member Author

bpradipt commented Feb 9, 2024

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

@mythi
Copy link
Contributor

mythi commented Feb 9, 2024

@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
Copy link
Member

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

Copy link
Member

@jensfr jensfr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still looks good to me. Thanks Cathy and Pradipta!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants