[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-21 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 Please go on! Thanks! --- 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

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2150 Thanks @andreaTP for the improvement! LGTM and merge to master if no further discussions. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2150 I made a mistake. And updated the jenkins command again. @andreaTP Could you close/reopen once more? :-) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-20 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 Done, let see the result! --- 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

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2150 I updated Jenkins build command. @andreaTP can you close / reopen this PR? Jenkins now will check https://travis-ci.org/nokia/zeppelin/builds/213131116 correctly. --- If your project is set

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2150 Looks like Jenkins build script still does not handle contribution from organization repository very well. Let me try fix this. --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-20 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 no luck again :-( --- 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

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-20 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 @Leemoonsoo rebased. Unfortunately again failures on CI, this is on Jenkins side I guess https://builds.apache.org/job/zeppelin-pull-request/580/console this is spurious since travis

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2150 @andreaTP Problem on CI just has fixed on master branch. Could you try rebase or merge master branch and see if CI passes? --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-20 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 probably you have a problem on Jenkins, Travis is currently building without errors: https://travis-ci.org/nokia/zeppelin still this PR is marked as Failed... --- If your project

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-20 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 @AhyoungRyu rebased on top of current *master* --- 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

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2150 It might be due to flaky tests. If so, please just restart failed jobs. I'v just looked your travis build and it's ok - https://travis-ci.org/andreaTP/zeppelin/builds/212066501

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 Thanks for the feedback @1ambda , travis should be configured properly now, I will try to dig into but it looks tests spuriously fail, anyhow any help on getting the right direction is highly

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2150 Hi @andreaTP. Thanks for contribution. - Zeppelin build system relies on travis. So please setup your travis account and travis project for Zeppelin. -

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 failures looks unrelated ... --- 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