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

Extend firewall management for agent role #597

Open
wants to merge 4 commits into
base: build/checkmk-2-3-support
Choose a base branch
from

Conversation

robin-checkmk
Copy link
Member

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: #593

What is the new behavior?

  • A new variable is introduced: checkmk_agent_server_ips. I can be set to a list of IPs, which will be allowed in the firewall.
  • This will not modify the agent's own whitelisting mechanisms!
  • The checkmk_agent_server will automatically be added to the above list, but only if it is an IP address.

Other information

This collection has an implicit dependency for the Python module netaddr. This is not new, but apparently never was an issue. This PR adds a hint to the README about this.

@robin-checkmk robin-checkmk added the role:agent This affects the agent role label Apr 26, 2024
@robin-checkmk robin-checkmk self-assigned this Apr 26, 2024
@robin-checkmk robin-checkmk changed the title Feature/agent adv fw mgmt Extend firewall management for agent role Apr 26, 2024
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation role:agent This affects the agent role
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant