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.

Sign up for GitHub

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

Jump to bottom

Improve zabbix_proxy type #679

Merged
merged 2 commits into from
Apr 21, 2020
Merged

Improve zabbix_proxy type #679

merged 2 commits into from
Apr 21, 2020

Conversation

alexjfisher
Copy link
Member

@alexjfisher alexjfisher commented Apr 17, 2020 •
edited
Loading

  • It is now possible to manage individual zabbix_proxy resource properties.
  • The type has been improved to do enhanced validation of all properties.
  • Switching between active and passive proxies (and vice-versa) is now supported.
  • Improved acceptance tests (Thanks @baurmatt )
  • Improved type unit tests.

@alexjfisher alexjfisher force-pushed the zabbix_proxy branch 17 times, most recently from 81fa140 to 9cf72a9 Compare April 20, 2020 11:54
@alexjfisher alexjfisher force-pushed the zabbix_proxy branch 7 times, most recently from 8b1057b to 2cfc1ee Compare April 21, 2020 09:55
@alexjfisher alexjfisher changed the title WIP: zabbix_proxy Improve zabbix_proxy type Apr 21, 2020
@alexjfisher alexjfisher added the enhancement New feature or request label Apr 21, 2020
@alexjfisher
Improve zabbix_proxy type
f125061 
It is now possible to manage individual `zabbix_proxy` resource
properties.

The type has been improved to do enhanced validation of all properties.
Switching between active and passive proxies (and vice-versa) is now
supported.
@baurmatt baurmatt mentioned this pull request Apr 21, 2020
Closed
@bastelfreak bastelfreak merged commit bd6c597 into voxpupuli:master Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baurmatt baurmatt baurmatt approved these changes

@bastelfreak bastelfreak bastelfreak approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alexjfisher @bastelfreak @baurmatt