[GitHub] cloudstack pull request: travis: use ipmitool from ubuntu reposito...

2016-05-29 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1570#issuecomment-222416264 @swill can you merge this, as it would fix the ipmitool issue. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request:

2016-05-29 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/commit/8af677a0f0b2a74db74223fc3d0e2d3e9549e960#commitcomment-17662189 In pom.xml: In pom.xml on line 91: @pdion891 I think it's a grey area on where to host the noredist jars, also I don't

[GitHub] cloudstack pull request: fix noredist build because of missing mav...

2016-05-29 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1569#issuecomment-222407957 Fixed... You can resume testing `master` @pdion891. Cheers... --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: fix noredist build because of missing mav...

2016-05-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1569 --- 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

[GitHub] cloudstack pull request: Update JuniperSrxResource.java

2016-05-29 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1148#issuecomment-222390201 ### ACS CI BVT Run **Sumarry:** Build Number 84 Hypervisor xenserver NetworkType Advanced Passed=73 Failed=0 Skipped=3

[GitHub] cloudstack pull request:

2016-05-29 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/commit/8af677a0f0b2a74db74223fc3d0e2d3e9549e960#commitcomment-17660284 In pom.xml: In pom.xml on line 91: https://github.com/CloudStack-extras --- If your project is set up for it, you

[GitHub] cloudstack pull request:

2016-05-29 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/commit/8af677a0f0b2a74db74223fc3d0e2d3e9549e960#commitcomment-17660235 In pom.xml: In pom.xml on line 91: I was not aware of a cloudstack extra org, but I like that idea. Once I get our main

[GitHub] cloudstack pull request: travis: use ipmitool from ubuntu reposito...

2016-05-29 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/1570#issuecomment-222373316 seams to be fixing travis and only related to travis so LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: fix noredist build because of missing mav...

2016-05-29 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/1569#issuecomment-222373191 @rhtyd why not update the main "deps/install-non-oss.sh" instead of duplicating it in another repo? Also, I think we are due to update the documentation on build

[GitHub] cloudstack pull request:

2016-05-29 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/commit/8af677a0f0b2a74db74223fc3d0e2d3e9549e960#commitcomment-17660217 In pom.xml: In pom.xml on line 91: 👍 Rohit for that git repo. shouldn't it be part le the cloudstack-extra org.

[GitHub] cloudstack pull request:

2016-05-29 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/commit/8af677a0f0b2a74db74223fc3d0e2d3e9549e960#commitcomment-17659126 In pom.xml: In pom.xml on line 91: Since noredist required additional jars, recommended approach is to git clone this

[GitHub] cloudstack pull request: travis: use ipmitool from ubuntu reposito...

2016-05-29 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1570 travis: use ipmitool from ubuntu repository @swill @pdion891 please review, merge this once Travis is green. The packages.shapeblue.com server is unreachable which is causing a wget command to

[GitHub] cloudstack pull request: fix noredist build because of missing mav...

2016-05-29 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1569#issuecomment-222354132 LGTM --- 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

[GitHub] cloudstack pull request: fix noredist build because of missing mav...

2016-05-29 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1569#issuecomment-222354130 @pdion891 usually, I clone this repo: https://github.com/rhtyd/cloudstack-nonoss and execute the install-non-oss.sh script in that repo:

[GitHub] cloudstack pull request: fix noredist build because of missing mav...

2016-05-29 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1569#issuecomment-222354088 @swill there is a ipmitool pkg that needs to be installed before each run. I'll send a PR for a workaround. The packages.shapeblue.com repository is down, which