Skip to content

Conversation

@jeremymaes
Copy link
Contributor

SUMMARY

Update the zabbix_settings module so it allows configuring the auditlog_mode setting.

This setting is available since Zabbix 7.0 and allows enabling/disabling logging system actions (low-level discovery, network discovery and autoregistration activities performed by the server (System user)) into the audit log.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

zabbix_settings

@nhicher
Copy link

nhicher commented Jan 8, 2025

recheck

@codecov
Copy link

codecov bot commented Jan 10, 2025

Codecov Report

Attention: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Project coverage is 78.77%. Comparing base (52d3307) to head (c74a3e0).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
plugins/modules/zabbix_settings.py 88.88% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1441      +/-   ##
==========================================
+ Coverage   78.75%   78.77%   +0.01%     
==========================================
  Files          47       47              
  Lines        5889     5898       +9     
  Branches     1508     1513       +5     
==========================================
+ Hits         4638     4646       +8     
  Misses        769      769              
- Partials      482      483       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@BGmot
Copy link
Collaborator

BGmot commented Feb 16, 2025

@jeremymaes please rebase against main branch of this repository and push your branch again.

@BGmot
Copy link
Collaborator

BGmot commented Feb 17, 2025

@jeremymaes thank you for your contribution.

@BGmot BGmot merged commit e5f1496 into ansible-collections:main Feb 17, 2025
52 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.

4 participants