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

Include SELinux tasks for zabbix_web and refactor selinux variable names #1220

Merged
merged 4 commits into from
May 9, 2024

Conversation

miikkajo
Copy link
Contributor

@miikkajo miikkajo commented May 7, 2024

SUMMARY

Selinux tasks were not included by zabbix_web role,
seems that include had been dropped by accident (?) on pull request #979 (v2.0.0)

Restored include_tasks: selinux.yml to roles/zabbix_web/tasks/RedHat.yml

Refactor zabbix_selinux variable to zabbix_web_selinux as variable names should be prefixed with role name,
also to use similar pattern with zabbix_proxy_selinux and zabbix_server_selinux variables.

Refactor selinux related variable names in zabbix_web role to correlate with selinux boolean names.

Refactor zabbix_apache_can_connect_ldap to selinux_allow_httpd_can_connect_ldap
as it's not apache specific and not correlating with actual selinux boolean name.

Don't set zabbix_can_network boolean in zabbix_web role as it's not relevant (AFAIK).

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

zabbix_web

Miikka Joutsenvirta and others added 2 commits May 3, 2024 10:59
…ariable names.

Selinux tasks were not included by zabbix_web role,
seems that include had been dropped by accident (?) on pull request ansible-collections#979 (v2.0.0)

Restored `include_tasks: selinux.yml` to roles/zabbix_web/tasks/RedHat.yml

Refactor `zabbix_selinux` variable to `zabbix_web_selinux` as variable names should be prefixed with role name,
also to use similar pattern with `zabbix_proxy_selinux` and `zabbix_server_selinux` variables.

Refactor `selinux_allow_*` variable names to correlate with selinux boolean names.

Refactor `zabbix_apache_can_connect_ldap` to `selinux_allow_httpd_can_connect_ldap` as it's not apache specific
and not correlating with actual seboolean name.
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.

Please add a change fragment.

@miikkajo miikkajo requested a review from pyrodie18 May 8, 2024 10:14
@pyrodie18 pyrodie18 merged commit 35f3d9a into ansible-collections:main May 9, 2024
46 checks passed
pyrodie18 pushed a commit to pyrodie18/community.zabbix that referenced this pull request May 14, 2024
…mes (ansible-collections#1220)

* Include missing SELinux tasks from selinux.yml and refactor selinux variable names.

Selinux tasks were not included by zabbix_web role,
seems that include had been dropped by accident (?) on pull request ansible-collections#979 (v2.0.0)

Restored `include_tasks: selinux.yml` to roles/zabbix_web/tasks/RedHat.yml

Refactor `zabbix_selinux` variable to `zabbix_web_selinux` as variable names should be prefixed with role name,
also to use similar pattern with `zabbix_proxy_selinux` and `zabbix_server_selinux` variables.

Refactor `selinux_allow_*` variable names to correlate with selinux boolean names.

Refactor `zabbix_apache_can_connect_ldap` to `selinux_allow_httpd_can_connect_ldap` as it's not apache specific
and not correlating with actual seboolean name.

---------

Co-authored-by: Miikka Joutsenvirta <miikka@joutsenvirta.fi>
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.

None yet

2 participants