-
Notifications
You must be signed in to change notification settings - Fork 630
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
Minor updates for requirements #43
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do want to make this AVD 4.5 and CVP 3.10.1 now?
We should also update the Lab guide's instructions (separate PR) to run the following commands.
to become:
|
Successful test in ATD Lab with AVD 4.5 and CVP 3.10.1. All AVD Labs run as desired. |
The reason I keep it like this is to ensure we always have the correct python packages installed vs maintaining a separate requirments.txt file. We only need this file for the CICD workflow to install ansible-core |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Minor updates for requirements
Updating AVD, CVP, and ansible-core versions