[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1847
  
@blueorangutan package


---
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 issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1847
  
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.


---
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 issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

[ustcweizhou]

Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/1915
  
1. As @dmabry said, the logrotate is moved from cron.daily to cron.hourly 
in isolated network VR in commit.
We should make similar change for VPC VR and VR for shared network.
```
commit 6ac06e5e5e3ceed4a3e3a86ea5f82ffb59c266f2
Author: Abhinandan Prateek 
AuthorDate: Mon Mar 30 11:36:11 2015 +0530
Commit: Abhinandan Prateek 
CommitDate: Mon Mar 30 13:49:23 2015 +0530

CLOUDSTACK-6885: rotate logs hourly on VR
---
 .../debian/config/etc/init.d/cloud-early-config|4 
 systemvm/patches/debian/config/etc/logrotate.conf  |2 +-
 .../patches/debian/config/etc/logrotate.d/cloud|3 ---
 .../patches/debian/config/etc/logrotate.d/rsyslog  |2 ++
 4 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/systemvm/patches/debian/config/etc/init.d/cloud-early-config 
b/systemvm/patches/debian/config/etc/init.d/cloud-early-config
index 729b58f..5a2ee6f 100755
--- a/systemvm/patches/debian/config/etc/init.d/cloud-early-config
+++ b/systemvm/patches/debian/config/etc/init.d/cloud-early-config
@@ -995,6 +995,10 @@ setup_router() {
   sed -i "s/-A INPUT -i eth0 -p udp -m udp --dport 53 -j ACCEPT/-A INPUT 
-i eth0 -p udp -m udp --dport 53 -s $DHCP_RANGE\/$CIDR_SIZE -j ACCEPT/g" 
/etc/iptables/rules
   sed -i "s/-A INPUT -i eth0 -p tcp -m tcp --dport 53 -j ACCEPT/-A INPUT 
-i eth0 -p tcp -m tcp --dport 53 -s $DHCP_RANGE\/$CIDR_SIZE -j ACCEPT/g" 
/etc/iptables/rules.v4
   sed -i "s/-A INPUT -i eth0 -p tcp -m tcp --dport 53 -j ACCEPT/-A INPUT 
-i eth0 -p tcp -m tcp --dport 53 -s $DHCP_RANGE\/$CIDR_SIZE -j ACCEPT/g" 
/etc/iptables/rules
+
+  #setup hourly logrotate
+  mv -n /etc/cron.daily/logrotate /etc/cron.hourly 2>&1
+
 }
```

2. size and maxsize have no difference in my testing. Anyway, maxsize is ok 
for me.

3. As I said in previous comment, we'd better use more 'rotate' for 
cloud.log

4. @dmabry If you are not confident with hourly file compress, we can add a 
file in /etc/cron.d/
```
# cat /etc/cron.d/logrorate.cloud
* * * * * root /usr/sbin/logrotate /etc/logrotate.d/cloud 2>&1 > /dev/null
```


---
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 issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
I agree with @serg38 and @ustcweizhou that upon deletion user should be 
displayed with an yes/no dialogue if there are deployed VMs, but a lot of CS 
user aren't using UI, but just the APIs. If the force option is enabled by 
default I guess the API call will go and delete the all selected templates 
without any warning. I think the `force=true` should only stay with the UI 
call. 


---
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 #1937: CLOUDSTACK-9779 : Releasing secondary guest I...

[niteshsarda]

Github user niteshsarda commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1937#discussion_r101966427
  
--- Diff: server/src/com/cloud/network/NetworkServiceImpl.java ---
@@ -852,7 +852,8 @@ public boolean releaseSecondaryIpFromNic(long 
ipAddressId) {
 throw new InvalidParameterValueException("Can' remove the 
ip " + secondaryIp + "is associate with static NAT rule public IP address id " 
+ publicIpVO.getId());
 }
 
-if (_lbService.isLbRuleMappedToVmGuestIp(secondaryIp)) {
+List lbRuleIdList = 
_firewallDao.listIdByNetworkAndPurposeAndNotRevoked(network.getId(), 
Purpose.LoadBalancing);
--- End diff --

@ustcweizhou  : As per your comment, do you expect Null and Empty check for 
list variable "lbRuleIdList " to be moved in LoadBalancerVMMapDao.java class ?


---
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 issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1847
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-506


---
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 issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1847
  
@blueorangutan test


---
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 issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1847
  
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


---
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 #1937: CLOUDSTACK-9779 : Releasing secondary guest I...

[ustcweizhou]

Github user ustcweizhou commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1937#discussion_r101968821
  
--- Diff: server/src/com/cloud/network/NetworkServiceImpl.java ---
@@ -852,7 +852,8 @@ public boolean releaseSecondaryIpFromNic(long 
ipAddressId) {
 throw new InvalidParameterValueException("Can' remove the 
ip " + secondaryIp + "is associate with static NAT rule public IP address id " 
+ publicIpVO.getId());
 }
 
-if (_lbService.isLbRuleMappedToVmGuestIp(secondaryIp)) {
+List lbRuleIdList = 
_firewallDao.listIdByNetworkAndPurposeAndNotRevoked(network.getId(), 
Purpose.LoadBalancing);
--- End diff --

@niteshsarda sorry what I said is not clear. You added two methods in Daos: 
listIdByNetworkAndPurposeAndNotRevoked and 
listByLoadBalancerIdVmIdAndInstanceIp, I suggest to implement it by one method 
in LoadBalancerDaoImpl.java


---
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 #1874: CLOUDSTACK-9711: Fixed error reporting while ...

[jayapalu]

Github user jayapalu closed the pull request at:

https://github.com/apache/cloudstack/pull/1874


---
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 issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[priyankparihar]

Github user priyankparihar commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
Hi @ustcweizhou @serg38 and @borisroman,
Thanks for your valuable suggestions. 

> @ustcweizhou  the default value of forced is false, might cause issue on 
backwards compatibility.

If the force option is enabled by default API call will go and delete the 
all selected templates without any warning so default value of forced is kept 
'false'.

> @ustcweizhou In the vm installation, as the template is still in use , so 
we can find it in the storage pool where the root disk is located. We need to 
improve it by using the existing template image on storage pool, or copying the 
template from storage pool where the root disk is located to another new 
storage pool (pool of allocated new root disk).

I think it will be good to take it in separate PR. 


---
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 issue #1379: CLOUDSTACK-8324: config drive data set/get scripts f...

[kishankavala]

Github user kishankavala commented on the issue:

https://github.com/apache/cloudstack/pull/1379
  
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 #1824: CLOUDSTACK-9657: Fixed security group ipset i...

[kishankavala]

Github user kishankavala commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1824#discussion_r101981007
  
--- Diff: scripts/vm/hypervisor/xenserver/vmops ---
@@ -232,28 +233,50 @@ def deleteFile(session, args):
 
 return txt
 
+#using all the iptables chain names length to 24 because cleanup_rules 
groups the vm chain excluding -def,-eg
+#to avoid multiple iptables chains for single vm there using length 24
 def chain_name(vm_name):
 if vm_name.startswith('i-') or vm_name.startswith('r-'):
 if vm_name.endswith('untagged'):
 return '-'.join(vm_name.split('-')[:-1])
 if len(vm_name) > 28:
--- End diff --

Check should be > 25


---
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 #1922: CLOUDSTACK-9757: Fixed issue in traffic from ...

[ramkatru]

Github user ramkatru commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1922#discussion_r101983289
  
--- Diff: server/src/com/cloud/network/router/CommandSetupHelper.java ---
@@ -703,6 +722,10 @@ public void createVpcAssociatePublicIPCommands(final 
VirtualRouter router, final
 sourceNatIpAdd = new Pair(ip, 
ipAddr.getNetworkId());
 addSourceNat = add;
 }
+
+if (!firstIP || add) {
+firstIP = false;
+}
--- End diff --

Please add a comment explaining this. Not very readable.



---
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 issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

[kishankavala]

Github user kishankavala commented on the issue:

https://github.com/apache/cloudstack/pull/1873
  
Code looks good.
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 issue #1885: CLOUDSTACK-9724: Fixed missing additional public ip ...

[jayapalu]

Github user jayapalu commented on the issue:

https://github.com/apache/cloudstack/pull/1885
  
Failed test cases are not related to this PR code changes


---
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 issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[priyankparihar]

Github user priyankparihar commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
Hi @ustcweizhou,
By default enabling force option(force=true) will be very risky, same is 
mentioned by @borisstoyanov.


---
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 issue #1779: CLOUDSTACK-9610: Disabled Host Keeps Being up status...

[priyankparihar]

Github user priyankparihar commented on the issue:

https://github.com/apache/cloudstack/pull/1779
  
Hi Bhai Sahab @rhtyd,
Integration test 'test_04_disable_host_unmanage_cluster_check_hosts_status' 
is validating this behavior.


---
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 #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/1847


---
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 #1758: CLOUDSTACK-9588: Add Load Balancer functional...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/1758


---
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 #1333: CLOUDSTACK-9228: Network update with mistmatc...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/1333


---
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 issue #1891: CLOUDSTACK-9730: [VMware] Unable to add a host with ...

[sureshanaparti]

Github user sureshanaparti commented on the issue:

https://github.com/apache/cloudstack/pull/1891
  
@rhtyd Can you kick off VMware CI on this PR.


---
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 issue #1880: CLOUDSTACK-9720: [VMware] template_spool_ref table i...

[sureshanaparti]

Github user sureshanaparti commented on the issue:

https://github.com/apache/cloudstack/pull/1880
  
@rhtyd Can you run VMware CI on this PR.


---
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 issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1953
  
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.


---
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 issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1953
  
@blueorangutan package


---
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 issue #1875: CLOUDSTACK-8608: [VMware] System VMs failed to start...

[sureshanaparti]

Github user sureshanaparti commented on the issue:

https://github.com/apache/cloudstack/pull/1875
  
@rhtyd Can you run VMware CI on this PR.


---
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 issue #1879: CLOUDSTACK-9719: [VMware] VR loses DHCP settings and...

[sureshanaparti]

Github user sureshanaparti commented on the issue:

https://github.com/apache/cloudstack/pull/1879
  
@rhtyd Can you kick off VMware CI on this PR.


---
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 issue #1879: CLOUDSTACK-9719: [VMware] VR loses DHCP settings and...

[rhtyd]

Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1879
  
Sure. 
@blueorangutan package 


---
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 issue #1879: CLOUDSTACK-9719: [VMware] VR loses DHCP settings and...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1879
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
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 issue #1875: CLOUDSTACK-8608: [VMware] System VMs failed to start...

[rhtyd]

Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1875
  
@blueorangutan package 


---
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 issue #1891: CLOUDSTACK-9730: [VMware] Unable to add a host with ...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1891
  
@blueorangutan package


---
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 issue #1891: CLOUDSTACK-9730: [VMware] Unable to add a host with ...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1891
  
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.


---
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 issue #1880: CLOUDSTACK-9720: [VMware] template_spool_ref table i...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1880
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
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 issue #1880: CLOUDSTACK-9720: [VMware] template_spool_ref table i...

[rhtyd]

Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1880
  
@blueorangutan package 


---
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 issue #1875: CLOUDSTACK-8608: [VMware] System VMs failed to start...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1875
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
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 issue #1891: CLOUDSTACK-9730: [VMware] Unable to add a host with ...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1891
  
@sureshanaparti since we're making changes of the return type can we also 
create/edit unit tests? 


---
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 issue #1862: CLOUDSTACK-9704 Remove dependency on VmwareContext o...

[sateesh-chodapuneedi]

Github user sateesh-chodapuneedi commented on the issue:

https://github.com/apache/cloudstack/pull/1862
  
@borisstoyanov Can you please run trillian tests for this PR?
This already has enough code LGTMs, and needs tests for merge.


---
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 issue #1841: CLOUDSTACK-9684 Invalid zone id error while listing ...

[sateesh-chodapuneedi]

Github user sateesh-chodapuneedi commented on the issue:

https://github.com/apache/cloudstack/pull/1841
  
@borisstoyanov Can you please run trillian tests for this PR?
This already has enough code LGTMs, and needs tests for merge.


---
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 issue #1872: CLOUDSTACK-3223 Exception observed while creating CP...

[sateesh-chodapuneedi]

Github user sateesh-chodapuneedi commented on the issue:

https://github.com/apache/cloudstack/pull/1872
  
@borisstoyanov Can you please run trillian tests for this PR?


---
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 issue #1861: CLOUDSTACK-9698 [VMware] Make hardcorded wait timeou...

[sateesh-chodapuneedi]

Github user sateesh-chodapuneedi commented on the issue:

https://github.com/apache/cloudstack/pull/1861
  
@borisstoyanov Can you please run trillian tests for this PR?


---
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 issue #1841: CLOUDSTACK-9684 Invalid zone id error while listing ...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1841
  
@blueorangutan package


---
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 issue #1841: CLOUDSTACK-9684 Invalid zone id error while listing ...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1841
  
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.


---
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 issue #1861: CLOUDSTACK-9698 [VMware] Make hardcorded wait timeou...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1861
  
@blueorangutan package


---
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 issue #1861: CLOUDSTACK-9698 [VMware] Make hardcorded wait timeou...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1861
  
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.


---
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 issue #843: Security group ingress/egress issues with xenserver 6...

[jayapalu]

Github user jayapalu commented on the issue:

https://github.com/apache/cloudstack/pull/843
  
@rhtyd  nethash is supported in xenserver 6.2 but the problem is when 
configure /32 cidr giving the error ' Out of range cidr `10.147.52.30/32' 
specified'. Please see the bug for more details.

I have also update the ipset version check. If version less than 6 use 
iptreemap else nethash.


---
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 issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1953
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-507


---
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 issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[ustcweizhou]

Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
@priyankparihar the default behaviour is forced=true from few years ago. If 
the template is removed, the user vm is not impacted which is very good. There 
is indeed an issue when reinstall the vm. For you it could be a very risky 
issue. For us at least it is not a big issue as we provide some other templates 
for vm reinstallation.


---
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 issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1953
  
@blueorangutan test


---
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 issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1953
  
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


---
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 issue #1872: CLOUDSTACK-3223 Exception observed while creating CP...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1872
  
@blueorangutan package


---
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 issue #1872: CLOUDSTACK-3223 Exception observed while creating CP...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1872
  
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.


---
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.
---

[PROPOSAL] add native vm-cluster orchestration service (was: [PROPOSAL] add native container orchestration service)

[Daan Hoogland]

So, being very late in the discussion but havingread the whole thread before 
editting the title of this thread,

Can we agree that we want a generic vm-cluster service and leave the container 
bits to containers? Kishan can you share your design? Shapeblue wants to rebase 
their k8 service on top of this and I would like yours and Murali's work to not 
conflict.

daan.hoogl...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, Utrecht Utrecht 3531 VENetherlands
@shapeblue
  
 


-Original Message-
From: Paul Angus [mailto:paul.an...@shapeblue.com] 
Sent: dinsdag 7 februari 2017 08:14
To: dev@cloudstack.apache.org
Subject: Re: [PROPOSAL] add native container orchestration service

Will is 100% correct.  As I mentioned the Title is misleading.  However, Murali 
did clarify in his explanation; this is really about vm cluster orchestration.




From: Will Stevens 
Sent: 6 Feb 2017 22:54
To: dev@cloudstack.apache.org
Subject: Re: [PROPOSAL] add native container orchestration service

​My understanding is that what Paul is talking about is what is already built 
and IS what the thread is talking about.​

*Will STEVENS*
Lead Developer



On Mon, Feb 6, 2017 at 2:29 PM, Rajesh Ramchandani < 
rajesh.ramchand...@accelerite.com> wrote:

> Hi Paul - I think this is different from what the thread was about. 
> The conversation was specifically about adding support for container 
> orchestrators. You are talking about provisioning a group of VMs. 
> Although, this is something I think several Cloudstack users have 
> requested before and we should propose a solution to this.
>
> Raj
>
> 
> From: Paul Angus 
> Sent: Monday, February 6, 2017 11:16:41 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [PROPOSAL] add native container orchestration service
>
> #WhatHeSaid
>
> The title is misleading.  The proposal is purely to add the notion of 
> Clusters of VMs to CloudStack.  These may be for databases, containers 
> or anything else that needs 'clusters' of compute. Self-healing and 
> autoscaling are logical next steps to be added.
>
> Those guys at ShapeBlue have open-sourced their whole k8s container 
> service piece.  If/when the 'cluster' part of that work is added into 
> CloudStack, the k8s specific pieces can be used by anyone who wants 
> to, alternatively they could be used for reference in order to create 
> another types of cluster.  (or ignored completely).
>
>
>
>
> paul.an...@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK @shapeblue
>
>
>
>
> -Original Message-
> From: Will Stevens [mailto:williamstev...@gmail.com]
> Sent: 31 January 2017 13:26
> To: dev@cloudstack.apache.org
> Subject: Re: [PROPOSAL] add native container orchestration service
>
> s/cloud-init/cloud-config/
>
> On Jan 31, 2017 7:24 AM, "Will Stevens"  wrote:
>
> > I think that is covered in this proposal. There is nothing k8s
> > specific in this integration (from what I understand), all the k8s
> > details are passed in via the cloud-init configuration after the cluster
> has been provisioned.
> >
> > On Jan 31, 2017 3:06 AM, "Lianghwa Jou" 
> > wrote:
> >
> >
> > There are many container orchestrators. Those container orchestrators
> > are happy to run on any VMs or bare metal machines. K8s is just one of
> > them and there will be more in the future. It may not be a good idea
> > to make CloudStack to be k8s aware. IMO, the relationship between k8s
> > and cloudstack should be similar to application and os. It probably
> > not a good idea to make your OS to be aware of any specific
> > applications so IMHO I don’t think k8s should be native to CloudStack.
> > It makes more sense to provide some generic services that many
> > applications can take advantages of. Some generic resource grouping
> > service makes sense so a group of VMs, baremetal machines or network
> > can be treated as a single entity. Some life cycle management will be
> > necessary for these entities too. We can deploy k8s, swarm, dcos or
> > even non-container specific services on top of CloudStack. The k8s is
> > changing fast. One single tenant installation may need more than one
> > VM group and may actually requires more (k8s federation). It will be a
> > struggle to be in sync if we try to bring k8s specific knowledge into
> cloudstack.
> >
> > Regards,
> >
> >
> > --
> > Lianghwa Jou
> > VP Engineering, Accelerite
> > email: lianghwa@accelerite.com
> >
> >
> >
> >
> >
> > On 1/29/17, 11:54 PM, "Murali Reddy"  wrote:
> >
> >
> > I agree with some good views Will has shared and I also agree to
> > the concerns raised by Wido and Eric. IMO we need balance of staying
> > relevant/add new features vs stability of CloudStack and take
> > corrective action if needed. We have two good examples for both. When
> > SDN was hot technology CloudStack ended up with bunch of SDN controll

[GitHub] cloudstack issue #1879: CLOUDSTACK-9719: [VMware] VR loses DHCP settings and...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1879
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-508


---
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 issue #1891: CLOUDSTACK-9730: [VMware] Unable to add a host with ...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1891
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-510


---
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 issue #1880: CLOUDSTACK-9720: [VMware] template_spool_ref table i...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1880
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-509


---
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 issue #1875: CLOUDSTACK-8608: [VMware] System VMs failed to start...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1875
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-511


---
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 issue #1841: CLOUDSTACK-9684 Invalid zone id error while listing ...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1841
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-512


---
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 issue #1861: CLOUDSTACK-9698 [VMware] Make hardcorded wait timeou...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1861
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-513


---
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 issue #1872: CLOUDSTACK-3223 Exception observed while creating CP...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1872
  
Packaging result: ✔centos6 ✔centos7 ✖debian. JID-514


---
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 #1954: CLOUDSTACK-9795: moved logrotate from cron.da...

[dmabry]

GitHub user dmabry opened a pull request:

https://github.com/apache/cloudstack/pull/1954

CLOUDSTACK-9795: moved logrotate from cron.daily to cron.hourly for 
vpcrouter

 in cloud-early-config

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

$ git pull https://github.com/myENA/cloudstack bug/vpc-vr-logrotate

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

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


commit 450deed8472c06c4fec88e045bef28091502c638
Author: David Mabry 
Date:   2017-02-20T15:06:46Z

moved logrotate from cron.daily to cron.hourly for vpcrouter in 
cloud-early-config




---
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 issue #1954: CLOUDSTACK-9795: moved logrotate from cron.daily to ...

[swill]

Github user swill commented on the issue:

https://github.com/apache/cloudstack/pull/1954
  
LGTM.  This change makes the VPC router behave the same way as the Isolated 
Guest Network router.  eg: 
https://github.com/myENA/cloudstack/blob/450deed8472c06c4fec88e045bef28091502c638/systemvm/patches/debian/config/etc/init.d/cloud-early-config#L959


---
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 issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

[dmabry]

Github user dmabry commented on the issue:

https://github.com/apache/cloudstack/pull/1915
  
I have submitted a PR against master that fixes cloud-early-config.
https://github.com/apache/cloudstack/pull/1954



---
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 issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

[dmabry]

Github user dmabry commented on the issue:

https://github.com/apache/cloudstack/pull/1915
  
@ustcweizhou Thanks for taking the time to test as well.  I confirm in our 
testing over the weekend that we were able to consistently 
create/destroy/expunge VMs every 2 minutes without issue.  Before the PR 
https://github.com/apache/cloudstack/pull/1954 and this PR, we were having the 
VPC VR fall over ~8 hours into the test.

In summary, this PR with the one I submitted this morning solved our issues.

Thanks again for the help.


---
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 issue #1954: CLOUDSTACK-9795: moved logrotate from cron.daily to ...

[kiwiflyer]

Github user kiwiflyer commented on the issue:

https://github.com/apache/cloudstack/pull/1954
  
Tested in our QA environment. We cycled 2800 VMs through a VPC VR pair 
successfully over the weekend. 

The logs are rotating correctly as per the following screenshot:


![image](https://cloud.githubusercontent.com/assets/17278194/23131539/fb06a772-f74f-11e6-9e86-afb6d84fe6a9.png)

LGTM based on testing.


---
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 issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1847
  
Trillian test result (tid-854)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26568 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1847-t854-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py
Test completed. 46 look ok, 2 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_04_rvpc_privategw_static_routes | `Failure` | 315.29 | 
test_privategw_acl.py
test_02_list_snapshots_with_removed_data_store | `Error` | 0.04 | 
test_snapshots.py
test_01_vpc_site2site_vpn | Success | 140.99 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 66.12 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 220.68 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 284.23 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 513.11 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 503.72 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1404.04 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 538.71 | test_vpc_redundant.py
test_02_redundant_VPC_default_routes | Success | 741.04 | 
test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1286.74 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 151.44 | test_volumes.py
test_08_resize_volume | Success | 156.61 | test_volumes.py
test_07_resize_fail | Success | 156.51 | test_volumes.py
test_06_download_detached_volume | Success | 157.60 | test_volumes.py
test_05_detach_volume | Success | 151.18 | test_volumes.py
test_04_delete_attached_volume | Success | 151.37 | test_volumes.py
test_03_download_attached_volume | Success | 156.77 | test_volumes.py
test_02_attach_volume | Success | 94.64 | test_volumes.py
test_01_create_volume | Success | 621.09 | test_volumes.py
test_deploy_vm_multiple | Success | 252.67 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.03 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.80 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.25 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 40.93 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.13 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 130.86 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 126.01 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.18 | test_vm_life_cycle.py
test_01_stop_vm | Success | 35.33 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 75.64 | test_templates.py
test_08_list_system_templates | Success | 0.03 | test_templates.py
test_07_list_public_templates | Success | 0.04 | test_templates.py
test_05_template_permissions | Success | 0.06 | test_templates.py
test_04_extract_template | Success | 5.16 | test_templates.py
test_03_delete_template | Success | 5.13 | test_templates.py
test_02_edit_template | Success | 90.18 | test_templates.py
test_01_create_template | Success | 25.33 | test_templates.py
test_10_destroy_cpvm | Success | 161.44 | test_ssvm.py
test_09_destroy_ssvm | Success | 163.81 | test_ssvm.py
test_08_reboot_cpvm | Success | 101.37 | test_ssvm.py
test_07_reboot_ssvm | Success | 133.56 | test_ssvm.py
test_06_stop_cpvm | Success | 131.60 | test_ssvm.py
test_05_stop_ssvm | Success | 133.67 | test_ssvm.py
test_04_cpvm_internals | Success | 1.02 | test_ssvm.py
test_03_ssvm_internals | Success | 3.29 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.15 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.13 | test_ssvm.py
test_01_snapshot_root_disk | Success | 11.10 | test_snapshots.py
test_04_change_offering_small | Success | 210.81 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.05 | test_service_offerings.py
test_01_create_service_offering | Success | 0.11 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.13 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.18 | test_secondary_storage.py
test_09_reboot_router | Success | 40.33 | test_routers.py
test_08_start_router | Success | 30.28 | test_routers.py
test_07_stop_router | Success | 10.16 | test_routers.py
test_06_router_advanced | Success | 0.06 | test_routers.py
test_05_router_basic | Success | 0.04 | test_routers.py
test_04_restart_network_wo_clean

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[serg38]

Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
@ustcweizhou @priyankparihar @borisstoyanov The default behavior was always 
equivalent of force=true. We seem to agree that the UI behavior should remain 
forced=true but with extra warning. We can either change default  confirmation 
"Please confirm that you want to delete this template." e.g. to "Please confirm 
that you want to delete this template.  Features that depends on the template 
presence might no longer work". But I think the better option will be to have 
response tag in listTemplate call to indicate if active VMs exists or not. Then 
warning can be given only to such templates.
As for API side we might still change default behavior but we need to 
explicitly document it in Release Notes so users can get prepared and change 
instances of deleteTemplate execution to use forced=true or not as needed.



---
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 #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for K...

[nathanejohnson]

GitHub user nathanejohnson opened a pull request:

https://github.com/apache/cloudstack/pull/1955

CLOUDSTACK-8239 Add VirtIO SCSI support for KVM hosts

This is an initial implementation that we have working in the lab.


![devmanagerscreenshot](https://cloud.githubusercontent.com/assets/1360240/23133419/981f89ce-f756-11e6-8d17-9ddae264fca8.png)

![spanned_sceenshot](https://cloud.githubusercontent.com/assets/1360240/23133418/981e55cc-f756-11e6-9fd7-f7757c82438c.png)



[win_virtio_scsi.xml.zip](https://github.com/apache/cloudstack/files/788014/win_virtio_scsi.xml.zip)

@wido @syed Would love to get some feedback on this approach.

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

$ git pull https://github.com/myENA/cloudstack virtio-scsi

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

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


commit d6d1cb20cd6f9984217a09c103610ffb2d4e6cf4
Author: Simon Weller 
Date:   2017-02-07T06:13:12Z

Added support for ViRTIO-SCSI based on template name.

commit 658a5e1b1ade2f936061695dd89ed8828751a3c2
Author: Nathan Johnson 
Date:   2017-02-08T17:02:13Z

CLOUDSTACK-8239 - Adding support for virtio-scsi on KVM hosts

This adds support for virtio-scsi on KVM hosts, but currently
only for guests that are associated with a new os_type of
'Other PV Virtio-SCSI (64-bit)'.  This is to prevent device
name changes on existing templates / VMs.




---
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 issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

[serg38]

Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1847
  
@nvazquez @rhtyd @anshul1886 We might need to tweak the test_data.py to add 
additional nfs mount e.g. nfs1 and use it in this test.  E.g adding 
 "nfs1": {
"url": "nfs://nfs/export/automation/1/testprimary1",
"name": "Primary XEN1"
},
Blueorangutain environment will need to be adjusted as well. Otherwise the 
test fails with 
CloudstackAPIException: Execute cmd: createstoragepool failed, due to: 
errorCode: 530, errorText:Failed to add data store: Storage pool 
NFS://10.2.0.16/acs/primary/pr1847-t854-kvm-centos7/marvin_pri1 already in use 
by another pod (id=1)\n']



---
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 issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

[serbaut]

Github user serbaut commented on the issue:

https://github.com/apache/cloudstack/pull/1915
  
@dmabry yes we are using redundant VPC routers. 

Using `size` instead of `maxsize` would delay the rotation of logs until 
they reach `size`. 10 logfiles with size 10MB could consume 200MB due to 
deplaycompress. Using `maxsize` would rotate the slowly growing logfiles daily 
and avoid this issue.


---
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.
---

[QUESTION] Upgrade path to JDK8

[Pierre-Luc Dion]

Hi,

In the context of deployment of CloudStack with VPCs,
What will happen to a cloud when upgrading to 4.10 that now use jdk8?

Does upgrading the management-server to 4.10 jdk8 and then keep the old VRs
run for a while that run on JDK7 will still work ?

Because If we upgrade the management-server to jdk8, we need to keep VR to
work until they get upgraded but we can't force an upgrade of VR just
because the management-server is now using JDK8.

Thanks,

Re: [QUESTION] Upgrade path to JDK8

[Will Stevens]

A question that is hidden in here is:
- Why does the JDK version on the management server have to match the JDK
version of the System VM?

*Will STEVENS*
Lead Developer



On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion 
wrote:

> Hi,
>
> In the context of deployment of CloudStack with VPCs,
> What will happen to a cloud when upgrading to 4.10 that now use jdk8?
>
> Does upgrading the management-server to 4.10 jdk8 and then keep the old VRs
> run for a while that run on JDK7 will still work ?
>
> Because If we upgrade the management-server to jdk8, we need to keep VR to
> work until they get upgraded but we can't force an upgrade of VR just
> because the management-server is now using JDK8.
>
> Thanks,
>

[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...

[kiwiflyer]

Github user kiwiflyer commented on the issue:

https://github.com/apache/cloudstack/pull/1955
  
@remibergsma @ustcweizhou This might be of interest to you guys 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.
---

Re: [QUESTION] Upgrade path to JDK8

[Ron Wheeler]

Are any Java enhancements being added to common libraries that would 
force the Client side to have to run Java 8?
Would running Java 7 cause any library to need to be available in 2 
versions?


Ron

On 20/02/2017 11:58 AM, Will Stevens wrote:

A question that is hidden in here is:
- Why does the JDK version on the management server have to match the JDK
version of the System VM?

*Will STEVENS*
Lead Developer



On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion 
wrote:


Hi,

In the context of deployment of CloudStack with VPCs,
What will happen to a cloud when upgrading to 4.10 that now use jdk8?

Does upgrading the management-server to 4.10 jdk8 and then keep the old VRs
run for a while that run on JDK7 will still work ?

Because If we upgrade the management-server to jdk8, we need to keep VR to
work until they get upgraded but we can't force an upgrade of VR just
because the management-server is now using JDK8.

Thanks,




--
Ron Wheeler
President
Artifact Software Inc
email: rwhee...@artifact-software.com
skype: ronaldmwheeler
phone: 866-970-2435, ext 102

Re: [QUESTION] Upgrade path to JDK8

[Will Stevens]

Regarding my question. Is it because of the version of Java that the
systemvm.iso is built on?

On Feb 20, 2017 11:58 AM, "Will Stevens"  wrote:

> A question that is hidden in here is:
> - Why does the JDK version on the management server have to match the JDK
> version of the System VM?
>
> *Will STEVENS*
> Lead Developer
>
> 
>
> On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion 
> wrote:
>
>> Hi,
>>
>> In the context of deployment of CloudStack with VPCs,
>> What will happen to a cloud when upgrading to 4.10 that now use jdk8?
>>
>> Does upgrading the management-server to 4.10 jdk8 and then keep the old
>> VRs
>> run for a while that run on JDK7 will still work ?
>>
>> Because If we upgrade the management-server to jdk8, we need to keep VR to
>> work until they get upgraded but we can't force an upgrade of VR just
>> because the management-server is now using JDK8.
>>
>> Thanks,
>>
>
>

[GitHub] cloudstack issue #1954: CLOUDSTACK-9795: moved logrotate from cron.daily to ...

[ustcweizhou]

Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/1954
  
quite simple and useful change. 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 issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

[ustcweizhou]

Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/1915
  
@serbaut maxsize is fine for me.
Could you please remove delaycompress, and increase rotate in 
systemvm/patches/debian/config/etc/logrotate.d/cloud (eg from 4 to 10) ?
LTGM after that.


---
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.
---

Re: [QUESTION] Upgrade path to JDK8

[Wei ZHOU]

We've tested management server 4.7.1 with ubuntu 16.04/openjdk8 and
systemvm 4.6 with debian7/openjdk7.
The systemvms (ssvm, cpvm) work fine.

I agree we need consider the openjdk upgrade in systemvm template.

-Wei

2017-02-20 18:15 GMT+01:00 Will Stevens :

> Regarding my question. Is it because of the version of Java that the
> systemvm.iso is built on?
>
> On Feb 20, 2017 11:58 AM, "Will Stevens"  wrote:
>
> > A question that is hidden in here is:
> > - Why does the JDK version on the management server have to match the JDK
> > version of the System VM?
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > 
> >
> > On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion 
> > wrote:
> >
> >> Hi,
> >>
> >> In the context of deployment of CloudStack with VPCs,
> >> What will happen to a cloud when upgrading to 4.10 that now use jdk8?
> >>
> >> Does upgrading the management-server to 4.10 jdk8 and then keep the old
> >> VRs
> >> run for a while that run on JDK7 will still work ?
> >>
> >> Because If we upgrade the management-server to jdk8, we need to keep VR
> to
> >> work until they get upgraded but we can't force an upgrade of VR just
> >> because the management-server is now using JDK8.
> >>
> >> Thanks,
> >>
> >
> >
>

Re: [QUESTION] Upgrade path to JDK8

[Will Stevens]

Did it work after resetting a VPC or when blowing away the SSVM or CPVM?  I
would not expect the SSVM or the CPVM to come up if the management server
was built with JDK8 and the system vm template is only using JDK7.  Can you
confirm?​

*Will STEVENS*
Lead Developer



On Mon, Feb 20, 2017 at 1:15 PM, Wei ZHOU  wrote:

> We've tested management server 4.7.1 with ubuntu 16.04/openjdk8 and
> systemvm 4.6 with debian7/openjdk7.
> The systemvms (ssvm, cpvm) work fine.
>
> I agree we need consider the openjdk upgrade in systemvm template.
>
> -Wei
>
> 2017-02-20 18:15 GMT+01:00 Will Stevens :
>
> > Regarding my question. Is it because of the version of Java that the
> > systemvm.iso is built on?
> >
> > On Feb 20, 2017 11:58 AM, "Will Stevens"  wrote:
> >
> > > A question that is hidden in here is:
> > > - Why does the JDK version on the management server have to match the
> JDK
> > > version of the System VM?
> > >
> > > *Will STEVENS*
> > > Lead Developer
> > >
> > > 
> > >
> > > On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion 
> > > wrote:
> > >
> > >> Hi,
> > >>
> > >> In the context of deployment of CloudStack with VPCs,
> > >> What will happen to a cloud when upgrading to 4.10 that now use jdk8?
> > >>
> > >> Does upgrading the management-server to 4.10 jdk8 and then keep the
> old
> > >> VRs
> > >> run for a while that run on JDK7 will still work ?
> > >>
> > >> Because If we upgrade the management-server to jdk8, we need to keep
> VR
> > to
> > >> work until they get upgraded but we can't force an upgrade of VR just
> > >> because the management-server is now using JDK8.
> > >>
> > >> Thanks,
> > >>
> > >
> > >
> >
>

Re: [QUESTION] Upgrade path to JDK8

[Syed Ahmed]

The problem is that systemvm.iso is built with java 8 whereas java on the
VR is java 7
On Mon, Feb 20, 2017 at 13:20 Will Stevens  wrote:

> Did it work after resetting a VPC or when blowing away the SSVM or CPVM?  I
> would not expect the SSVM or the CPVM to come up if the management server
> was built with JDK8 and the system vm template is only using JDK7.  Can you
> confirm?​
>
> *Will STEVENS*
> Lead Developer
>
> 
>
> On Mon, Feb 20, 2017 at 1:15 PM, Wei ZHOU  wrote:
>
> > We've tested management server 4.7.1 with ubuntu 16.04/openjdk8 and
> > systemvm 4.6 with debian7/openjdk7.
> > The systemvms (ssvm, cpvm) work fine.
> >
> > I agree we need consider the openjdk upgrade in systemvm template.
> >
> > -Wei
> >
> > 2017-02-20 18:15 GMT+01:00 Will Stevens :
> >
> > > Regarding my question. Is it because of the version of Java that the
> > > systemvm.iso is built on?
> > >
> > > On Feb 20, 2017 11:58 AM, "Will Stevens" 
> wrote:
> > >
> > > > A question that is hidden in here is:
> > > > - Why does the JDK version on the management server have to match the
> > JDK
> > > > version of the System VM?
> > > >
> > > > *Will STEVENS*
> > > > Lead Developer
> > > >
> > > > 
> > > >
> > > > On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion <
> pd...@cloudops.com>
> > > > wrote:
> > > >
> > > >> Hi,
> > > >>
> > > >> In the context of deployment of CloudStack with VPCs,
> > > >> What will happen to a cloud when upgrading to 4.10 that now use
> jdk8?
> > > >>
> > > >> Does upgrading the management-server to 4.10 jdk8 and then keep the
> > old
> > > >> VRs
> > > >> run for a while that run on JDK7 will still work ?
> > > >>
> > > >> Because If we upgrade the management-server to jdk8, we need to keep
> > VR
> > > to
> > > >> work until they get upgraded but we can't force an upgrade of VR
> just
> > > >> because the management-server is now using JDK8.
> > > >>
> > > >> Thanks,
> > > >>
> > > >
> > > >
> > >
> >
>

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
@blueorangutan package


---
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 issue #1872: CLOUDSTACK-3223 Exception observed while creating CP...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1872
  
@blueorangutan test centos6 vmware-60u1


---
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 issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.


---
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 issue #1841: CLOUDSTACK-9684 Invalid zone id error while listing ...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1841
  
@blueorangutan test centos7 vmware-60u2


---
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 issue #1872: CLOUDSTACK-3223 Exception observed while creating CP...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1872
  
@borisstoyanov unsupported parameters provided. Supported mgmt server os 
are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6, 
kvm-centos7, kvm-ubuntu, xenserver-65sp1, xenserver-62sp1, vmware-60u2, 
vmware-55u3, vmware-51u1, vmware-50u1`


---
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 issue #1861: CLOUDSTACK-9698 [VMware] Make hardcorded wait timeou...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1861
  
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-60u2) has 
been kicked to run smoke tests


---
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 issue #1841: CLOUDSTACK-9684 Invalid zone id error while listing ...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1841
  
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-60u2) has 
been kicked to run smoke tests


---
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 issue #1861: CLOUDSTACK-9698 [VMware] Make hardcorded wait timeou...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1861
  
@blueorangutan test centos7 vmware-60u2


---
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 issue #1872: CLOUDSTACK-3223 Exception observed while creating CP...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1872
  
@blueorangutan test centos7 vmware-60u2


---
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 issue #1872: CLOUDSTACK-3223 Exception observed while creating CP...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1872
  
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-60u2) has 
been kicked to run smoke tests


---
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.
---

Re: [QUESTION] Upgrade path to JDK8

[Will Stevens]

yes, that is what I was expecting.  which is why I was asking about Wei's
setup because he seems to have worked around that problem.  Or he has a
custom SystemVM template running with both JDK7 and JDK8.

*Will STEVENS*
Lead Developer



On Mon, Feb 20, 2017 at 2:20 PM, Syed Ahmed  wrote:

> The problem is that systemvm.iso is built with java 8 whereas java on the
> VR is java 7
> On Mon, Feb 20, 2017 at 13:20 Will Stevens  wrote:
>
> > Did it work after resetting a VPC or when blowing away the SSVM or
> CPVM?  I
> > would not expect the SSVM or the CPVM to come up if the management server
> > was built with JDK8 and the system vm template is only using JDK7.  Can
> you
> > confirm?​
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > 
> >
> > On Mon, Feb 20, 2017 at 1:15 PM, Wei ZHOU  wrote:
> >
> > > We've tested management server 4.7.1 with ubuntu 16.04/openjdk8 and
> > > systemvm 4.6 with debian7/openjdk7.
> > > The systemvms (ssvm, cpvm) work fine.
> > >
> > > I agree we need consider the openjdk upgrade in systemvm template.
> > >
> > > -Wei
> > >
> > > 2017-02-20 18:15 GMT+01:00 Will Stevens :
> > >
> > > > Regarding my question. Is it because of the version of Java that the
> > > > systemvm.iso is built on?
> > > >
> > > > On Feb 20, 2017 11:58 AM, "Will Stevens" 
> > wrote:
> > > >
> > > > > A question that is hidden in here is:
> > > > > - Why does the JDK version on the management server have to match
> the
> > > JDK
> > > > > version of the System VM?
> > > > >
> > > > > *Will STEVENS*
> > > > > Lead Developer
> > > > >
> > > > > 
> > > > >
> > > > > On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion <
> > pd...@cloudops.com>
> > > > > wrote:
> > > > >
> > > > >> Hi,
> > > > >>
> > > > >> In the context of deployment of CloudStack with VPCs,
> > > > >> What will happen to a cloud when upgrading to 4.10 that now use
> > jdk8?
> > > > >>
> > > > >> Does upgrading the management-server to 4.10 jdk8 and then keep
> the
> > > old
> > > > >> VRs
> > > > >> run for a while that run on JDK7 will still work ?
> > > > >>
> > > > >> Because If we upgrade the management-server to jdk8, we need to
> keep
> > > VR
> > > > to
> > > > >> work until they get upgraded but we can't force an upgrade of VR
> > just
> > > > >> because the management-server is now using JDK8.
> > > > >>
> > > > >> Thanks,
> > > > >>
> > > > >
> > > > >
> > > >
> > >
> >
>

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-515


---
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 issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[borisstoyanov]

Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
@blueorangutan test


---
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 issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

[blueorangutan]

Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1773
  
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


---
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 issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...

[remibergsma]

Github user remibergsma commented on the issue:

https://github.com/apache/cloudstack/pull/1955
  
Nice work @kiwiflyer @dmabry, thanks! Will give it a go soon.


---
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.
---

Re: [QUESTION] Upgrade path to JDK8

[Chiradeep Vittal]

You can build the system vm with  -source 1.7 -target 1.7 
Also unless you are using Java8 features (lambda) the classfiles produced by 
javac 8 should work in a 1.7 JVM

Sent from my iPhone

> On Feb 20, 2017, at 11:51 AM, Will Stevens  wrote:
> 
> yes, that is what I was expecting.  which is why I was asking about Wei's
> setup because he seems to have worked around that problem.  Or he has a
> custom SystemVM template running with both JDK7 and JDK8.
> 
> *Will STEVENS*
> Lead Developer
> 
> 
> 
>> On Mon, Feb 20, 2017 at 2:20 PM, Syed Ahmed  wrote:
>> 
>> The problem is that systemvm.iso is built with java 8 whereas java on the
>> VR is java 7
>>> On Mon, Feb 20, 2017 at 13:20 Will Stevens  wrote:
>>> 
>>> Did it work after resetting a VPC or when blowing away the SSVM or
>> CPVM?  I
>>> would not expect the SSVM or the CPVM to come up if the management server
>>> was built with JDK8 and the system vm template is only using JDK7.  Can
>> you
>>> confirm?​
>>> 
>>> *Will STEVENS*
>>> Lead Developer
>>> 
>>> 
>>> 
 On Mon, Feb 20, 2017 at 1:15 PM, Wei ZHOU  wrote:
 
 We've tested management server 4.7.1 with ubuntu 16.04/openjdk8 and
 systemvm 4.6 with debian7/openjdk7.
 The systemvms (ssvm, cpvm) work fine.
 
 I agree we need consider the openjdk upgrade in systemvm template.
 
 -Wei
 
 2017-02-20 18:15 GMT+01:00 Will Stevens :
 
> Regarding my question. Is it because of the version of Java that the
> systemvm.iso is built on?
> 
> On Feb 20, 2017 11:58 AM, "Will Stevens" 
>>> wrote:
> 
>> A question that is hidden in here is:
>> - Why does the JDK version on the management server have to match
>> the
 JDK
>> version of the System VM?
>> 
>> *Will STEVENS*
>> Lead Developer
>> 
>> 
>> 
>> On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion <
>>> pd...@cloudops.com>
>> wrote:
>> 
>>> Hi,
>>> 
>>> In the context of deployment of CloudStack with VPCs,
>>> What will happen to a cloud when upgrading to 4.10 that now use
>>> jdk8?
>>> 
>>> Does upgrading the management-server to 4.10 jdk8 and then keep
>> the
 old
>>> VRs
>>> run for a while that run on JDK7 will still work ?
>>> 
>>> Because If we upgrade the management-server to jdk8, we need to
>> keep
 VR
> to
>>> work until they get upgraded but we can't force an upgrade of VR
>>> just
>>> because the management-server is now using JDK8.
>>> 
>>> Thanks,
>>> 
>> 
>> 
> 
 
>>> 
>> 

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

[nathanejohnson]

GitHub user nathanejohnson opened a pull request:

https://github.com/apache/cloudstack/pull/1956

CLOUDSTACK-9796 - Fix NPE in VirtualMachineManagerImpl.java

This checks the work variable for NULL in all cases where it is
used.  Fixes CLOUDSTACK-9796.

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

$ git pull https://github.com/myENA/cloudstack bug/49npe_vmimpl

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

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


commit 7f62924f582671ae264ddab779c30ef5180bcbb4
Author: Nathan Johnson 
Date:   2017-02-20T20:58:20Z

CLOUDSTACK-9796 - Fix NPE in VirtualMachineManagerImpl.java

This checks the work variable for NULL in all cases where it is
used.  Fixes CLOUDSTACK-9796.




---
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.
---

Re: [QUESTION] Upgrade path to JDK8

[Pierre-Luc Dion]

That's quite interesting Chiradeep!

so I could do something like this I guest:

mvn clean install

and then this one to build the systemvm.iso:
mvn -Psystemvm -source 1.7 -target 1.7 install


I'll give it a try! but for now, I'm worried about existing VR, they must
continue to work while running on jdk7.  newer VPC would be ok to run with
jdk8.  but we need to make sure while upgrading the management-server we
are not in the obligation to upgrade VR's.

For sure it is required for strongswan + JDK8 to ugprade the VR, but a
 existing VR should remain usable for port forwarding, vm creation and
such...

I'll post my finding...

Thanks !



On Mon, Feb 20, 2017 at 3:59 PM, Chiradeep Vittal 
wrote:

> You can build the system vm with  -source 1.7 -target 1.7
> Also unless you are using Java8 features (lambda) the classfiles produced
> by javac 8 should work in a 1.7 JVM
>
> Sent from my iPhone
>
> > On Feb 20, 2017, at 11:51 AM, Will Stevens 
> wrote:
> >
> > yes, that is what I was expecting.  which is why I was asking about Wei's
> > setup because he seems to have worked around that problem.  Or he has a
> > custom SystemVM template running with both JDK7 and JDK8.
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > 
> >
> >> On Mon, Feb 20, 2017 at 2:20 PM, Syed Ahmed 
> wrote:
> >>
> >> The problem is that systemvm.iso is built with java 8 whereas java on
> the
> >> VR is java 7
> >>> On Mon, Feb 20, 2017 at 13:20 Will Stevens 
> wrote:
> >>>
> >>> Did it work after resetting a VPC or when blowing away the SSVM or
> >> CPVM?  I
> >>> would not expect the SSVM or the CPVM to come up if the management
> server
> >>> was built with JDK8 and the system vm template is only using JDK7.  Can
> >> you
> >>> confirm?​
> >>>
> >>> *Will STEVENS*
> >>> Lead Developer
> >>>
> >>> 
> >>>
>  On Mon, Feb 20, 2017 at 1:15 PM, Wei ZHOU 
> wrote:
> 
>  We've tested management server 4.7.1 with ubuntu 16.04/openjdk8 and
>  systemvm 4.6 with debian7/openjdk7.
>  The systemvms (ssvm, cpvm) work fine.
> 
>  I agree we need consider the openjdk upgrade in systemvm template.
> 
>  -Wei
> 
>  2017-02-20 18:15 GMT+01:00 Will Stevens :
> 
> > Regarding my question. Is it because of the version of Java that the
> > systemvm.iso is built on?
> >
> > On Feb 20, 2017 11:58 AM, "Will Stevens" 
> >>> wrote:
> >
> >> A question that is hidden in here is:
> >> - Why does the JDK version on the management server have to match
> >> the
>  JDK
> >> version of the System VM?
> >>
> >> *Will STEVENS*
> >> Lead Developer
> >>
> >> 
> >>
> >> On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion <
> >>> pd...@cloudops.com>
> >> wrote:
> >>
> >>> Hi,
> >>>
> >>> In the context of deployment of CloudStack with VPCs,
> >>> What will happen to a cloud when upgrading to 4.10 that now use
> >>> jdk8?
> >>>
> >>> Does upgrading the management-server to 4.10 jdk8 and then keep
> >> the
>  old
> >>> VRs
> >>> run for a while that run on JDK7 will still work ?
> >>>
> >>> Because If we upgrade the management-server to jdk8, we need to
> >> keep
>  VR
> > to
> >>> work until they get upgraded but we can't force an upgrade of VR
> >>> just
> >>> because the management-server is now using JDK8.
> >>>
> >>> Thanks,
> >>>
> >>
> >>
> >
> 
> >>>
> >>
>

Re: [QUESTION] Upgrade path to JDK8

[Wei ZHOU]

@Will and @Syed, I build the packages of 4.9.2+systemd patches on ubuntu
16.04 (openjdk 8).
Then install the packages to management server and kvm hosts (all are
ubuntu 16.04 with openjdk8).
The systemvm template is 4.6 with openjdk7.

cpvm and ssvm work fine.

As there is no java process in VR, so I did not check, VR should not be
impacted.

-Wei

2017-02-20 22:16 GMT+01:00 Pierre-Luc Dion :

> That's quite interesting Chiradeep!
>
> so I could do something like this I guest:
>
> mvn clean install
>
> and then this one to build the systemvm.iso:
> mvn -Psystemvm -source 1.7 -target 1.7 install
>
>
> I'll give it a try! but for now, I'm worried about existing VR, they must
> continue to work while running on jdk7.  newer VPC would be ok to run with
> jdk8.  but we need to make sure while upgrading the management-server we
> are not in the obligation to upgrade VR's.
>
> For sure it is required for strongswan + JDK8 to ugprade the VR, but a
>  existing VR should remain usable for port forwarding, vm creation and
> such...
>
> I'll post my finding...
>
> Thanks !
>
>
>
> On Mon, Feb 20, 2017 at 3:59 PM, Chiradeep Vittal 
> wrote:
>
> > You can build the system vm with  -source 1.7 -target 1.7
> > Also unless you are using Java8 features (lambda) the classfiles produced
> > by javac 8 should work in a 1.7 JVM
> >
> > Sent from my iPhone
> >
> > > On Feb 20, 2017, at 11:51 AM, Will Stevens 
> > wrote:
> > >
> > > yes, that is what I was expecting.  which is why I was asking about
> Wei's
> > > setup because he seems to have worked around that problem.  Or he has a
> > > custom SystemVM template running with both JDK7 and JDK8.
> > >
> > > *Will STEVENS*
> > > Lead Developer
> > >
> > > 
> > >
> > >> On Mon, Feb 20, 2017 at 2:20 PM, Syed Ahmed 
> > wrote:
> > >>
> > >> The problem is that systemvm.iso is built with java 8 whereas java on
> > the
> > >> VR is java 7
> > >>> On Mon, Feb 20, 2017 at 13:20 Will Stevens 
> > wrote:
> > >>>
> > >>> Did it work after resetting a VPC or when blowing away the SSVM or
> > >> CPVM?  I
> > >>> would not expect the SSVM or the CPVM to come up if the management
> > server
> > >>> was built with JDK8 and the system vm template is only using JDK7.
> Can
> > >> you
> > >>> confirm?​
> > >>>
> > >>> *Will STEVENS*
> > >>> Lead Developer
> > >>>
> > >>> 
> > >>>
> >  On Mon, Feb 20, 2017 at 1:15 PM, Wei ZHOU 
> > wrote:
> > 
> >  We've tested management server 4.7.1 with ubuntu 16.04/openjdk8 and
> >  systemvm 4.6 with debian7/openjdk7.
> >  The systemvms (ssvm, cpvm) work fine.
> > 
> >  I agree we need consider the openjdk upgrade in systemvm template.
> > 
> >  -Wei
> > 
> >  2017-02-20 18:15 GMT+01:00 Will Stevens :
> > 
> > > Regarding my question. Is it because of the version of Java that
> the
> > > systemvm.iso is built on?
> > >
> > > On Feb 20, 2017 11:58 AM, "Will Stevens" 
> > >>> wrote:
> > >
> > >> A question that is hidden in here is:
> > >> - Why does the JDK version on the management server have to match
> > >> the
> >  JDK
> > >> version of the System VM?
> > >>
> > >> *Will STEVENS*
> > >> Lead Developer
> > >>
> > >> 
> > >>
> > >> On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion <
> > >>> pd...@cloudops.com>
> > >> wrote:
> > >>
> > >>> Hi,
> > >>>
> > >>> In the context of deployment of CloudStack with VPCs,
> > >>> What will happen to a cloud when upgrading to 4.10 that now use
> > >>> jdk8?
> > >>>
> > >>> Does upgrading the management-server to 4.10 jdk8 and then keep
> > >> the
> >  old
> > >>> VRs
> > >>> run for a while that run on JDK7 will still work ?
> > >>>
> > >>> Because If we upgrade the management-server to jdk8, we need to
> > >> keep
> >  VR
> > > to
> > >>> work until they get upgraded but we can't force an upgrade of VR
> > >>> just
> > >>> because the management-server is now using JDK8.
> > >>>
> > >>> Thanks,
> > >>>
> > >>
> > >>
> > >
> > 
> > >>>
> > >>
> >
>

Re: [QUESTION] Upgrade path to JDK8

[Wei ZHOU]

We tested 4.7.1+systemd patches as well, it also works fine.

-Wei

2017-02-20 22:34 GMT+01:00 Wei ZHOU :

> @Will and @Syed, I build the packages of 4.9.2+systemd patches on ubuntu
> 16.04 (openjdk 8).
> Then install the packages to management server and kvm hosts (all are
> ubuntu 16.04 with openjdk8).
> The systemvm template is 4.6 with openjdk7.
>
> cpvm and ssvm work fine.
>
> As there is no java process in VR, so I did not check, VR should not be
> impacted.
>
> -Wei
>
> 2017-02-20 22:16 GMT+01:00 Pierre-Luc Dion :
>
>> That's quite interesting Chiradeep!
>>
>> so I could do something like this I guest:
>>
>> mvn clean install
>>
>> and then this one to build the systemvm.iso:
>> mvn -Psystemvm -source 1.7 -target 1.7 install
>>
>>
>> I'll give it a try! but for now, I'm worried about existing VR, they must
>> continue to work while running on jdk7.  newer VPC would be ok to run with
>> jdk8.  but we need to make sure while upgrading the management-server we
>> are not in the obligation to upgrade VR's.
>>
>> For sure it is required for strongswan + JDK8 to ugprade the VR, but a
>>  existing VR should remain usable for port forwarding, vm creation and
>> such...
>>
>> I'll post my finding...
>>
>> Thanks !
>>
>>
>>
>> On Mon, Feb 20, 2017 at 3:59 PM, Chiradeep Vittal 
>> wrote:
>>
>> > You can build the system vm with  -source 1.7 -target 1.7
>> > Also unless you are using Java8 features (lambda) the classfiles
>> produced
>> > by javac 8 should work in a 1.7 JVM
>> >
>> > Sent from my iPhone
>> >
>> > > On Feb 20, 2017, at 11:51 AM, Will Stevens 
>> > wrote:
>> > >
>> > > yes, that is what I was expecting.  which is why I was asking about
>> Wei's
>> > > setup because he seems to have worked around that problem.  Or he has
>> a
>> > > custom SystemVM template running with both JDK7 and JDK8.
>> > >
>> > > *Will STEVENS*
>> > > Lead Developer
>> > >
>> > > 
>> > >
>> > >> On Mon, Feb 20, 2017 at 2:20 PM, Syed Ahmed 
>> > wrote:
>> > >>
>> > >> The problem is that systemvm.iso is built with java 8 whereas java on
>> > the
>> > >> VR is java 7
>> > >>> On Mon, Feb 20, 2017 at 13:20 Will Stevens 
>> > wrote:
>> > >>>
>> > >>> Did it work after resetting a VPC or when blowing away the SSVM or
>> > >> CPVM?  I
>> > >>> would not expect the SSVM or the CPVM to come up if the management
>> > server
>> > >>> was built with JDK8 and the system vm template is only using JDK7.
>> Can
>> > >> you
>> > >>> confirm?​
>> > >>>
>> > >>> *Will STEVENS*
>> > >>> Lead Developer
>> > >>>
>> > >>> 
>> > >>>
>> >  On Mon, Feb 20, 2017 at 1:15 PM, Wei ZHOU 
>> > wrote:
>> > 
>> >  We've tested management server 4.7.1 with ubuntu 16.04/openjdk8 and
>> >  systemvm 4.6 with debian7/openjdk7.
>> >  The systemvms (ssvm, cpvm) work fine.
>> > 
>> >  I agree we need consider the openjdk upgrade in systemvm template.
>> > 
>> >  -Wei
>> > 
>> >  2017-02-20 18:15 GMT+01:00 Will Stevens :
>> > 
>> > > Regarding my question. Is it because of the version of Java that
>> the
>> > > systemvm.iso is built on?
>> > >
>> > > On Feb 20, 2017 11:58 AM, "Will Stevens" 
>> > >>> wrote:
>> > >
>> > >> A question that is hidden in here is:
>> > >> - Why does the JDK version on the management server have to match
>> > >> the
>> >  JDK
>> > >> version of the System VM?
>> > >>
>> > >> *Will STEVENS*
>> > >> Lead Developer
>> > >>
>> > >> 
>> > >>
>> > >> On Mon, Feb 20, 2017 at 11:50 AM, Pierre-Luc Dion <
>> > >>> pd...@cloudops.com>
>> > >> wrote:
>> > >>
>> > >>> Hi,
>> > >>>
>> > >>> In the context of deployment of CloudStack with VPCs,
>> > >>> What will happen to a cloud when upgrading to 4.10 that now use
>> > >>> jdk8?
>> > >>>
>> > >>> Does upgrading the management-server to 4.10 jdk8 and then keep
>> > >> the
>> >  old
>> > >>> VRs
>> > >>> run for a while that run on JDK7 will still work ?
>> > >>>
>> > >>> Because If we upgrade the management-server to jdk8, we need to
>> > >> keep
>> >  VR
>> > > to
>> > >>> work until they get upgraded but we can't force an upgrade of VR
>> > >>> just
>> > >>> because the management-server is now using JDK8.
>> > >>>
>> > >>> Thanks,
>> > >>>
>> > >>
>> > >>
>> > >
>> > 
>> > >>>
>> > >>
>> >
>>
>
>

[GitHub] cloudstack issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

[sureshanaparti]

Github user sureshanaparti commented on the issue:

https://github.com/apache/cloudstack/pull/1953
  
@remibergsma The KVM support for max data volumes 32 was updated in the DB 
as part of CLOUDSTACK-8992 (PR #980). CloudStack supports till device Id 15, 
without this changes. Might be the limit 32 was tested manually in the VM.  As 
you pointed, in KVM, I see there is a problem with the device name after device 
id 25. As soon as the device name ends with suffix 'z' for device id 25, it 
should take the next suffix 'aa', 'ab',...so on. I think it is better to 
improve the code to generate a valid device name suffix with the given device 
id. As this is specific to KVM, I'll create a separate PR with these changes. 
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.
---