You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
panos_match_rule fails to handle retries due to a Python exception being thrown instead of having an Ansible error thrown. This was observed when having a Session timeout happen. Received the following two traceback errors that exited the Ansible execution:
pan.xapi.PanXapiError: Sessiontimedout
Also in the same it received another exception:
panos.errors.PanSessionTimedOut: Sessiontimedout
Expected behavior
Expect that the module would accept retries in in a timeout scenario.
Current behavior
Python exceptions cause Ansible to exit rather than allow for the Ansible module to handle the retry.
Possible solution
Wrap the module data inside of a try/except block, raising the error data to a fail_json() method
Steps to reproduce
Create a policy-rule-match test
Have a timeout occur
Screenshots
Context
Working to complete some rule match verification
Your Environment
Collection:
Python:
Ansible:
PAN-OS Python Library & version (e.g. pandevice 0.14.0, pan-os-python 1.0.2):
The text was updated successfully, but these errors were encountered:
Describe the bug
panos_match_rule fails to handle retries due to a Python exception being thrown instead of having an Ansible error thrown. This was observed when having a Session timeout happen. Received the following two traceback errors that exited the Ansible execution:
Also in the same it received another exception:
Expected behavior
Expect that the module would accept retries in in a timeout scenario.
Current behavior
Python exceptions cause Ansible to exit rather than allow for the Ansible module to handle the retry.
Possible solution
Wrap the module data inside of a try/except block, raising the error data to a fail_json() method
Steps to reproduce
Screenshots
Context
Working to complete some rule match verification
Your Environment
The text was updated successfully, but these errors were encountered: