David Wittman
David Wittman
No reason other than your stated theory: it was good enough in its infancy, which is about when I made this role. And honestly -- as evidenced by the backlog...
> @DavidWittman > > Hey David, I took the time to create a PR as approved by you earlier. It would be great if you could have a look at...
My only concern with this is that it changes behavior for users that relied only on the `redis_sentinel` flag, but I don't think that's _too_ big of a deal. I'll...
Thanks @YitianJiang, this should probably be called out in the README
@jlecour Agreed. I'll look into this. It gets a little hairy because there are differences in the packages between distributions too, but I can at least give a shot at...
https://github.com/ANXS/nginx/ does this too via the `nginx_install_method` variable.
No news from my end, but I'd accept a PR which implements this via `redis_install_method`.
@awr Sorry for the slow feedback. I think I'm going to implement this as a role dependency, which will require pushing it up to Galaxy. If you haven't got authorization...
@zbal Can you remove the commit to add Redis to the path? Since the issues have been addressed in the `alternatives` module, I can explore adding Redis to the path...