search

metal3-io / utility-images

Containerized utility images for the Metal3 project
Apache License 2.0
2 stars 2 forks source link

Post merge container build is broken for ipxe #10

Closed NymanRobin closed 5 months ago

NymanRobin commented 6 months ago

The IPXE container builds is not passing this needs to be fixed before any notification system is added to the builds. The current error looks like the paths are relative to repository in container build but jenkins uses absolute path when running build this causes error. Fix might be to make all paths absolute

#4 CANCELED
------
 > [3/3] COPY buildipxe embed.ipxe.j2 /bin/:
------
Dockerfile:5
--------------------
   3 |     RUN dnf install -y gcc git-core make perl xz-devel python3-setuptools python3-jinja2
   4 |     
   5 | >>> COPY buildipxe embed.ipxe.j2 /bin/
   6 |     
   7 |     CMD /bin/buildipxe.sh
--------------------
ERROR: failed to solve: failed to compute cache key: failed to calculate checksum of ref b2e5b9f7-0fd7-4608-8569-947c5fc96142::3hjpyyfbhby6np7ztniq5vp5f: "/buildipxe": not found
make: *** [Makefile:27: build-container-image] Error 1`
NymanRobin commented 6 months ago

The description of the issue is not correct got reminded that the build context of docker is relative to the dockerfile. The error seems to be that .sh ending is missing for the buildipxe file

NymanRobin commented 6 months ago

/triage accepted

Rozzii commented 5 months ago

Thanks for the fix. /close

metal3-io-bot commented 5 months ago

@Rozzii: Closing this issue.

In response to [this](https://github.com/metal3-io/utility-images/issues/10#issuecomment-2196469827): >Thanks for the fix. >/close Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.