Skip to content
This repository has been archived by the owner on Nov 30, 2023. It is now read-only.

Add plain Ansible devcontainer #1228

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

rockaut
Copy link

@rockaut rockaut commented Jan 3, 2022

A simple devcontainer (based on the python-3) for developing Ansible projects and collections/roles - without ballast not really needed.

@ghost
Copy link

ghost commented Jan 3, 2022

CLA assistant check
All CLA requirements met.

@bamurtaugh
Copy link
Member

bamurtaugh commented Feb 25, 2022

Thanks for opening @rockaut! Do you think the current Ansible definition could be suitable or adapted for your scenario? https://github.com/microsoft/vscode-dev-containers/tree/main/containers/azure-ansible

@bamurtaugh bamurtaugh added the info-needed Issue requires more information from poster label Mar 4, 2022
@rockaut
Copy link
Author

rockaut commented Sep 13, 2022

Sorry I missed the comments. I will have a look at it again over the day.

@rockaut
Copy link
Author

rockaut commented Sep 14, 2022

Thanks for opening @rockaut! Do you think the current Ansible definition could be suitable or adapted for your scenario? https://github.com/microsoft/vscode-dev-containers/tree/main/containers/azure-ansible

@bamurtaugh IDK ... I guess it could. But should it be? If you pack all the Ansible tools into, it would bloat the azure side of things and also the other way around I guess. Then there is also the question for the usability/understandability for the users. I guess a lot of people would wonder about the "azure" part of the devenv?

@bamurtaugh
Copy link
Member

Thanks for your response @rockaut. I think this definition would be a great fit for community hosting and publishing. We recently announced the process for creating and distributing Features, and we're discussing a similar process for templates / definitions here - please let us know if you have any thoughts or questions!

@rockaut
Copy link
Author

rockaut commented Sep 21, 2022

Sounds good to me, thx.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
info-needed Issue requires more information from poster
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants