[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 we'll chase with BT 
who is hosting the server on Monday.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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: 
https://github.com/rhtyd/cloudstack-nonoss/blob/master/install-non-oss.sh


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 
fail. This fixes this by installing `ipmitool` from ubuntu repositories.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shapeblue/cloudstack travis-ipmitool-remove

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1570.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 #1570


commit 91c9fc9f6ef5c09e788c20dececd0ee4d7325d29
Author: Rohit Yadav 
Date:   2016-05-29T10:44:28Z

travis: use ipmitool from ubuntu repository

Signed-off-by: Rohit Yadav 




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 repo https://github.com/rhtyd/cloudstack-nonoss and execute the 
install-non-oss.sh script in that repo. I've been maintaining this for build 
purposes since 4.3.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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.  Did create #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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 ACS with noredist. It's an oudated documentation :-S 




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 cloudstack repo moved we can figure out how to handle what we can put 
in the apachecloudstack org and maybe we use another org for stuff that we are 
not allowed to put inside apache (now apachecloudstack).  More discussion 
needed on this I think. 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**

**Skipped tests:**
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_vpc_vpn.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_routers.py
test_reset_vm_on_reboot.py
test_snapshots.py
test_deploy_vms_with_varied_deploymentplanners.py
test_login.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 have access to the above org. I'm open to transferring this repository 
to any org if needed. I can maintain it there as well.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9385 - Ensure passwd srvr is r...

2016-05-29 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1568#issuecomment-222421440
  
@remibergsma is correct, password server should not be running on router on 
BACKUP/FAULT.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---