Skip to content

fix: Fix rendering Match/Host defaults when user provides their own#104

Merged
Jakuje merged 4 commits intolinux-system-roles:mainfrom
Jakuje:defaults
Jul 10, 2023
Merged

fix: Fix rendering Match/Host defaults when user provides their own#104
Jakuje merged 4 commits intolinux-system-roles:mainfrom
Jakuje:defaults

Conversation

@Jakuje
Copy link
Copy Markdown
Collaborator

@Jakuje Jakuje commented Jun 29, 2023

The condition in the template caused that the default Match or Host options were not rendered when the user provided their Match/Host options in the ssh dictionary.

Fixes #92

@Jakuje Jakuje requested a review from richm as a code owner June 29, 2023 17:34
@Jakuje Jakuje changed the title bufgix: Fix rendering Match/Host defaults when user provides their own bugfix: Fix rendering Match/Host defaults when user provides their own Jun 29, 2023
@Jakuje Jakuje changed the title bugfix: Fix rendering Match/Host defaults when user provides their own fix: Fix rendering Match/Host defaults when user provides their own Jun 29, 2023
@richm
Copy link
Copy Markdown
Contributor

richm commented Jun 29, 2023

[citest]

@Jakuje
Copy link
Copy Markdown
Collaborator Author

Jakuje commented Jul 10, 2023

Seems like the CI is stuck. If it will happen also on the master, I will have a look into that in separate PR.

@Jakuje Jakuje merged commit 7390748 into linux-system-roles:main Jul 10, 2023
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.

Defaults hostblock ignored when non-default hostblock is used

3 participants