-
Notifications
You must be signed in to change notification settings - Fork 4
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
Removed library-staging1 &2 from inventory #5702
Conversation
9a55cf9
to
bbee6f0
Compare
Can you also remove the group from our other inventory files? See princeton_ansible/inventory/by_team/dacs Line 20 in eff2dfb
|
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.
Thanks, @dphillips-39 and @VickieKarasic ! When I run a playbook from this branch, I get:
* Failed to parse /Users/sandbergj/repos/princeton_ansible/inventory/by_environment/staging with ini plugin:
/Users/sandbergj/repos/princeton_ansible/inventory/by_environment/staging:27: Section [staging:children] includes undefined group: libwww_staging
Could you please remove libwww_staging
from inventory/by_environment
too so that we don't get this warning?
Edit: I just saw @acozine already said that! Apologies for the duplicate!
@sandbergja and @acozine thanks for pointing this out! Just removed those files from our other groups and pushed my changes. I was going to hold off on removing libwww_production until we decommission those VMs at the end of this month, but let me know if removing sooner would be better (related to #5651) |
Co-authored-by: Vickie Karasic <[email protected]>
4e4e7e7
to
77a3609
Compare
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.
This works great, so nice to get this old group out of princeton_ansible. Thanks @VickieKarasic and @dphillips-39 !
Co-authored-by: Vickie Karasic [email protected]