-
-
Notifications
You must be signed in to change notification settings - Fork 599
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
Appears to install nginx even in cases when it's not requested (no proxy) #23
Comments
That does sound like a possible bug. You need to, however, explicitely add the |
Thanks for the fast dialog @karmi, the exact role definition looks like: {
"name": "elasticsearch_server",
"chef_type": "role",
"json_class": "Chef::Role",
"default_attributes": {
"elasticsearch" : {
"min_mem" : "128m",
"max_mem" : "128m"
}
},
"run_list": [
"role[base]",
"recipe[java]",
"recipe[elasticsearch]",
]
} The nginx version is being set in my
The machine, FWIW is a small VPS slice at Linode, thus the obscenely low memory cap (OOM killer is such a bitch). My next step is to work out how badly the nginx proxy will conflict with my existing |
Last word is that I'm using chef-solo via the excellent (newbie tool) knife-solo, effectively it _rsync_s everything up to the server, selects the appropriate node config based on the hostname you SSH to, and then runs chef out of the temporary directory, it's not the fastest way to drive Chef work, but it works essentially well enough for me. |
Notice the cookbook has support for |
Great article @karmi - I'll fly through that, and when I get into the office tomorrow (we're putting an ES backend into a new project this week for MLT searches) - I'll see if I can snatch 20 mins to dive into this cookbook and make it work without the proxy thing. |
Do not “fly” through that, try it :) The cookbook should definitely work without Nginx, if not, I'll fix it. In any case, it should not conflict with your existing Nginx setup, it just adds a |
hehe, point taken :-) I'm up and running now, that's the most important thing - I'll work through it with my pair, he's never used ES before, trying to get ahead and have a clue about driving tomorrow's session. I can confirm if I remove that variable (nginx version) and skip the proxy cookbook, it fails with an nginx missing download. ( no big surprise there! ) |
This is a simple test for karmi/cookbook-elasticsearch#23, which demonstrates it's possible to install just elasticsearch with the cookbook, skipping the nginx part. To test, run: vagrant up lucid64 vagrant ssh lucid64 -c 'curl localhost:9200'
@leehambley Check out https://github.com/karmi/cookbook-elasticsearch/compare/no_nginx — it appears it's perfectly possible to install elasticsearch only with the cookbook, skipping the nginx part? |
Completely doing it wrong at my side... not sure why, closing issue. |
@leehambley Ping me on IRC when you'll see me -- let's get it sorted out. |
Also, watch out for Chef versions -- in my experience, versions above Chef 0.10.8 are badly messed up, particularly 0.10.9, 0.10.10. |
Good to know @karmi, thanks! |
Can't confirm, but I was seeing weird errors about missing
nginx-.tar.gz
without specifying an nginx version in the attribute list.Not a huge problem, as I plan on using the nginx proxy support (excellent idea, by the way) - but weird when I was starting out. if I get chance, I might send a patch, but I'm quite new to Chef.
The text was updated successfully, but these errors were encountered: