Skip to content

fix(live): make the resolv.conf accesible from /sysroot#2669

Merged
imobachgs merged 3 commits intomasterfrom
fix-dud-apply-network
Aug 20, 2025
Merged

fix(live): make the resolv.conf accesible from /sysroot#2669
imobachgs merged 3 commits intomasterfrom
fix-dud-apply-network

Conversation

@imobachgs
Copy link
Copy Markdown
Contributor

@imobachgs imobachgs commented Aug 19, 2025

Problem

When applying a DUD, it is not possible to access the network when running agama download from the /sysroot.

Solution

  • Mount the /run file system on /sysroot/run.
  • Link /sysroot/etc/resolv.conf to /sysroot/run/etc/NetworkManager/resolv.conf file.
  • Remove the /run/agama/dud directory to release some memory before pivoting.

Testing

  • Tested manually.

A testing build should be available home:IGonzalezSosa:branches:fix-dud-apply-network at some point.

@imobachgs imobachgs requested a review from teclator August 19, 2025 14:19
@imobachgs imobachgs force-pushed the fix-dud-apply-network branch 3 times, most recently from b195cb1 to ffc9740 Compare August 20, 2025 09:32
@imobachgs imobachgs marked this pull request as ready for review August 20, 2025 09:41
@imobachgs imobachgs force-pushed the fix-dud-apply-network branch from ffc9740 to 45accf5 Compare August 20, 2025 09:42
Copy link
Copy Markdown
Contributor

@teclator teclator left a comment

Choose a reason for hiding this comment

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

LGTM

@imobachgs imobachgs merged commit 23acf60 into master Aug 20, 2025
11 checks passed
@imobachgs imobachgs deleted the fix-dud-apply-network branch August 20, 2025 10:37
teclator added a commit that referenced this pull request Feb 3, 2026
## Problem

After the installation, when running a post-script there is a leftover
symlink linking the /etc/resolv.conf to
/mnt/run/NetworkManager/resolv.conf

- *https://bugzilla.suse.com/show_bug.cgi?id=1257468*


## Solution

Link the `/mnt/etc/resolv.conf` to `/run/NetworkManager/resolv.conf` as
it was done in #2179 or
#2669


## Testing

- *Added a new unit test*
- *Tested manually*
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.

2 participants