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

more ansible concepts #75

Merged
merged 17 commits into from
Jun 26, 2022

Conversation

lilatomic
Copy link
Contributor

closes #70

Adds many more ansible concepts!
One thing it doesn't add is role dependencies on galaxy items. I left this out since I we'd need to pick a no-op role from galaxy.

Also, ansible-playbook is currently failing since "my_namespace.my_collection.collection_role" needs to be installed. do we want to comment the role line out?

lilatomic added 15 commits June 25, 2022 15:23
useful for coupling the same playbook to different inventories
ansible can use local roles and collections without them being built and installed,
so long as they exist withing the directory convention
It has some YAML problems,
but isn't taken in by pants as an ansible file,
so there were conflicting lint error counts
Local collections are poorly documented.
Like local roles, they don't need to be installed
to be referenced in playbooks.
waiting on other features to be implemented
for now, they just depend on local roles
and don't require galaxy installs.
Because I don't know of a good no-op role
running it with pants would not find this error
@lilatomic
Copy link
Contributor Author

I guess we also have to decide if the ansible should or should not have errors

@sureshjoshi sureshjoshi merged commit e9ffe21 into sureshjoshi:main Jun 26, 2022
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.

[ansible] Update example to use more Ansible concepts
2 participants