-
Notifications
You must be signed in to change notification settings - Fork 10
Fixed missing solr version #5
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
base: master
Are you sure you want to change the base?
Conversation
|
I will merge this after someone else has confirmed this fix! I may also be able to test this in recent future. |
|
This is going to be prone to fail in the future too though. I wonder if we On Mon, 8 Feb 2016 21:13 jiv_e notifications@github.com wrote:
|
|
That's a great point! I hope it could be possible to select the version. If you have some solution for selecting the newest version I'm happy to include it in. In fact I'm planning to rewrite this role to be available in Ansible Galaxy. I could try to figure this out for that release. |
|
Let me have a look tomorrow, see if I can find /do something. That'd great, I could give you a hand as in fact I was planning to use this On Mon, 8 Feb 2016, 22:50 jiv_e notifications@github.com wrote:
|
|
Yes! Exactly! Like this: |
|
Very good stuff. It doesn't look like you are miles away, do you have an Cheers :-) On Mon, 8 Feb 2016, 23:02 jiv_e notifications@github.com wrote:
|
|
I think I will use this repo. Feel free to push! :) |
|
I've just opened a tentative, it's my first role and I'm not how I can test it properly ? I'll give it a go in a different machine tomorrow morning from work. |
|
Thanks, worked like a charm for me! |
|
This doesn't work anymore either, version 4 is gone from there. If you need to use version 4, you can find it here: https://archive.apache.org/dist/lucene/solr/4.10.4/ |
|
I have updated the PR with a possible solution, thanks @badrange it does not seem to be much interest on this PR though :-( |
|
I'm sorry for my low level of activity! I have too much stuff happening in my life at the moment. I don't have time to test this. I ask someone to test Alex's pull request! I will merge this when it is tested. |
|
no worries at all jiv-e, sorry to chase you on this. host could also be used as localhost, it normally works as this by default |
4.7.2 version does not work anymore:
this PR should fix it