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

Add mkosi config for openSUSE Tumbleweed #15527

Merged
merged 4 commits into from Apr 23, 2020
Merged

Conversation

Werkov
Copy link
Contributor

@Werkov Werkov commented Apr 22, 2020

This series adds support for mkosi images of openSUSE Tumbleweed. It also touches the mkosi build process but keeps its distrubution-oblivious nature.

Distributions may be build with various configs, e.g. customized
rootprefix. It'd be unmaintanable to have specific mkosi.build for each
supported distro. Hence, make the build script flexible yet distro
oblivious.
Both hostname and uname utilities boil down to uname(2) syscall. Reduce
tests dependency footprint by using uname for checking hostname too.
Some testcases in test-execute rely on existence of user groups with
certain gids. However, their existence is not universally granted [1].
Although the test could be skipped in their absence and give up testing
some code paths, different approach was chosen -- create dummy groups in
the testing image.

[1] See how systemd-sysusers allocates gids (e.g.
src/test/test-uid-range.c).
@anitazha anitazha added the mkosi label Apr 23, 2020
Copy link
Member

@keszybz keszybz left a comment

Choose a reason for hiding this comment

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

LGTM.

@@ -0,0 +1,62 @@
# SPDX-License-Identifier: LGPL-2.1+
#
# Copyright © 2018 Michal Koutný
Copy link
Member

Choose a reason for hiding this comment

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

We removed (almost all) copyright lines, see #9320. I'd prefer to not add this one.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No problem, removed in a force push.

The setup is meant to run against openSUSE Tumbleweed and install
minimum packages necessary for successful build and passing tests.

To speed up incremental build an external build directory is used.
@keszybz
Copy link
Member

keszybz commented Apr 23, 2020

We don't run mkosi in CI, so let's just merge this.

@keszybz keszybz merged commit ca57eed into systemd:master Apr 23, 2020
Werkov added a commit to Werkov/systemd that referenced this pull request May 1, 2020
User accounts of given names are necessary for the build script and
tests to succeed.

(Fixup of systemd#15527, too keen cleaning of dependencies and cached image
caused this.)
keszybz pushed a commit that referenced this pull request May 4, 2020
User accounts of given names are necessary for the build script and
tests to succeed.

(Fixup of #15527, too keen cleaning of dependencies and cached image
caused this.)
Werkov added a commit to Werkov/systemd that referenced this pull request May 28, 2020
User accounts of given names are necessary for the build script and
tests to succeed.

(Fixup of systemd#15527, too keen cleaning of dependencies and cached image
caused this.)

(cherry picked from commit 05d9125)
Werkov added a commit to Werkov/systemd that referenced this pull request May 29, 2020
User accounts of given names are necessary for the build script and
tests to succeed.

(Fixup of systemd#15527, too keen cleaning of dependencies and cached image
caused this.)

(cherry picked from commit 05d9125)
Werkov added a commit to Werkov/systemd that referenced this pull request Jun 2, 2020
User accounts of given names are necessary for the build script and
tests to succeed.

(Fixup of systemd#15527, too keen cleaning of dependencies and cached image
caused this.)

(cherry picked from commit 05d9125)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

3 participants