-
Notifications
You must be signed in to change notification settings - Fork 159
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
Enable DEV=y build support on HV=kubevirt #4074
Enable DEV=y build support on HV=kubevirt #4074
Conversation
build failure fixed by recently merged #4073
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Then why not to rebase this PR so that we can check that it does not break anything? |
758e1ec
to
10d9d11
Compare
latest push is a rebase, we'll see how it does |
Only arm64 builds failed and I think all identically like this:
I'm not sure what to make of this at the moment. |
@eriknordmark , @andrewd-zededa , I've started to see this error recently, @jsfakian also observed on his PR... some Alpine packages are failing from time to time, it needs investigation. In this case iproute2 and radvd packages installation failed, causing the job to fail: |
Triggering failed jobs again, let's see if it works now... |
It isn't just those 2 packages, I see bash, crony, iprotue2, radvd, even busybox and ca-certificates. It is strange, because none of this goes to the Internet; it all uses stuff cached in Can you rebase this on latest |
10d9d11
to
d62c9fe
Compare
pushed up a rebase on latest master |
Still build failures in apk package installs and all error 127 and I can't reproduce this on my local build environment. My arm builds succeed.
The first failure though is in eve-alpine-deploy.sh and I'm curious if the rest are symptoms of that:
|
The alpine-baselayout pre-install script is here. Although that is the latest, and ours might be slightly different, I don't think it is material. The bash post-install script is here. They are as simple as it gets. What could possibly be failing with so many of them with 127? Normally that means command not found, but what could not be found there? |
And you aren't seeing it locally, correct? I would worry about full filesystems, but given that other packages are being installed successfully - it is only the scripts - that doesn't seem logical to me. |
Use correct build yml for pillar so that image is built with -dev suffix. Signed-off-by: Andrew Durbin <andrewd@zededa.com>
d62c9fe
to
f8f5812
Compare
latest push was just a rebase on latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Run eden
Use correct build yml for pillar so that image is
built with -dev suffix.