-
Notifications
You must be signed in to change notification settings - Fork 203
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
Doc: AVD with Ansible Automation Platform Guide #3910
Conversation
Quality Gate passedIssues Measures |
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, great job Julio!
Going to set overwrite on the variables as well to make the source of truth rule all the things. |
Disregard, the way we set overwrite on inventory sync, even if folks added a custom var to host or groups, those would be removed when the inventory is updated. Only difference is if someone supplies a job specific variable. |
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.
@JulioPDX while I requested some changes, I don't think that is critical and overall it's a great guide that has to be published as soon as possible. Ping me any time for 2nd LGTM if you prefer not to fix some comments (like testing with the recent AVD version).
requested changes have been addressed
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, great job on this guide!
Change Summary
This getting started guide will walk AVD users through getting up and running with AVD and AAP.
Proposed changes
Please see change summary
How to test
View docs generated with PR
Checklist
Repository Checklist