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

This repository needs an integration test CI #9

Open
galmasi opened this issue Jun 8, 2023 · 3 comments
Open

This repository needs an integration test CI #9

galmasi opened this issue Jun 8, 2023 · 3 comments

Comments

@galmasi
Copy link

galmasi commented Jun 8, 2023

Not sure where we can get our hands on a functional kube deployment with GHA. I will investigate.

@sarroutbi
Copy link
Collaborator

My proposal is to use Minikube. There are some deployments that can be done there. An example:
https://github.com/latchset/tang-operator/actions/runs/6264024941/job/17009764176

@mheese
Copy link
Contributor

mheese commented Sep 25, 2023

yeah, I've been thinking about this already as well. The difficulty is around what functionality we want to include with the integration tests. Particularly the operator relies on agents running that must have a TPM, a minikube setup unfortunately might not cut it.

@sarroutbi
Copy link
Collaborator

Sorry, I might not have explained correctly. The approach I propose would be something like:
1 - Install a host machine, and install minikube inside
2 - Start Verifier/Registrar (and any other component which is not the agent) in Minikube through attestation operator
3 - Run agents in the "host machine" (based host machine should support TPM)
4 - "Attack" Verifier/Registrar through Minikube (minikube ip will provide the host to connect to)
I will try to upload a Draft PR to demonstrate how to do so, just as Proof of Concept

sarroutbi added a commit to sarroutbi/attestation-operator that referenced this issue Oct 5, 2023
sarroutbi added a commit to sarroutbi/attestation-operator that referenced this issue Oct 5, 2023
sarroutbi added a commit to sarroutbi/attestation-operator that referenced this issue Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants