Skip to content

Conversation

@imobachgs
Copy link
Contributor

@imobachgs imobachgs commented May 7, 2025

Problem

The DNS settings included in an AutoYaST profile are ignored.

Solution

Convert the DNS settings while importing an AutoYaST profile. The main problem is that, in AutoYaST, these are top-level settings. However, in Agama they are defined for each connection. As a first attempt, we are copying these settings to each connection.

Testing

  • Added a new unit test
  • Tested manually

Documentation

See agama-project/agama-project.github.io#75.

@imobachgs imobachgs marked this pull request as ready for review May 7, 2025 10:49
Copy link
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

Co-authored-by: Knut Alejandro Anderssen González <[email protected]>
@imobachgs imobachgs merged commit a38fc45 into master May 7, 2025
10 of 12 checks passed
@imobachgs imobachgs deleted the convert-dns branch May 7, 2025 11:39
@imobachgs imobachgs mentioned this pull request May 26, 2025
imobachgs added a commit that referenced this pull request May 26, 2025
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.

3 participants