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

New function to recover active checks #105

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adrianlzt
Copy link

The new function get_active_checks() return a list of active items that
a Zabbix Agent should execute.

@coveralls
Copy link

coveralls commented Jan 25, 2019

Pull Request Test Coverage Report for Build 267

  • 7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 89.597%

Totals Coverage Status
Change from base Build 262: 0.3%
Covered Lines: 267
Relevant Lines: 298

💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build 265

  • 3 of 7 (42.86%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.09%) to 88.255%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pyzabbix/sender.py 3 7 42.86%
Totals Coverage Status
Change from base Build 262: -1.09%
Covered Lines: 263
Relevant Lines: 298

💛 - Coveralls

@adubkov
Copy link
Owner

adubkov commented Jan 25, 2019

Thank you for your contribution. Could you update tests to support new functions you add?

@adrianlzt adrianlzt force-pushed the feature/remote_config branch from 99bee1d to b8919da Compare January 27, 2019 12:03
The new function get_active_checks() return a list of active items that
a Zabbix Agent should execute.
@adrianlzt adrianlzt force-pushed the feature/remote_config branch from b8919da to 3a03c14 Compare January 28, 2019 19:50
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.

3 participants