PaloAltoNetworks / pan-os-ansible

Ansible collection for easy automation of Palo Alto Networks next generation firewalls and Panorama, in both physical and virtual form factors.

https://pan.dev/ansible/docs/panos

Apache License 2.0

204 stars 97 forks source link

feat: Add additional error handling to some upgrade assurance modules #561

Closed acelebanski closed 4 months ago

acelebanski commented 5 months ago

Description

In customer projects we were using local copies of the following modules:

panos_readiness_checks.py
panos_snapshot_report.py

They had some additional Try Except blocks that were not present in public modules. This PR aims to introduce them.

Motivation and Context

559

How Has This Been Tested?

These changes were introduced in local libraries used earlier in Upgrade Automation playbooks.

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.