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

ES 2.2.0 installation fails #429

Closed
heffergm opened this issue Feb 3, 2016 · 1 comment

Comments

Projects
None yet
1 participant
@heffergm
Copy link

commented Feb 3, 2016

Using the latest cookbook release, I'm getting the following nilclass error when attempting to install 2.2.0. However, the same block works when passed a version of 2.1.1, so I'm tentatively suspicious that the failure in determine_download_checksum is an uncaught exception resulting from a new download location (let me re-iterate the 'tentative' part... I haven't looked in great detail as yet).

NoMethodError
-------------
undefined method `[]' for nil:NilClass

Cookbook Trace:
---------------
/var/chef/runs/6f90dc7c-9ce5-4136-a581-45f0a59c583e/local-mode-cache/cache/cookbooks/elasticsearch/libraries/helpers.rb:118:in `determine_download_checksum'
/var/chef/runs/6f90dc7c-9ce5-4136-a581-45f0a59c583e/local-mode-cache/cache/cookbooks/elasticsearch/libraries/provider_install.rb:107:in `block in install_tarball_wrapper_action'
/var/chef/runs/6f90dc7c-9ce5-4136-a581-45f0a59c583e/local-mode-cache/cache/cookbooks/elasticsearch/libraries/provider_install.rb:101:in `install_tarball_wrapper_action'
/var/chef/runs/6f90dc7c-9ce5-4136-a581-45f0a59c583e/local-mode-cache/cache/cookbooks/elasticsearch/libraries/provider_install.rb:20:in `block in <class:InstallProvider>'

Resource Declaration:
---------------------
# In /var/chef/runs/6f90dc7c-9ce5-4136-a581-45f0a59c583e/local-mode-cache/cache/cookbooks/mapzen_es/recipes/_install.rb

15:   elasticsearch_install layer do
16:     action  :install
17:     type    node[:mapzen_es][:elasticsearch][:install_type]
18:     version node[:mapzen_es][:elasticsearch][basename.to_sym][:version]
19:   end
20: 

Compiled Resource:
------------------
# Declared in /var/chef/runs/6f90dc7c-9ce5-4136-a581-45f0a59c583e/local-mode-cache/cache/cookbooks/mapzen_es/recipes/_install.rb:15:in `block in from_file'

elasticsearch_install("elasticsearch-logstash") do
action [:install]
retries 0
retry_delay 2
default_guard_interpreter :default
declared_type :elasticsearch_install
cookbook_name "mapzen_es"
recipe_name "_install"
type :tarball
version "2.2.0"
end

[2016-02-03T22:20:53+00:00] INFO: Running queued delayed notifications before re-raising exception
[2016-02-03T22:20:53+00:00] ERROR: Running exception handlers
[2016-02-03T22:20:53+00:00] ERROR: Exception handlers complete
[2016-02-03T22:20:53+00:00] FATAL: Stacktrace dumped to /var/chef/runs/6f90dc7c-9ce5-4136-a581-45f0a59c583e/local-mode-cache/cache/chef-stacktrace.out
[2016-02-03T22:20:53+00:00] ERROR: elasticsearch_install[elasticsearch-logstash] (mapzen_es::_install line 15) had an error: NoMethodError: undefined method `[]' for nil:NilClass
[2016-02-03T22:20:55+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
@heffergm

This comment has been minimized.

Copy link
Author

commented Feb 3, 2016

Ack, nevermind, my bad. I see the checksums are provided by version in attributes. Closing...

@heffergm heffergm closed this Feb 3, 2016

martinb3 added a commit that referenced this issue Feb 4, 2016

Add safety to the version check
When passing a version we don't have checksums for, do a better job of determining the checksum. We should return a falsey value, not a stack trace.

Fixes #429.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.