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

fix: Use correct syntax for keyserver port and test check #220

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

richm
Copy link
Collaborator

@richm richm commented Nov 19, 2024

Cause: The role was using incorrect syntax for specifying the
keyserver port for the firewall.

Consequence: The role would give an error when configuring
the keyserver port in the firewall.

Fix: Use the correct syntax in the role and in the test
used to check the firewall.

Result: The role can configure the firewall port for the
keyserver correctly.

Signed-off-by: Rich Megginson [email protected]

@richm richm requested a review from natoscott as a code owner November 19, 2024 19:23
@richm
Copy link
Collaborator Author

richm commented Nov 19, 2024

[citest]

Cause: The role was using incorrect syntax for specifying the
keyserver port for the firewall.

Consequence: The role would give an error when configuring
the keyserver port in the firewall.

Fix: Use the correct syntax in the role and in the test
used to check the firewall.

Result: The role can configure the firewall port for the
keyserver correctly.

Signed-off-by: Rich Megginson <[email protected]>
@richm
Copy link
Collaborator Author

richm commented Nov 19, 2024

[citest]

@richm richm merged commit ed68fdd into linux-system-roles:main Nov 19, 2024
16 checks passed
@richm richm deleted the firewall-port branch November 19, 2024 20:59
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.

1 participant