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

podvm: Fix podvm-binaries build #1726

Conversation

stevenhorsman
Copy link
Member

Nightly podvm-binaries build has been failing with:

# Set default policy
956.9 cd /src/cloud-api-adaptor/podvm/files/etc/kata-opa && ln -s -f "" default-policy.rego
956.9 ln: failed to create symbolic link 'default-policy.rego' -> '': No such file or directory

So port over the mkosi fix for this that Magnus did in #1718

@stevenhorsman
Copy link
Member Author

https://github.com/confidential-containers/cloud-api-adaptor/actions/runs/8160892320/job/22310210579?pr=1726 shows that the podvm image build is resolved in this issue. The tests didn't pass as they failed to get a runner, so back to the garm issue, but I think this is progress and worth merging.

Copy link
Member

@bpradipt bpradipt left a comment

Choose a reason for hiding this comment

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

/lgtm
nit: typo in the commit message - povm -> podvm

Nightly podvm-binaries build has been failing with:
```
956.9 cd /src/cloud-api-adaptor/podvm/files/etc/kata-opa && ln -s -f "" default-policy.rego
956.9 ln: failed to create symbolic link 'default-policy.rego' -> '': No such file or directory
```

So port over the mkosi fix for this that Magnus did in confidential-containers#1718

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
@stevenhorsman stevenhorsman force-pushed the podvm-binaries-default-agent-policy-fix branch from f67faba to 5c3071b Compare March 6, 2024 11:44
@stevenhorsman stevenhorsman changed the title podvm: Fix povm-binaries build podvm: Fix podvm-binaries build Mar 6, 2024
@stevenhorsman
Copy link
Member Author

Since I ran it last night govuln has updated and we need to bump to 1.21.8 now, so I've raised #1728 for this

Copy link
Contributor

@huoqifeng huoqifeng left a comment

Choose a reason for hiding this comment

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

LGTM

@stevenhorsman
Copy link
Member Author

govuln checks are addressed under #1728

@stevenhorsman stevenhorsman merged commit 06fc733 into confidential-containers:main Mar 6, 2024
25 of 27 checks passed
@stevenhorsman stevenhorsman deleted the podvm-binaries-default-agent-policy-fix branch March 6, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test_e2e_libvirt Run Libvirt e2e tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants