Skip to content
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

updated upgradepackage and unit_test workflow as an example of end to end ftd upgrade #196

Merged
merged 2 commits into from
Apr 28, 2024

Conversation

manofcolombia
Copy link
Contributor

updated upgradepackage to include the ability to trigger readiness checks and other newer flags in that api endpoint (based on fmc 7.4.1.1)

upgrade unit_test workflow now has an example of end to end ftd upgrade (push file, readiness check, upgrade, post upgrade deployment)

fmcapi/api_objects/deployment_services/deploymentrequests.py was modified and may break some code in the wild with this modification.

Instead of returning just the device list in the deployment response, it made more sense to me to return the entire response and then pull the device list out outside of the object code. This also means that deploymentrequest now works and fits better with the existing wait_for_task unit_test

Zachery Lantz added 2 commits April 25, 2024 13:26
…_post bug, added additional code to unit test for examples of how to do an entire upgrade start to finish, fixed a broken import for wait_for_task
…quest to be the entire response so it can be piped into wait_for_task. This doesn't seem to break anything else in the library but could break code in the wild, but I believe returning the full response from a deployment request to be worth the cost
@manofcolombia
Copy link
Contributor Author

I have used this to upgrade standalone FTDs and HA FTDs. I still have to confirm that the example upgrade code works with multiple standalone devices at one time. Hoping to get a window to run it against the remaining devices I need to upgrade in our environment.

@marksull marksull marked this pull request as ready for review April 28, 2024 04:32
@marksull marksull merged commit 648b8ac into marksull:master Apr 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants