Skip to content

fix(ruby): fix network and hostname conversion from AutoYaST#2349

Merged
imobachgs merged 8 commits intomasterfrom
convert-bootloader
May 9, 2025
Merged

fix(ruby): fix network and hostname conversion from AutoYaST#2349
imobachgs merged 8 commits intomasterfrom
convert-bootloader

Conversation

@imobachgs
Copy link
Contributor

@imobachgs imobachgs commented May 9, 2025

Problem

  • The network and the hostname sections are not processed either (although the readers are implemented).
  • The bootloader section is not processed when importing the AutoYaST profile.

Solution

Add support for just a few bootloader parameters and enable network (with fixes) and hostname sections.

Testing

  • Added a new unit test
  • Tested manually

@imobachgs imobachgs changed the title feat(ruby): convert part of the AutoYaST bootloader section fix(ruby): fix network and hostname conversion from AutoYaST May 9, 2025
@imobachgs imobachgs force-pushed the convert-bootloader branch from 4da723d to ab5783f Compare May 9, 2025 16:03
@imobachgs imobachgs marked this pull request as ready for review May 9, 2025 16:03
Copy link
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.

Just minor comments about doc, but approved anyway 👍

Copy link
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.

Codewise LGTM. I fully trust you have checked the behavior beyond the unit tests. Thanks!

@imobachgs
Copy link
Contributor Author

Codewise LGTM. I fully trust you have checked the behavior beyond the unit tests. Thanks!

Yes, I built the package and tested the changes.

@imobachgs imobachgs merged commit 0baf7d2 into master May 9, 2025
7 of 11 checks passed
@imobachgs imobachgs deleted the convert-bootloader branch May 9, 2025 17:21
@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.

2 participants