-
Notifications
You must be signed in to change notification settings - Fork 24
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
Contribute upstream to ansible-modules-extras #1
Comments
I'll have to talk to my supervisors about this, but I'm 100% for it. Do you think the module needs to wrap up anymore conda functionality like environments? |
@jiffyclub Good point. I imagine that the module could be merged as-is, but it's going to much more difficult to maintain once it's merged into an upstream repo. I'll ask some of the other Anaconda users in my organization to see if there are any feature requests that we could add now. |
@jiffyclub Also, when the PR to ansible-modules-extras is made, I'd prefer it be made in a way that preserves the full development history from here. It's not that difficult to graft this repo's history into the upstream one, so once we proceed, I can make the new branch in ansible-modules-extras, but you can issue the actual PR, if you'd like. |
I've just come upon a situation with a shared web server where I think it would be nice to be able to manage conda environments via Ansible. So it might be worthwhile keeping control of this for now. |
+1, I'll pass this along to the Anaconda Cluster dev team |
I just talked to @koverholt at SciPy today. He suggested that we also include support for fetching packages from custom channels. Once the channels feature is added, and we ensure that environments are working properly, I think we'll be ready to submit this upstream. |
Also, apparently Conda already has support in Salt. We should probably look at that and see if there are any features we want to |
@jiffyclub The current version does in fact allow managing environments by passing the |
+1 Has this been submitted upstream ? |
@lowks it has not. |
Oh. Any reason why not ? |
No particularly good reason, I think. If other people on this thread (particularly @brianhelba) are still interested and think it's ready we should see about it. But I've since changed jobs and I no longer actively work on this or have ultimate control. |
Right. @brianhelba ??
|
Sent for PR. ansible/ansible-modules-core#3780 |
Is there any work going on this PR? |
Hi guys, I've created a PR for a conda module upstream ansible/ansible#40455 |
This seems like it would be a worthy addition to upstream Ansible. Would you consider submitting this to https://github.com/ansible/ansible-modules-extras ?
If you're not interested in dealing with that process, I'd be happy to submit a PR to ansible-modules-extras myself (maintaining credit to you as Git author and copyright owner).
The text was updated successfully, but these errors were encountered: