-
Notifications
You must be signed in to change notification settings - Fork 292
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
Unable to create actions in Zabbix 6.0 #661
Comments
Hi @felix-s84 , thank you for reporting this issue. Would you be able to test with with community.zabbix checked out from master branch? There were a few PRs merged specifically for zabbix_action with zbx 5.4/6.0 |
for zabbix 6.0.2 I'm getting the same error code while trying to create an email media type. I am using the git repo https://github.com/ansible-collections/community.zabbix.git **ansible [core 2.12.3] TASK [Create an email mediatype with message templates] ************************************************************************************************************** |
Hi! Tested with latest version from master branch:
|
The error does not make any sense |
Hi! Cheked my zabbix_action.py: Found "def_longdata" here:
|
Hi!
and give us the message you get when the run fails? |
I think i figured out: |
Run
See the docs for your ansible version. |
Hi! |
remote formulaid. First forumula in conditions list will be assigned 'A', the second - 'B' etc |
What do you mean "remote"? |
Sorry, meant "remove" in here:
|
But what if we have custom formula |
I can confirm that the error we saw against Zabbix 6.x related to acknowledge_operations as below is broken in 1.5.1 but works when the
Is there any estimate as to when the next version will be published? |
There are numerious fixes for zabbix_action in #664 for Zabbix 6.0. Most of them due to introduced "added strict validation of the methods parameters" https://www.zabbix.com/documentation/current/en/manual/api/changes_5.4_-_6.0#action |
@felix-s84 can you please test with latest changes in |
still the same |
@felix-s84 what does your "community.zabbix.zabbix_action:" look like right now? |
|
we need to see what your variables are when the error happens... please attache the log from ansible-playbook run, similar to #661 (comment) "Actual result" section |
@felix-s84 I reproduced your error. One of your conditions is:
According to https://www.zabbix.com/documentation/current/en/manual/api/reference/action/object it should not have 'value' so it should be:
But if we run it this way we'll get a response: |
This issue happens during value attribued marked 'required' in module arguments. |
With latest patch from @BGmot merged to master, I will now push for 1.6.0 release. @felix-s84 please feel free to re-open this issue if #667 and advice given is this thread wasn't enough to help your issue. |
I downloaded latest zabbix_action.py and removed But we don't use |
@felix-s84 please post what you actually provide to community.zabbix.zabbix_action: (run your playbook with -vvv) |
@D3DeFi can we not wait for a bit with pushing 1.6.0? @felix-s84 is a master in finding bugs and I feel like we need much more testing -(. |
Yeah, I will not wait with 1.6.0 on further development in this issue. We are lucky @felix-s84 reports hip findings :) Thank you both for testing/contributing and involving yourself! Much appreciated |
@felix-s84 you last problem should be fixed by #661 . Plus fixed similar issues with 'update_operations:' |
Not fixed |
you copied something wrong - it is there https://github.com/BGmot/community.zabbix/blob/main/plugins/modules/zabbix_action.py#L1 |
Yeah, sorry, wrong download. Everything works now. Thank you very much! |
SUMMARY
Unable to create actions in Zabbix 6.0
ISSUE TYPE
COMPONENT NAME
community.zabbix.zabbix_action
ANSIBLE VERSION
CONFIGURATION
OS / ENVIRONMENT / Zabbix Version
zabbix_server --version
zabbix_server (Zabbix) 6.0.2
Revision d726a4d 14 March 2022, compilation time: Mar 14 2022 12:17:53
STEPS TO REPRODUCE
run playbook
EXPECTED RESULTS
action created
ACTUAL RESULTS
The text was updated successfully, but these errors were encountered: