Skip to content

Commit

Permalink
Merge pull request #291 from cdapio/feature/nodejs-cdap-6-1
Browse files Browse the repository at this point in the history
[COOK-138] use node v10 for cdap 6.1
  • Loading branch information
dereklwood authored Sep 4, 2019
2 parents 18798a8 + d1b13d9 commit b1ad37e
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion attributes/sdk.rb
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,11 @@
default['cdap']['sdk']['init_actions'] = [:enable, :start]

# Get proper Node.js version on SDK
if node['cdap']['version'].to_i >= 5
if node['cdap']['version'].to_f >= 6.1
default['nodejs']['install_method'] = 'binary'
default['nodejs']['version'] = '10.16.2'
default['nodejs']['binary']['checksum']['linux_x64'] = nil
elsif node['cdap']['version'].to_i >= 5
default['nodejs']['install_method'] = 'binary'
default['nodejs']['version'] = '8.7.0'
default['nodejs']['binary']['checksum']['linux_x64'] = nil
Expand Down

0 comments on commit b1ad37e

Please sign in to comment.