Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not print empty searchlist tags if name_servers_list array is empt… #2629

Merged
merged 1 commit into from
May 10, 2021

Conversation

bruncsak
Copy link
Contributor

@bruncsak bruncsak commented May 7, 2021

…y.

Fixes #2628

@SchoolGuy
Copy link
Member

@bruncsak Since we don't have tests for templates so far... Did you test then in your Setup?

@SchoolGuy SchoolGuy requested review from nodeg and SchoolGuy May 7, 2021 15:25
@SchoolGuy SchoolGuy added 3.2.0 Release 3.2.0 Bug Report Reporting a bug labels May 7, 2021
@SchoolGuy SchoolGuy added this to Pull Requests in Backlog via automation May 7, 2021
@SchoolGuy SchoolGuy added this to the v3.3.0 milestone May 7, 2021
@bruncsak
Copy link
Contributor Author

bruncsak commented May 7, 2021 via email

Copy link
Member

@SchoolGuy SchoolGuy 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
Member

@nodeg nodeg left a comment

Choose a reason for hiding this comment

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

LGTM.

@nodeg nodeg merged commit bcaf26a into cobbler:master May 10, 2021
Backlog automation moved this from Pull Requests to Done May 10, 2021
@SchoolGuy SchoolGuy removed this from Done in Backlog Jun 29, 2021
@SchoolGuy SchoolGuy added this to Pull Requests in V3.3.0 via automation Jun 29, 2021
@SchoolGuy SchoolGuy moved this from Pull Requests to Done in V3.3.0 Jun 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.2.0 Release 3.2.0 Bug Report Reporting a bug
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Existing <searchlist></searchlist> tag in autoyast.xml even if name_servers_list is empty
3 participants