zabbix icon indicating copy to clipboard operation
zabbix copied to clipboard
verified publisher icon laradji

Added screen resource with provider, supporting the usual actions

Open nightw opened this issue 11 years ago • 2 comments

About half a year ago I've already submitted this change and ( #177 ) and @guilhem asked for a test for this. I've not provided it back then for a long time, so the Pull Request got closed without merging it. Sorry for taking so long, here is the change with a test case attached to it. :)

The test case is a bit big (installs MySQL, Zabbix, Zabbix web, etc. to be able to use the API for the LWRP), very specific (only works using Ubuntu 14.04 because of some package/cookbook breaking changes), but if it does not break, that means the screen resource could be created with this LWRP using the Zabbix API.

Please review (and run the test) and if you find it appropriate then merge it. Thanks.

nightw avatar Feb 23 '15 13:02 nightw

Also corrected the RuboCop offenses. Now the test fails for some reason on Ruby 1.9 during bundle install, so it's not related to this pull request (and should be fixed separately). On Ruby2.0 the test passes.

nightw avatar Feb 26 '15 12:02 nightw

Could you please review this?

nightw avatar Oct 20 '15 11:10 nightw