Skip to content

Fix network configuration#3306

Merged
imobachgs merged 4 commits intomasterfrom
network-fixes
Mar 19, 2026
Merged

Fix network configuration#3306
imobachgs merged 4 commits intomasterfrom
network-fixes

Conversation

@imobachgs
Copy link
Copy Markdown
Contributor

This PR fixes two problems with network configuration.

Importing connection ids from AutoYaST

When importing an AutoYaST interface that does not have a name, Agama leaves sets the id to an empty string, which is an invalid id.
This patch fallsback to using the device if the name is empty.

SetConfig should not fail when network configuration fails

Similarly to what happened with scripts/files, the SetConfig action stops when there is a network configuration error. This PR
modifies this behavior to just log the error and continue.

Follow-up

  • Report configuration errors from the agama-network service.

  • We need to make sure that none of the services returns an error in SetConfig. The only valid error in those cases are actors communication problems.
    Any other error should be handled by the service. It could:

    1. Notify the user.
    2. Just log the problem.
    3. Register an issue.

Copy link
Copy Markdown
Contributor

@dgdavid dgdavid left a comment

Choose a reason for hiding this comment

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

LGTM 👍

CI failing because something addressed at #3299

@imobachgs imobachgs merged commit 696578d into master Mar 19, 2026
20 of 22 checks passed
@imobachgs imobachgs deleted the network-fixes branch March 19, 2026 22:35
@imobachgs imobachgs mentioned this pull request Apr 14, 2026
imobachgs added a commit that referenced this pull request Apr 14, 2026
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