Server image cleanup #9577
Server image cleanup #9577
Conversation
|
👋 Hello! Thanks for contributing to our project. If you are unsure the failing tests are related to your code, you can check the "reference jobs". These are jobs that run on a scheduled time with code from master. If they fail for the same reason as your build, it means the tests or the infrastructure are broken. If they do not fail, but yours do, it means it is related to your code. Reference tests: KNOWN ISSUES Sometimes the build can fail when pulling new jar files from download.opensuse.org . This is a known limitation. Given this happens rarely, when it does, all you need to do is rerun the test. Sorry for the inconvenience. For more tips on troubleshooting, see the troubleshooting guide. Happy hacking! |
cbosdo
force-pushed
the
server-image-cleanup
branch
2 times, most recently
from
57ada03 to
9b032f1
Compare
cbosdo
force-pushed
the
server-image-cleanup
branch
4 times, most recently
from
e5e7589 to
c0245d4
Compare
cbosdo
force-pushed
the
server-image-cleanup
branch
from
c0245d4 to
df850c2
Compare
|
Sumaform PR matching this cleanup: uyuni-project/sumaform#1772 |
cbosdo
force-pushed
the
server-image-cleanup
branch
from
df850c2 to
ed21dd7
Compare
|
Addressing part of https://github.com/SUSE/spacewalk/issues/23617 |
| else | ||
| server.run('ipmi_sim -n < /dev/null > /dev/null &', verbose: true, check_errors: true) | ||
| end | ||
| server.run_local('podman run -d --rm --network uyuni -p [::]:623:623/udp -p [::]:9002:9002 --name fakeipmi ghcr.io/uyuni-project/uyuni/ci-fakeipmi:master', verbose: true, check_errors: true) |
There was a problem hiding this comment.
Fine for now.
Maybe in the future we want to have the url of this container image as a env. variables passed through sumaform.
cbosdo
force-pushed
the
server-image-cleanup
branch
3 times, most recently
from
1485f50 to
5864623
Compare
cbosdo
force-pushed
the
server-image-cleanup
branch
from
5864623 to
46b71b3
Compare
OBS cannot handle folders as sources, compress a folder named root in the container folder into a root.tar.gz before pushing to OBS.
In order to remove the repositories from the container image the testsuite needs to stop relying on sumaform to add any package like expect.
The CI should never rely on packages being installed in the container. The OpenIPMI package needed for the simulation is now in a container.
cbosdo
force-pushed
the
server-image-cleanup
branch
from
46b71b3 to
9094459
Compare
What does this PR change?
Reduce the number of unneeded layers by grouping the RUN commands and all the files in an ADD from a tarball.
Also remove depending on packages added in the container by the CI to allow removing the repos from the image.
GUI diff
No difference.
Documentation
No documentation needed: only internal and user invisible changes
DONE
Test coverage
ℹ️ If a major new functionality is added, it is strongly recommended that tests for the new functionality are added to the Cucumber test suite
No tests: cucumber tests changed
DONE
Links
Issue(s): https://github.com/SUSE/spacewalk/issues/26016
Port(s): # add downstream PR(s), if any
Changelogs
Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository
If you don't need a changelog check, please mark this checkbox:
If you uncheck the checkbox after the PR is created, you will need to re-run
changelog_test(see below)Re-run a test
If you need to re-run a test, please mark the related checkbox, it will be unchecked automatically once it has re-run:
Before you merge
Check How to branch and merge properly!