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

Bump dom0-ztools: disk group change #4004

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

andrewd-zededa
Copy link
Contributor

part 2 of #3996

Signed-off-by: Andrew Durbin <andrewd@zededa.com>
@github-actions github-actions bot requested a review from eriknordmark June 26, 2024 07:03
@andrewd-zededa
Copy link
Contributor Author

ok, amd64 build jet failed not related to the change:

#3 resolve image config for docker.io/docker/buildkit-syft-scanner:stable-1
Error: error building "lfedge/eve-acrn-kernel:ea77a29869f19dfd83e1ca495099013b0df746f1": error building for arch amd64: failed to solve: DeadlineExceeded: failed to do request: Head "https://registry-1.docker.io/v2/docker/buildkit-syft-scanner/manifests/stable-1": dial tcp 54.196.99.49:443: i/o timeout
2024/06/26 07:08:40 error during command execution: error building "lfedge/eve-acrn-kernel:ea77a29869f19dfd83e1ca495099013b0df746f1": error building for arch amd64: failed to solve: DeadlineExceeded: failed to do request: Head "https://registry-1.docker.io/v2/docker/buildkit-syft-scanner/manifests/stable-1": dial tcp 54.196.99.49:443: i/o timeout
make: *** [Makefile:1019: eve-acrn-kernel] Error 1
Error: Process completed with exit code 2.

go tests still having issues but havent dug in all the way yet

@andrewd-zededa andrewd-zededa marked this pull request as ready for review June 26, 2024 08:23
Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

LGTM

@eriknordmark
Copy link
Contributor

The go tests are failing because we failed to build in master (due to bpftrace not building)
@christoph-zededa I'm re-running https://github.com/lf-edge/eve/actions/runs/9674377506 to see if the bpftrace build passes on a second try.

@milan-zededa
Copy link
Contributor

Looks like the ZFS/volumes test is fixed by this.

@milan-zededa milan-zededa merged commit 0f8ff89 into lf-edge:master Jun 26, 2024
25 of 32 checks passed
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 this pull request may close these issues.

3 participants