GitHub user necosta reopened a pull request:
https://github.com/apache/zeppelin/pull/2349
[ZEPPELIN-2214] Set npm installer default npm registry
### What is this PR for?
Complements ZEPPELIN-2278, this is a fix for "connection timeout" bug when
trying to run "npm install" behind a proxy. Instead of connecting to the
official nodejs site from behind a proxy, we should use the already soft-coded
env variable ZEPPELIN_HELIUM_NPM_REGISTRY repository.
### What type of PR is it?
Bug Fix
### ToDos
* [x] - Soft-code npm folder name as well (currently hardcoded "npm") - To
be agreed on...
### What is the Jira issue?
* [ZEPPELIN-2214 - Bug on connecting to the official nodejs site from
behind a proxy]
### How should this be tested?
Enable any Helium plugin from behind a proxy
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nokia/zeppelin zeppelin2214
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2349.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2349
----
commit 9fbf9638a0f4647cf3642220c9a5ad908ac8dfe3
Author: Nelson Costa <[email protected]>
Date: 2017-05-19T09:40:29Z
[ZEPPELIN-2214] Set npm installer default npm registry
commit 877dd9d1c31bb2544109146f4a69045b3ba6e0bb
Author: Nelson Costa <[email protected]>
Date: 2017-05-19T09:56:12Z
[ZEPPELIN-2214] Fixed broken CI
commit c6fc912aeffcb4b917e17a8dbec2b59a5eb7ed33
Author: Nelson Costa <[email protected]>
Date: 2017-05-22T07:33:13Z
[ZEPPELIN-2214] Renamed variables
commit f27ce8a80551a33027ad78474b3981d84d0f7aa9
Author: Nelson Costa <[email protected]>
Date: 2017-05-22T08:02:43Z
[ZEPPELIN-2214] Fixed broken CI
commit 5010a8c32c7ad03bc13bb9b515dbd94b0cf13250
Author: Nelson Costa <[email protected]>
Date: 2017-05-23T06:57:53Z
[ZEPPELIN-2214] Renamed yarn to yarnpkg
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
