An efficient tool to execute configuration backups, network state snapshots, system readiness checks, and operating system upgrades of Palo Alto Networks firewalls and Panorama appliances.
[x] Ensure you are submitting your pull request to a branch dedicated to a specific topic/feature/bugfix. Avoid using the master branch for pull requests.
[x] Target your pull request to the main development branch in this repository.
[x] Ensure your commit messages follow the project's preferred format.
[x] Check that your code additions do not fail any linting checks or unit tests.
Pull Request Description
This pull request addresses a critical bug identified in the post-upgrade process of the firewall upgrade script. Previously, the generation of post-upgrade snapshots and diff reports was skipped if a settings.yaml file was not present in the working directory. This pull request ensures that these crucial steps are executed regardless of the settings.yaml presence, thus guaranteeing the availability of post-upgrade snapshots and reports for every upgrade.
What does this pull request accomplish?
[ ] Feature addition
[x] Bug fix
[ ] Performance improvement
[ ] Refactoring
[ ] Documentation update
[ ] Other
Are there any breaking changes included?
[ ] Yes
[x] No
Is there anything the reviewers should know?
This pull request introduces changes that ensure post-upgrade steps are executed consistently, enhancing the reliability and traceability of firewall upgrades. It is a vital fix to ensure that upgrades' effects can be thoroughly reviewed and verified.
Related Issue: Describes a bug where post-upgrade snapshots and diff reports were skipped if settings.yaml was not found. This PR ensures these steps are always executed to maintain upgrade integrity.
Commits Included:
Add successful connection logging to device.py
Move logic for post-upgrade snapshots outside of conditional
Update project description and add key features and logic workflow
Checklist for This Pull Request
🚨Please adhere to the guidelines for contributing to this repository.
Pull Request Description
This pull request addresses a critical bug identified in the post-upgrade process of the firewall upgrade script. Previously, the generation of post-upgrade snapshots and diff reports was skipped if a
settings.yamlfile was not present in the working directory. This pull request ensures that these crucial steps are executed regardless of thesettings.yamlpresence, thus guaranteeing the availability of post-upgrade snapshots and reports for every upgrade.What does this pull request accomplish?
Are there any breaking changes included?
Is there anything the reviewers should know?
This pull request introduces changes that ensure post-upgrade steps are executed consistently, enhancing the reliability and traceability of firewall upgrades. It is a vital fix to ensure that upgrades' effects can be thoroughly reviewed and verified.
Related Issue: Describes a bug where post-upgrade snapshots and diff reports were skipped if
settings.yamlwas not found. This PR ensures these steps are always executed to maintain upgrade integrity.Commits Included:
Thank you for your contributions!