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

Deprecate this module in favour of https://github.com/choria-io/puppet-mcollective? #375

Open
bastelfreak opened this issue Aug 18, 2018 · 10 comments
Labels
needs-feedback Further information is requested

Comments

@bastelfreak
Copy link
Member

Hi everybody,

I would like to deprecate this module. Volcane put a lot of effort into the choria modules. The specific module is located at: https://forge.puppet.com/choria/mcollective_choria and the whole choria suite is documented at: https://choria.io/docs/deployment/mcollective/

I do not see why we should maintain an additional module next to the choria ones. @voxpupuli/collaborators please provide feedback.

@bastelfreak bastelfreak added the needs-feedback Further information is requested label Aug 18, 2018
@dhollinger
Copy link
Member

dhollinger commented Aug 18, 2018

Since @ripienaar has taken over much of the MCO maintenance, I think it would be best to point users to the choria implementation instead of ours. I believe the official Puppet MCollective docs even point to Choria now.

https://puppet.com/docs/mcollective/current/deploy/plugins.html - they do for plugins

@jorhett
Copy link

jorhett commented Aug 18, 2018

I was going to ask the same question myself-- do we want to continue this as an alternative, or deprecate it?

I agree that the choria modules seem well designed, and I have no customers who wouldn't be best served by using those.

@ripienaar
Copy link

Soonish all the old mco code and modules and everything related will move to the choria-legacy GitHub org (that’s the current plan anyway) and mco will leave puppet-agent rpm. At which point choria will be the only option and it will provide a comparability layer for old mco agents.

Much going on now but it seems to me worth consolidating around 1 project given the long abandonment phase etc we should focus around one activity for remaining users and future ones.

@raphink
Copy link
Member

raphink commented Aug 19, 2018

Hmmm, that means those of us who use MCollective in a not very conventional way will need to migrate to Choria…

@ripienaar
Copy link

ripienaar commented Aug 19, 2018

Afraid so. No plans currently to do any further mco releases. What will be released will be a stripped down gem that provides RPC compat and ruby clients.

Be interested to hear your non conventional use cases. If you looked at choria you will see we are doing a ton of new interesting stuff that I doubt was ever done by any mco people. So I am keen to explore new areas.

@raphink
Copy link
Member

raphink commented Aug 20, 2018

I have looked at choria, and implemented it (happily) at a customer. My problem is our old internal implementation, which uses e.g. SSH keys for authentication. It's actually the only PR left open on MCollective I see: https://github.com/puppetlabs/marionette-collective/pull/372

I'm guessing we'll have to let go of this and switch to Puppet certificates instead, right?

@ripienaar
Copy link

Yeah I don’t intend to support SSH keys unfortunately.

I am doing a bunch of work on the security system though - support other CAs etc - though as I don’t want to be tied to PupprtCA

The turnkey choria suite is all puppet but the new choria daemon is for all sorts of places - IoT etc - so it’s a lot more flexible about things.

@raphink
Copy link
Member

raphink commented Aug 20, 2018

I'll have to find time to test all this. Our server side is deployed with Docker, as part of our Dockerized Puppet stack, so this shouldn't be too hard to adapt I hope.

@netdata-be
Copy link

@ripienaar The thing I'm missing in choria (or I missed it) is windows support.
Are you planning to add that?

@ripienaar
Copy link

ripienaar commented Aug 31, 2018

@netdata as it stands today with the mcollectived it is supported. So right now on puppet < 6 you can run it on windows.

The future replacement daemon not yet, its on the list but time is tight, will be some time after 6 is released I suspect.

Already tests pass on windows etc, but I dont have a package (and dont know how to make one lol) so it'll be a bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-feedback Further information is requested
Projects
None yet
Development

No branches or pull requests

6 participants