Skip to content

Fix /var warning for sudo, networkmanager and softhsm.#1092

Merged
ricardosalveti merged 3 commits intofoundriesio:mainfrom
ricardosalveti:var
Apr 3, 2023
Merged

Fix /var warning for sudo, networkmanager and softhsm.#1092
ricardosalveti merged 3 commits intofoundriesio:mainfrom
ricardosalveti:var

Conversation

@ricardosalveti
Copy link
Copy Markdown
Member

No description provided.

@ricardosalveti ricardosalveti requested a review from a team March 31, 2023 21:16
Copy link
Copy Markdown
Contributor

@MrCry0 MrCry0 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@igoropaniuk igoropaniuk left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread meta-lmp-base/recipes-security/softhsm/softhsm_2.%.bbappend Outdated
Comment thread meta-lmp-base/recipes-connectivity/networkmanager/networkmanager_1.36.%.bbappend Outdated
Comment thread meta-lmp-base/recipes-extended/sudo/sudo_%.bbappend Outdated
Files are instead created at runtime via systemd tmpfiles.d.

Signed-off-by: Ricardo Salveti <[email protected]>
/var is not preserved when generating ostree-based images and NM is able
to dynamically create the required folder at runtime.

Signed-off-by: Ricardo Salveti <[email protected]>
/var is not preserved when generating ostree-based images, so drop
it and replace the needed folders via tmpfiles (created at runtime).

Signed-off-by: Ricardo Salveti <[email protected]>
Copy link
Copy Markdown
Member

@quaresmajose quaresmajose left a comment

Choose a reason for hiding this comment

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

LGTM

@ricardosalveti ricardosalveti merged commit 27af118 into foundriesio:main Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants