8000 Sane Selinux defaults by eb4x · Pull Request #1537 · ansible-collections/community.zabbix · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Sane Selinux defaults #1537

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

Merged
merged 3 commits into from
May 17, 2025
Merged

Sane Selinux defaults #1537

merged 3 commits into from
May 17, 2025

Conversation

eb4x
Copy link
Collaborator
@eb4x eb4x commented May 15, 2025
SUMMARY

Deal with selinux variables we know something about so this just works out of the box.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

roles for server, proxy and web

ADDITIONAL INFORMATION

This alters the defaults, but for good reason, and the people deploying are probably already setting these, so there shouldn't be any surprises.

eb4x added 2 commits May 15, 2025 20:32
zabbix-server and zabbix-proxy are pretty useless without networking,
and the web interface keeps complaining if it can't connect to zabbix
over port 10050, so might aswell let it, and allow it to connect to
connect to databases over the network, as we don't have configuration
in place for using unix sockets.
I don't think this is ever required by zabbix-proxy.
@eb4x eb4x requested review from BGmot and masa-orca May 15, 2025 18:41
Copy link
Collaborator
@pyrodie18 pyrodie18 left a comment

Choose a reason for hiding this comment

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

Add a change fragment and then I think you're good.

@eb4x eb4x merged commit 287343c into ansible-collections:main May 17, 2025
298 checks passed
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
0