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

Fix mtda user creation and lava dropin #460

Merged
merged 2 commits into from
Mar 13, 2025

Conversation

fmoessbauer
Copy link
Member

For details, see individual commit messages.

In the ISAR integration, we change the mtda user to be used for
interactive logins. However, this requires a homedir, as usermod does
not create a homedir for a user that did not have one before (which is
also correct, as the home bootstrapping only happens on adduser).

As the ISAR user creation logic is executed after the rootfs install,
the postinst script of mtda-service does not know about the mtda user
and creates no homedir. Then, when ISAR changes the user, it was created
without a home, hence the changed user also not gets any home. By that,
the mtda user remains without a home, which leads to various warnings on
login.

We fix this by always creating the mtda user with a homedir.

Fixes: 5bf7a48 ("fix(www): switch from flask to tornado")
Signed-off-by: Felix Moessbauer <[email protected]>
When enabling the lava override, it completely clears the
IMAGE_PREINSTALL list (except for the lava-dispatcher package). This is
because an incorrect append is used (we append to IMAGE_PREINSTALL:lava
instead of just appending in case lava is selected).

Fixes: 6bf2e42 ("meta-isar: images: make LAVA an optional feature")
Signed-off-by: Felix Moessbauer <[email protected]>
@chombourger
Copy link
Collaborator

thank you!

@chombourger chombourger merged commit 09c78b8 into siemens:master Mar 13, 2025
6 checks passed
@chombourger chombourger added this to the 0.36 milestone Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants