[GitHub] cloudstack issue #1604: improvement: systemvm slow operations

2016-10-25 Thread romain-dartigues
Github user romain-dartigues commented on the issue:

https://github.com/apache/cloudstack/pull/1604
  
@rhtyd < JIRA issue 
[#9568](https://issues.apache.org/jira/browse/CLOUDSTACK-9568)

I'll rebase as soon as possible.


---
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 #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue:

https://github.com/apache/cloudstack/pull/1732
  
TestVolumes.py (no resigning)

test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName: 
test_00_check_template_cache | Status : SUCCESS ===
ok
Attach a volume to a stopped virtual machine, then start VM ... === 
TestName: test_01_attach_new_volume_to_stopped_VM | Status : SUCCESS ===
ok
Attach, detach, and attach volume to a running VM ... === TestName: 
test_02_attach_detach_attach_volume | Status : SUCCESS ===
ok
Attach volume to running VM, then reboot. ... === TestName: 
test_03_attached_volume_reboot_VM | Status : SUCCESS ===
ok
Detach volume from a running VM, then reboot. ... === TestName: 
test_04_detach_volume_reboot | Status : SUCCESS ===
ok
Detach volume from a stopped VM, then start. ... === TestName: 
test_05_detach_vol_stopped_VM_start | Status : SUCCESS ===
ok
Attach a volume to a stopped virtual machine, then start VM ... === 
TestName: test_06_attach_volume_to_stopped_VM | Status : SUCCESS ===
ok
Destroy and expunge VM with attached volume ... === TestName: 
test_07_destroy_expunge_VM_with_volume | Status : SUCCESS ===
ok
Delete volume that was attached to a VM and is detached now ... === 
TestName: test_08_delete_volume_was_attached | Status : SUCCESS ===
ok
Attach a data disk to a VM in one account and attach another data disk to a 
VM in another account ... === TestName: 
test_09_attach_volumes_multiple_accounts | Status : SUCCESS ===
ok
Attach more than one disk to a VM ... === TestName: 
test_10_attach_more_than_one_disk_to_VM | Status : SUCCESS ===
ok

--
Ran 11 tests in 1370.642s

OK


---
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 #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue:

https://github.com/apache/cloudstack/pull/1732
  
TestVolumes (resigning)

test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName: 
test_00_check_template_cache | Status : SUCCESS ===
ok
Attach a volume to a stopped virtual machine, then start VM ... === 
TestName: test_01_attach_new_volume_to_stopped_VM | Status : SUCCESS ===
ok
Attach, detach, and attach volume to a running VM ... === TestName: 
test_02_attach_detach_attach_volume | Status : SUCCESS ===
ok
Attach volume to running VM, then reboot. ... === TestName: 
test_03_attached_volume_reboot_VM | Status : SUCCESS ===
ok
Detach volume from a running VM, then reboot. ... === TestName: 
test_04_detach_volume_reboot | Status : SUCCESS ===
ok
Detach volume from a stopped VM, then start. ... === TestName: 
test_05_detach_vol_stopped_VM_start | Status : SUCCESS ===
ok
Attach a volume to a stopped virtual machine, then start VM ... === 
TestName: test_06_attach_volume_to_stopped_VM | Status : SUCCESS ===
ok
Destroy and expunge VM with attached volume ... === TestName: 
test_07_destroy_expunge_VM_with_volume | Status : SUCCESS ===
ok
Delete volume that was attached to a VM and is detached now ... === 
TestName: test_08_delete_volume_was_attached | Status : SUCCESS ===
ok
Attach a data disk to a VM in one account and attach another data disk to a 
VM in another account ... === TestName: 
test_09_attach_volumes_multiple_accounts | Status : SUCCESS ===
ok
Attach more than one disk to a VM ... === TestName: 
test_10_attach_more_than_one_disk_to_VM | Status : SUCCESS ===
ok

--
Ran 11 tests in 1673.081s

OK


---
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 #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue:

https://github.com/apache/cloudstack/pull/1732
  
TestVMSnapshots.py

test_01_take_VM_snapshot (TestVMSnapshots.TestVMSnapshots) ... === 
TestName: test_01_take_VM_snapshot | Status : SUCCESS ===
ok
test_02_take_VM_snapshot_with_data_disk (TestVMSnapshots.TestVMSnapshots) 
... === TestName: test_02_take_VM_snapshot_with_data_disk | Status : SUCCESS ===
ok

--
Ran 2 tests in 608.358s

OK


---
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 #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue:

https://github.com/apache/cloudstack/pull/1732
  
TestVMMigrationWithStorage

test_01_storage_migrate_root_and_data_disks 
(TestVMMigrationWithStorage.TestVMMigrationWithStorage) ... === TestName: 
test_01_storage_migrate_root_and_data_disks | Status : SUCCESS ===
ok
test_02_storage_migrate_root_and_data_disks 
(TestVMMigrationWithStorage.TestVMMigrationWithStorage) ... === TestName: 
test_02_storage_migrate_root_and_data_disks | Status : SUCCESS ===
ok
test_03_storage_migrate_root_and_data_disks_fail 
(TestVMMigrationWithStorage.TestVMMigrationWithStorage) ... === TestName: 
test_03_storage_migrate_root_and_data_disks_fail | Status : SUCCESS ===
ok
test_04_storage_migrate_root_disk_fails 
(TestVMMigrationWithStorage.TestVMMigrationWithStorage) ... === TestName: 
test_04_storage_migrate_root_disk_fails | Status : SUCCESS ===
ok
test_05_storage_migrate_data_disk_fails 
(TestVMMigrationWithStorage.TestVMMigrationWithStorage) ... === TestName: 
test_05_storage_migrate_data_disk_fails | Status : SUCCESS ===
ok

--
Ran 5 tests in 2941.392s

OK


---
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 #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue:

https://github.com/apache/cloudstack/pull/1732
  
TestSnapshots.py: The failure in test 04 makes sense as that test case is 
actually dependent upon #1600, which has not yet been merged in.

test_01_create_volume_snapshot_using_sf_snapshot 
(TestSnapshots.TestSnapshots) ... === TestName: 
test_01_create_volume_snapshot_using_sf_snapshot | Status : SUCCESS ===
ok
test_02_create_volume_snapshot_using_sf_volume 
(TestSnapshots.TestSnapshots) ... === TestName: 
test_02_create_volume_snapshot_using_sf_volume | Status : SUCCESS ===
ok
test_03_create_volume_snapshot_using_sf_volume_and_sf_snapshot 
(TestSnapshots.TestSnapshots) ... === TestName: 
test_03_create_volume_snapshot_using_sf_volume_and_sf_snapshot | Status : 
SUCCESS ===
ok
test_04_create_volume_snapshot_using_sf_snapshot_and_archiving 
(TestSnapshots.TestSnapshots) ... === TestName: 
test_04_create_volume_snapshot_using_sf_snapshot_and_archiving | Status : 
FAILED ===
FAIL

test_05_create_volume_snapshot_using_sf_snapshot_and_revert_volume_to_snapshot 
(TestSnapshots.TestSnapshots) ... === TestName: 
test_05_create_volume_snapshot_using_sf_snapshot_and_revert_volume_to_snapshot 
| Status : SUCCESS ===
ok


---
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 #1732: Switched to the official SolidFire SDK for Java

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue:

https://github.com/apache/cloudstack/pull/1732
  
TestAddRemoveHosts.py

test_add_remove_host_with_solidfire_plugin_1 
(TestAddRemoveHosts.TestAddRemoveHosts) ... === TestName: 
test_add_remove_host_with_solidfire_plugin_1 | Status : SUCCESS ===
ok
test_add_remove_host_with_solidfire_plugin_2 
(TestAddRemoveHosts.TestAddRemoveHosts) ... === TestName: 
test_add_remove_host_with_solidfire_plugin_2 | Status : SUCCESS ===
ok
test_add_remove_host_with_solidfire_plugin_3 
(TestAddRemoveHosts.TestAddRemoveHosts) ... === TestName: 
test_add_remove_host_with_solidfire_plugin_3 | Status : SUCCESS ===
ok
test_add_remove_host_with_solidfire_plugin_4 
(TestAddRemoveHosts.TestAddRemoveHosts) ... === TestName: 
test_add_remove_host_with_solidfire_plugin_4 | Status : SUCCESS ===
ok

--
Ran 4 tests in 3435.128s

OK


---
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 #1732: Switched to the official SolidFire SDK for Ja...

2016-10-25 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request:

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

Switched to the official SolidFire SDK for Java

SolidFire has recently released an official SDK for Java.

I have converted the SolidFire storage plug-ins to make use of this new SDK.


http://search.maven.org/#artifactdetails%7Ccom.solidfire%7Csolidfire-sdk-java%7C1.2.0.29%7Cjar

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

$ git pull https://github.com/mike-tutkowski/cloudstack use_sf_sdk_for_java

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

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


commit a877fdda6e0648c0a9b98f035904a901d8d66d32
Author: Mike Tutkowski 
Date:   2016-09-14T20:03:54Z

Making use of the new SolidFire SDK for 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 issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-25 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1615
  
ok. Thank you. 


---
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 #1712: CLOUDSTACK-9550: Use context to filter items in a me...

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1712
  
This should be good to merge, purely UI change? /cc @murali-reddy 
@abhinandanprateek @karuturi 


---
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 #1701: CLOUDSTACK-9534: Allow VR to be destroy in UI when i...

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1701
  
Is this good to merge, purely UI change? /cc @murali-reddy 
@abhinandanprateek @karuturi 


---
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 #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

2016-10-25 Thread koushik-das
Github user koushik-das commented on the issue:

https://github.com/apache/cloudstack/pull/1726
  
Code 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 #1711: Xenserver7 Support

2016-10-25 Thread koushik-das
Github user koushik-das commented on the issue:

https://github.com/apache/cloudstack/pull/1711
  
One small comment, rest of the code 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 #1711: Xenserver7 Support

2016-10-25 Thread koushik-das
Github user koushik-das commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1711#discussion_r85050747
  
--- Diff: setup/db/db/schema-4910to41000.sql ---
@@ -35,4 +35,192 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, 
hypervisor_type, hypervi
 
 INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created) VALUES (UUID(), 'KVM', 
'default', 'Windows 10', 257, now());
 INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created) VALUES (UUID(), 'KVM', 
'default', 'Windows 10', 258, now());
-INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created) VALUES (UUID(), 'KVM', 
'default', 'Windows Server 2012', 259, now());
\ No newline at end of file
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created) VALUES (UUID(), 'KVM', 
'default', 'Windows Server 2012', 259, now());
+
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 4.5 (32-bit)', 1, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 4.6 (32-bit)', 2, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 4.7 (32-bit)', 3, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 4.8 (32-bit)', 4, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (32-bit)', 5, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (64-bit)', 6, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (32-bit)', 7, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (64-bit)', 8, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (32-bit)', 9, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (64-bit)', 10, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (32-bit)', 11, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (64-bit)', 12, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (32-bit)', 13, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (64-bit)', 14, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (32-bit)', 111, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, 
hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) 
VALUES (UUID(),'Xenserver', '7.0.0', 'CentOS 5 (64-bit)', 112, utc_timestamp(), 
0);
+INSERT IGNORE INTO `cloud`.`guest_os_hypervisor

[GitHub] cloudstack issue #1716: CLOUDSTACK-9555 when a template is deleted and then ...

2016-10-25 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1716
  
@yvsubhash Can you check why travis failed? (force push to re run it if its 
a false alarm)


---
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 #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1615
  
@karuturi tests are about to finish


---
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 #1731: CLOUDSTACK-9565: Fix intermittent failure in ...

2016-10-25 Thread rhtyd
GitHub user rhtyd opened a pull request:

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

CLOUDSTACK-9565: Fix intermittent failure in test_oobm_zchange_password

Fixes intermittent integration smoke test failures caused in
test_oobm_zchange_password test.

The scope is limited to the integration test only, and full integration 
test suite is not necessary. We can only consider code reviews and merge on 
basis of Travis results.

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

$ git pull https://github.com/shapeblue/cloudstack oobm-changepasswd-fix

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

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


commit 29844a3ec94e1dc40da9c6ac90d5345b89e82360
Author: Rohit Yadav 
Date:   2016-10-26T05:01:35Z

CLOUDSTACK-9565: Fix intermittent failure in test_oobm_zchange_password

Fixes intermittent integration smoke test failures caused in
test_oobm_zchange_password test.

Signed-off-by: Rohit Yadav 




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1642: CLOUDSTACK-9504: System VMs on Managed Storag...

2016-10-25 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-25 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1615
  
@rhtyd any update on the 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 #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1642
  
LGTM. The one failing test is due to environment issue, I've fixed it and 
will send a PR shortly.


---
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 #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-25 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1642
  
yes. I am merging this now.


---
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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) 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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
@blueorangutan test centos7 vmware-55u3


---
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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

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


---
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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
@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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
@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 #1711: Xenserver7 Support

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1711
  
@rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt 
+ vmware55u3, centos7 mgmt + kvmcentos7) 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 #1711: Xenserver7 Support

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

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


---
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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
Trillian test result (tid-182)
Environment: vmware-60u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26183 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1729-t182-vmware-60u2.zip
Test completed. 39 look ok, 9 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_vpc_site2site_vpn | `Failure` | 105.66 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | `Failure` | 110.63 | test_vpc_vpn.py
test_02_VPC_default_routes | `Failure` | 1136.76 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | `Failure` | 424.69 | test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 399.44 
| test_vpc_redundant.py
test_02_revert_vm_snapshots | `Failure` | 90.24 | test_vm_snapshots.py
test_04_rvpc_privategw_static_routes | `Failure` | 665.48 | 
test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 283.27 | 
test_privategw_acl.py
test_02_vpc_privategw_static_routes | `Failure` | 414.15 | 
test_privategw_acl.py
test_01_vpc_privategw_acl | `Failure` | 96.57 | test_privategw_acl.py
test_oobm_zchange_password | `Failure` | 20.41 | test_outofbandmanagement.py
test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | `Failure` | 
211.17 | test_internal_lb.py
test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | `Failure` | 100.87 | 
test_internal_lb.py
test_01_redundant_vpc_site2site_vpn | `Error` | 631.61 | test_vpc_vpn.py
ContextSuite context=TestRVPCSite2SiteVpn>:teardown | `Error` | 692.04 | 
test_vpc_vpn.py
test_01_VPC_nics_after_destroy | `Error` | 106.64 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | `Error` | 510.49 | test_vpc_redundant.py
test_02_redundant_VPC_default_routes | `Error` | 262.33 | 
test_vpc_redundant.py
test_01_test_vm_volume_snapshot | `Error` | 110.69 | test_vm_snapshots.py
test_01_create_vm_snapshots | `Error` | 161.57 | test_vm_snapshots.py
test_04_rvpc_privategw_static_routes | `Error` | 700.79 | 
test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanup | `Error` | 318.60 | 
test_privategw_acl.py
test_01_nic | `Error` | 147.30 | test_nic.py
test_reboot_router | `Error` | 292.29 | test_network.py
test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | `Error` | 
36.03 | test_network.py
test_network_rules_acquired_public_ip_1_static_nat_rule | `Error` | 88.89 | 
test_network.py
test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 246.92 | 
test_internal_lb.py
test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `Error` | 362.59 | 
test_internal_lb.py
test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `Error` | 377.95 | 
test_internal_lb.py
test_deployvm_firstfit | `Error` | 35.46 | 
test_deploy_vms_with_varied_deploymentplanners.py
test_04_rvpc_network_garbage_collector_nics | Success | 1527.93 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 636.27 | test_vpc_redundant.py
test_09_delete_detached_volume | Success | 20.68 | test_volumes.py
test_06_download_detached_volume | Success | 45.42 | test_volumes.py
test_05_detach_volume | Success | 100.21 | test_volumes.py
test_04_delete_attached_volume | Success | 10.16 | test_volumes.py
test_03_download_attached_volume | Success | 15.19 | test_volumes.py
test_02_attach_volume | Success | 48.60 | test_volumes.py
test_01_create_volume | Success | 434.85 | test_volumes.py
test_03_delete_vm_snapshots | Success | 275.16 | test_vm_snapshots.py
test_deploy_vm_multiple | Success | 207.20 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.01 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.83 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.20 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 50.67 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.06 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 10.10 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 5.09 | test_vm_life_cycle.py
test_02_start_vm | Success | 15.13 | test_vm_life_cycle.py
test_01_stop_vm | Success | 5.08 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 175.92 | test_templates.py
test_08_list_system_templates | Success | 0.02 | test_templates.py
test_07_list_public_templates | Success | 0.03 | test_templates.py
test_05_template_permissions | Success | 0.04 | test_templates.py
test_04_extract_template | Success | 10.17 | test_templates.py
test_03_delete_template | Success | 5.12 | test_templates.py
test_02_edit_template | Success | 90.15 | test_templates.py
tes

[GitHub] cloudstack issue #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
Trillian test result (tid-181)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 6
Total time taken: 39964 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1729-t181-vmware-55u3.zip
Test completed. 32 look ok, 16 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_vpc_site2site_vpn | `Failure` | 293.36 | test_vpc_vpn.py
test_02_VPC_default_routes | `Failure` | 163.46 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | `Failure` | 1172.39 | 
test_vpc_router_nics.py
test_02_redundant_VPC_default_routes | `Failure` | 1385.81 | 
test_vpc_redundant.py
test_04_rvpc_privategw_static_routes | `Failure` | 218.35 | 
test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 468.24 | 
test_privategw_acl.py
test_02_vpc_privategw_static_routes | `Failure` | 117.64 | 
test_privategw_acl.py
test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | `Failure` | 25.74 
| test_internal_lb.py
test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `Failure` | 1204.62 | 
test_internal_lb.py
test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | `Failure` | 515.79 | 
test_internal_lb.py
test_01_redundant_vpc_site2site_vpn | `Error` | 780.17 | test_vpc_vpn.py
ContextSuite context=TestRVPCSite2SiteVpn>:teardown | `Error` | 916.49 | 
test_vpc_vpn.py
test_05_rvpc_multi_tiers | `Error` | 1210.47 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | `Error` | 37.10 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
`Error` | 1457.18 | test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Error` | 208.62 | 
test_vpc_redundant.py
ContextSuite context=TestVmSnapshot>:setup | `Error` | 400.79 | 
test_vm_snapshots.py
test_deploy_vm_multiple | `Error` | 2545.00 | test_vm_life_cycle.py
ContextSuite context=TestVMLifeCycle>:setup | `Error` | 2911.25 | 
test_vm_life_cycle.py
ContextSuite context=TestSnapshotRootDisk>:teardown | `Error` | 82.19 | 
test_snapshots.py
test_09_reboot_router | `Error` | 5.17 | test_routers.py
test_08_start_router | `Error` | 2583.70 | test_routers.py
test_03_restart_network_cleanup | `Error` | 40.64 | test_routers.py
test_03_vpc_privategw_restart_vpc_cleanup | `Error` | 508.86 | 
test_privategw_acl.py
test_01_nic | `Error` | 224.38 | test_nic.py
test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | `Error` | 
36.37 | test_network.py
test_delete_account | `Error` | 203.59 | test_network.py
test_assign_and_removal_lb | `Error` | 0.13 | test_loadbalance.py
test_02_create_lb_rule_non_nat | `Error` | 92.09 | test_loadbalance.py
test_02_create_lb_rule_non_nat | `Error` | 97.20 | test_loadbalance.py
ContextSuite context=TestListIdsParams>:setup | `Error` | 0.00 | 
test_list_ids_parameter.py
test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 192.27 | 
test_internal_lb.py
test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 197.39 | 
test_internal_lb.py
test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | `Error` | 586.78 | 
test_internal_lb.py
test_deployvm_userdata_post | `Error` | 60.87 | 
test_deploy_vm_with_userdata.py
test_deployvm_userdata | `Error` | 171.92 | test_deploy_vm_with_userdata.py
test_deploy_vm_from_iso | `Error` | 72.66 | test_deploy_vm_iso.py
test_DeployVmAntiAffinityGroup | `Error` | 257.79 | test_affinity_groups.py
test_01_vpc_remote_access_vpn | Success | 207.35 | test_vpc_vpn.py
test_09_delete_detached_volume | Success | 36.11 | test_volumes.py
test_06_download_detached_volume | Success | 50.61 | test_volumes.py
test_05_detach_volume | Success | 110.33 | test_volumes.py
test_04_delete_attached_volume | Success | 15.27 | test_volumes.py
test_03_download_attached_volume | Success | 20.37 | test_volumes.py
test_02_attach_volume | Success | 53.82 | test_volumes.py
test_01_create_volume | Success | 518.04 | test_volumes.py
test_01_test_vm_volume_snapshot | Success | 171.83 | test_vm_snapshots.py
test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.03 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 247.66 | test_templates.py
test_08_list_system_templates | Success | 0.03 | test_templates.py
test_07_list_public_templates | Success | 0.03 | test_templates.py
test_05_template_permissions | Success | 0.07 | test_templates.py
test_04_extract_template | Success | 15.23 | test_templates.py
test_03_delete_template | Success | 5.12 | test_templates.py
test_02_edit_template | Success | 90.20 | test_templates.py
test_01_create_template | Success | 131.28 

[GitHub] cloudstack issue #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 ...

2016-10-25 Thread jburwell
Github user jburwell commented on the issue:

https://github.com/apache/cloudstack/pull/1677
  
@blueorangutan test centos7 vmware55u3


---
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 #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 ...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1677
  
@jburwell a Trillian-Jenkins test job (centos7 mgmt + vmware55u3) 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 #1700: CLOUDSTACK-9359: IPv6 for Basic Networking

2016-10-25 Thread NuxRo
Github user NuxRo commented on the issue:

https://github.com/apache/cloudstack/pull/1700
  
Wido,

Will do, but need to wait until next week to get some help re getting IPv6 
on my test rig. As soon as I have feedback, I'll post here.

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Wido den Hollander" 
> To: "apache/cloudstack" 
> Cc: "NuxRo" , "Mention" 
> Sent: Friday, 21 October, 2016 16:23:30
> Subject: Re: [apache/cloudstack] CLOUDSTACK-9359: IPv6 for Basic 
Networking (#1700)

> @NuxRo The first test would be to run this code and actually see that the 
API
> returns a IPv6 address which you can then use to reach your Instance.
> 
> --
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub:
> https://github.com/apache/cloudstack/pull/1700#issuecomment-255406953



---
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 #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue:

https://github.com/apache/cloudstack/pull/1642
  
@rhtyd @karuturi I think we are good to go on this PR now. Do you agree?


---
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 #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 ...

2016-10-25 Thread serg38
Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1677
  
LGTM for testing
```
2016-10-24 20:52:55 47 test files ro run
2016-10-24 20:52:55 Running test file: 
test/integration/smoke/test_affinity_groups_projects.py
2016-10-24 20:56:07 Running test file: 
test/integration/smoke/test_affinity_groups.py
2016-10-24 20:59:18 Running test file: 
test/integration/smoke/test_deploy_vgpu_enabled_vm.py
2016-10-24 20:59:19 Running test file: 
test/integration/smoke/test_deploy_vm_iso.py
2016-10-24 21:16:20 Running test file: 
test/integration/smoke/test_deploy_vm_root_resize.py
2016-10-24 21:16:44 Running test file: 
test/integration/smoke/test_deploy_vms_with_varied_deploymentplanners.py
2016-10-24 21:21:22 Running test file: 
test/integration/smoke/test_deploy_vm_with_userdata.py
2016-10-24 21:24:32 Running test file: 
test/integration/smoke/test_disk_offerings.py
2016-10-24 21:24:33 Running test file: 
test/integration/smoke/test_dynamicroles.py
2016-10-24 21:24:34 Running test file: 
test/integration/smoke/test_global_settings.py
2016-10-24 21:24:35 Running test file: 
test/integration/smoke/test_guest_vlan_range.py
2016-10-24 21:24:59 Running test file: test/integration/smoke/test_hosts.py
2016-10-24 21:25:00 Running test file: 
test/integration/smoke/test_internal_lb.py
2016-10-24 22:05:15 Running test file: test/integration/smoke/test_iso.py
2016-10-24 22:10:16 Running test file: 
test/integration/smoke/test_list_ids_parameter.py
2016-10-24 22:24:18 Running test file: 
test/integration/smoke/test_loadbalance.py
2016-10-24 22:45:45 Running test file: test/integration/smoke/test_login.py
2016-10-24 22:46:09 Running test file: 
test/integration/smoke/test_multipleips_per_nic.py
2016-10-24 22:48:45 Running test file: 
test/integration/smoke/test_network_acl.py
2016-10-24 22:51:21 Running test file: 
test/integration/smoke/test_network.py
2016-10-24 23:23:44 Running test file: 
test/integration/smoke/test_nic_adapter_type.py
2016-10-24 23:23:52 Running test file: test/integration/smoke/test_nic.py
2016-10-24 23:33:13 Running test file: 
test/integration/smoke/test_non_contigiousvlan.py
2016-10-24 23:33:30 Running test file: 
test/integration/smoke/test_over_provisioning.py
2016-10-24 23:33:31 Running test file: 
test/integration/smoke/test_password_server.py
2016-10-24 23:37:44 Running test file: 
test/integration/smoke/test_portable_publicip.py
2016-10-24 23:39:07 Running test file: 
test/integration/smoke/test_primary_storage.py
2016-10-24 23:39:08 Running test file: 
test/integration/smoke/test_public_ip_range.py
2016-10-24 23:39:17 Running test file: test/integration/smoke/test_pvlan.py
2016-10-24 23:39:23 Running test file: 
test/integration/smoke/test_regions.py
2016-10-24 23:39:25 Running test file: 
test/integration/smoke/test_reset_vm_on_reboot.py
2016-10-24 23:42:47 Running test file: 
test/integration/smoke/test_resource_detail.py
2016-10-24 23:43:00 Running test file: 
test/integration/smoke/test_router_dhcphosts.py
2016-10-24 23:47:42 Running test file: 
test/integration/smoke/test_routers_iptables_default_policy.py
2016-10-24 23:54:49 Running test file: 
test/integration/smoke/test_routers.py
2016-10-25 00:02:55 Running test file: 
test/integration/smoke/test_scale_vm.py
2016-10-25 00:06:15 Running test file: 
test/integration/smoke/test_secondary_storage.py
2016-10-25 00:06:18 Running test file: 
test/integration/smoke/test_service_offerings.py
2016-10-25 00:10:42 Running test file: 
test/integration/smoke/test_snapshots.py
2016-10-25 00:15:15 Running test file: test/integration/smoke/test_ssvm.py
2016-10-25 00:33:50 Running test file: 
test/integration/smoke/test_staticroles.py
2016-10-25 00:34:31 Running test file: 
test/integration/smoke/test_templates.py
2016-10-25 00:50:07 Running test file: 
test/integration/smoke/test_usage_events.py
2016-10-25 00:50:08 Running test file: 
test/integration/smoke/test_vm_life_cycle.py
2016-10-25 01:05:18 Running test file: 
test/integration/smoke/test_vm_snapshots.py
2016-10-25 01:20:58 Running test file: 
test/integration/smoke/test_volumes.py
2016-10-25 01:38:21 Running test file: 
test/integration/smoke/test_vpc_vpn.py

Test to update a physical network and extend its vlan ... === TestName: 
test_extendPhysicalNetworkVlan | Status : SUCCESS ===
ok
Test Site 2 Site VPN Across redundant VPCs ... === TestName: 
test_01_redundant_vpc_site2site_vpn | Status : SUCCESS ===
ok
Test Remote Access VPN in VPC ... === TestName: 
test_01_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test Site 2 Site VPN Across VPCs ... === TestName: 
test_01_vpc_site2site_vpn | Status : SUCCESS ===
ok
Test to create service offering ... === TestName: 
test_01_create_service_offering | Status : SUCCESS ===

[GitHub] cloudstack issue #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 ...

2016-10-25 Thread serg38
Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/1677
  
@rhtyd @murali-reddy @PaulAngus @jburwell Thanks for fixing all outstanding 
Marvin test issues. We have now 100% passing score on Marvin integration tests 
on RHEL 6 management server , advanced networking, Vmware 5.5/6.0 hypervisors. 
Please find results 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.
---


RE: [GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-25 Thread Paul Angus
@rohit
Can you set a BlueO job off from this please?


Kind regards,

Paul Angus

paul.an...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue
  
 


-Original Message-
From: syed [mailto:g...@git.apache.org] 
Sent: 25 October 2016 17:27
To: dev@cloudstack.apache.org
Subject: [GitHub] cloudstack issue #1711: Xenserver7 Support

Github user syed commented on the issue:

https://github.com/apache/cloudstack/pull/1711
  
@PaulAngus would it be possible to run another round of tests on this? 


---
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 #1711: Xenserver7 Support

2016-10-25 Thread syed
Github user syed commented on the issue:

https://github.com/apache/cloudstack/pull/1711
  
@PaulAngus would it be possible to run another round of tests on this? 


---
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 #1728: CLOUDSTACK-9551: Move java tmp dir to cloudstack-age...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1728
  
Trillian test result (tid-180)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26449 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1728-t180-kvm-centos7.zip
Test completed. 47 look ok, 1 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_02_redundant_VPC_default_routes | `Failure` | 875.27 | 
test_vpc_redundant.py
test_oobm_zchange_password | `Failure` | 21.16 | test_outofbandmanagement.py
test_01_vpc_site2site_vpn | Success | 164.89 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 66.11 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 250.62 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 274.88 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 558.92 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 514.13 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1416.24 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 568.97 | test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1275.56 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 15.45 | test_volumes.py
test_08_resize_volume | Success | 15.40 | test_volumes.py
test_07_resize_fail | Success | 20.45 | test_volumes.py
test_06_download_detached_volume | Success | 15.28 | test_volumes.py
test_05_detach_volume | Success | 100.23 | test_volumes.py
test_04_delete_attached_volume | Success | 10.19 | test_volumes.py
test_03_download_attached_volume | Success | 15.29 | test_volumes.py
test_02_attach_volume | Success | 44.33 | test_volumes.py
test_01_create_volume | Success | 711.34 | test_volumes.py
test_deploy_vm_multiple | Success | 248.52 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.67 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.25 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 41.25 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.12 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 125.85 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 125.91 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.17 | test_vm_life_cycle.py
test_01_stop_vm | Success | 40.33 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 141.20 | 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.19 | test_templates.py
test_03_delete_template | Success | 5.11 | test_templates.py
test_02_edit_template | Success | 90.18 | test_templates.py
test_01_create_template | Success | 40.45 | test_templates.py
test_10_destroy_cpvm | Success | 161.86 | test_ssvm.py
test_09_destroy_ssvm | Success | 168.73 | test_ssvm.py
test_08_reboot_cpvm | Success | 131.59 | test_ssvm.py
test_07_reboot_ssvm | Success | 133.82 | test_ssvm.py
test_06_stop_cpvm | Success | 131.78 | test_ssvm.py
test_05_stop_ssvm | Success | 133.83 | test_ssvm.py
test_04_cpvm_internals | Success | 1.22 | test_ssvm.py
test_03_ssvm_internals | Success | 3.31 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.12 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.15 | test_ssvm.py
test_01_snapshot_root_disk | Success | 11.28 | test_snapshots.py
test_04_change_offering_small | Success | 239.70 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.06 | 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.20 | test_secondary_storage.py
test_09_reboot_router | Success | 35.34 | test_routers.py
test_08_start_router | Success | 30.32 | 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_cleanup | Success | 5.69 | test_routers.py
test_03_restart_network_cleanup | Success | 60.54 | test_routers.py
test_02_router_internal_adv | Success | 1.00 | test_routers.py
test_01_router_internal_basic | Success | 0.57 | test_routers.py

[GitHub] cloudstack issue #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1726
  
Trillian test result (tid-179)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 24929 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1726-t179-kvm-centos7.zip
Test completed. 43 look ok, 0 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_01_vpc_site2site_vpn | Success | 160.69 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 66.88 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 251.75 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 284.86 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 557.95 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 515.21 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1310.57 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 556.64 | test_vpc_redundant.py
test_02_redundant_VPC_default_routes | Success | 763.30 | 
test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1301.57 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 15.53 | test_volumes.py
test_08_resize_volume | Success | 15.48 | test_volumes.py
test_07_resize_fail | Success | 20.45 | test_volumes.py
test_06_download_detached_volume | Success | 15.55 | test_volumes.py
test_05_detach_volume | Success | 100.34 | test_volumes.py
test_04_delete_attached_volume | Success | 10.20 | test_volumes.py
test_03_download_attached_volume | Success | 15.44 | test_volumes.py
test_02_attach_volume | Success | 45.26 | test_volumes.py
test_01_create_volume | Success | 714.64 | test_volumes.py
test_deploy_vm_multiple | Success | 274.23 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.97 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.19 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 41.34 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.29 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 126.03 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 126.33 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.21 | test_vm_life_cycle.py
test_01_stop_vm | Success | 35.40 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 81.08 | test_templates.py
test_08_list_system_templates | Success | 0.05 | 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.21 | test_templates.py
test_02_edit_template | Success | 90.27 | test_templates.py
test_01_create_template | Success | 40.54 | test_templates.py
test_10_destroy_cpvm | Success | 161.68 | test_ssvm.py
test_09_destroy_ssvm | Success | 164.54 | test_ssvm.py
test_08_reboot_cpvm | Success | 101.60 | test_ssvm.py
test_07_reboot_ssvm | Success | 163.81 | test_ssvm.py
test_06_stop_cpvm | Success | 132.02 | test_ssvm.py
test_05_stop_ssvm | Success | 164.12 | test_ssvm.py
test_04_cpvm_internals | Success | 1.32 | test_ssvm.py
test_03_ssvm_internals | Success | 3.27 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.12 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.13 | test_ssvm.py
test_01_snapshot_root_disk | Success | 11.99 | test_snapshots.py
test_04_change_offering_small | Success | 209.95 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.09 | test_service_offerings.py
test_01_create_service_offering | Success | 0.10 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.12 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.20 | test_secondary_storage.py
test_09_reboot_router | Success | 40.45 | test_routers.py
test_08_start_router | Success | 30.30 | test_routers.py
test_07_stop_router | Success | 10.17 | test_routers.py
test_06_router_advanced | Success | 0.05 | test_routers.py
test_05_router_basic | Success | 0.04 | test_routers.py
test_04_restart_network_wo_cleanup | Success | 5.68 | test_routers.py
test_03_restart_network_cleanup | Success | 60.53 | test_routers.py
test_02_router_internal_adv | Success | 1.16 | test_routers.py
test_01_router_internal_basic | Success | 0.63 | test_routers.py
test_router_dhcphosts | Success | 277.91 | test_router_dhcphosts.py
test_01_

[GitHub] cloudstack pull request #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareCo...

2016-10-25 Thread rhtyd
Github user rhtyd closed the pull request at:

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


---
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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareCo...

2016-10-25 Thread rhtyd
GitHub user rhtyd reopened a pull request:

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

CLOUDSTACK-9564: Fix memory leaks in VmwareContextPool

In a recent management server crash, it was found that the largest 
contributor
to memory leak was in VmwareContextPool where a registry is held (arraylist)
that grows indefinitely. The list itself is not used anywhere or consumed. 
There
exists a hashmap (pool) that returns a list of contexts for existing poolkey
(address/username) that is used instead.

This fixes the issue by removing the arraylist registry, and limiting the
length of the context list for a given poolkey.

@blueorangutan package

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

$ git pull https://github.com/shapeblue/cloudstack vmware-memleak-fix

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

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


commit ed964aa2a61e3a36fcb9b5bd8fb80c3f1434
Author: Rohit Yadav 
Date:   2016-10-25T09:50:33Z

CLOUDSTACK-9564: Fix memory leaks in VmwareContextPool

In a recent management server crash, it was found that the largest 
contributor
to memory leak was in VmwareContextPool where a registry is held (arraylist)
that grows indefinitely. The list itself is not used anywhere or consumed. 
There
exists a hashmap (pool) that returns a list of contexts for existing poolkey
(address/username) that is used instead.

This fixes the issue by removing the arraylist registry, and limiting the
length of the context list for a given poolkey.

Signed-off-by: Rohit Yadav 




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1642
  
Trillian test result (tid-178)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 25671 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1642-t178-kvm-centos7.zip
Test completed. 48 look ok, 0 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_oobm_zchange_password | `Failure` | 20.50 | test_outofbandmanagement.py
test_01_vpc_site2site_vpn | Success | 175.25 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 66.25 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 255.94 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 310.18 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 541.03 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 515.16 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1322.44 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 559.41 | test_vpc_redundant.py
test_02_redundant_VPC_default_routes | Success | 755.94 | 
test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1296.27 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 16.10 | test_volumes.py
test_08_resize_volume | Success | 15.42 | test_volumes.py
test_07_resize_fail | Success | 20.67 | test_volumes.py
test_06_download_detached_volume | Success | 15.35 | test_volumes.py
test_05_detach_volume | Success | 100.26 | test_volumes.py
test_04_delete_attached_volume | Success | 10.21 | test_volumes.py
test_03_download_attached_volume | Success | 15.31 | test_volumes.py
test_02_attach_volume | Success | 45.61 | test_volumes.py
test_01_create_volume | Success | 735.31 | test_volumes.py
test_deploy_vm_multiple | Success | 268.64 | 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.76 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.16 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 40.94 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.13 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 125.88 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 125.84 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.19 | test_vm_life_cycle.py
test_01_stop_vm | Success | 40.33 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 90.80 | test_templates.py
test_08_list_system_templates | Success | 0.04 | test_templates.py
test_07_list_public_templates | Success | 0.07 | test_templates.py
test_05_template_permissions | Success | 0.06 | test_templates.py
test_04_extract_template | Success | 5.17 | test_templates.py
test_03_delete_template | Success | 5.10 | test_templates.py
test_02_edit_template | Success | 90.15 | test_templates.py
test_01_create_template | Success | 40.45 | test_templates.py
test_10_destroy_cpvm | Success | 131.83 | test_ssvm.py
test_09_destroy_ssvm | Success | 163.88 | test_ssvm.py
test_08_reboot_cpvm | Success | 131.76 | test_ssvm.py
test_07_reboot_ssvm | Success | 133.61 | test_ssvm.py
test_06_stop_cpvm | Success | 131.84 | test_ssvm.py
test_05_stop_ssvm | Success | 138.72 | test_ssvm.py
test_04_cpvm_internals | Success | 1.18 | test_ssvm.py
test_03_ssvm_internals | Success | 3.36 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.14 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.16 | test_ssvm.py
test_01_snapshot_root_disk | Success | 11.26 | test_snapshots.py
test_04_change_offering_small | Success | 242.72 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.09 | test_service_offerings.py
test_01_create_service_offering | Success | 0.12 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.16 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.19 | test_secondary_storage.py
test_09_reboot_router | Success | 45.42 | test_routers.py
test_08_start_router | Success | 35.35 | 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_cleanup | Success | 5.74 | test_routers.py
test_03_restart_network_cleanup | Success | 60.55 | test_routers.py
test_02_router_internal_adv | Success | 1.21 | test_routers.py
test_01_router_internal_basic | Success | 0.69 | test_routers.py
   

[GitHub] cloudstack pull request #1730: VMTemplateZone needs some love

2016-10-25 Thread marcaurele
GitHub user marcaurele opened a pull request:

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

VMTemplateZone needs some love

It's incorrect to use the findIncludingRemovedBy and
listIncludingRemovedBy for the common list and find operation. Moreover
the DAO is trying to update the `removed` attribute to null but will
failed (GenericDBaseDao.generateValue() will overwrite the null value
and set a new date.

Signed-off-by: Marc-Aurèle Brothier 

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

$ git pull https://github.com/exoscale/cloudstack fix/VMTemplateZoneDao

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

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


commit 3519bfb8d8aa7b90e97a57c0e79a1d520e210592
Author: Marc-Aurèle Brothier 
Date:   2016-10-25T13:18:59Z

VMTemplateZone needs some love

It's incorrect to use the findIncludingRemovedBy and
listIncludingRemovedBy for the common list and find operation. Moreover
the DAO is trying to update the `removed` attribute to null but will
failed (GenericDBaseDao.generateValue() will overwrite the null value
and set a new date.

Signed-off-by: Marc-Aurèle Brothier 




---
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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
@rhtyd a Trillian-Jenkins test job (centos6 mgmt + vmware-55u3) 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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
@blueorangutan test centos6 vmware-55u3


---
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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

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


---
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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
@rhtyd I understand these words: "help", "hello", "thanks", "package", 
"test"
Test run usage: test (mgmt server, one of: centos6, centos7, ubuntu) 
(hypervisor, one of: kvm-centos6, kvm-centos7, kvm-ubuntu, xenserver-65sp1, 
xenserver-62sp1, vmware-60u2, vmware-55u3, vmware-51u1, vmware-50u1)
Authorized contributors for kicking Trillian Jenkins test jobs are: 
['rhtyd', 'jburwell', 'murali-reddy', 'abhinandanprateek', 'PaulAngus', 
'borisstoyanov', 'karuturi']


---
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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
@blueorangutan 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 #1451: CLOUDSTACK-9319: Use timeout when applying config to...

2016-10-25 Thread cloudmonger
Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1451
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 121
 Hypervisor xenserver
 NetworkType Advanced
 Passed=103
 Failed=2
 Skipped=5

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


**Failed tests:**
* test_deploy_vm_iso.py

 * test_deploy_vm_from_iso Failing since 6 runs

* test_vm_life_cycle.py

 * test_10_attachAndDetach_iso Failing since 7 runs


**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1729
  
@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 pull request #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareCo...

2016-10-25 Thread rhtyd
GitHub user rhtyd opened a pull request:

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

CLOUDSTACK-9564: Fix memory leaks in VmwareContextPool

In a recent management server crash, it was found that the largest 
contributor
to memory leak was in VmwareContextPool where a registry is held (arraylist)
that grows indefinitely. The list itself is not used anywhere or consumed. 
There
exists a hashmap (pool) that returns a list of contexts for existing poolkey
(address/username) that is used instead.

This fixes the issue by removing the arraylist registry, and limiting the
length of the context list for a given poolkey.

@blueorangutan package

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

$ git pull https://github.com/shapeblue/cloudstack vmware-memleak-fix

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

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


commit dfabcc8a5eb0f0b78003c2f849f32e899a74ad48
Author: Rohit Yadav 
Date:   2016-10-25T09:50:33Z

CLOUDSTACK-9564: Fix memory leaks in VmwareContextPool

In a recent management server crash, it was found that the largest 
contributor
to memory leak was in VmwareContextPool where a registry is held (arraylist)
that grows indefinitely. The list itself is not used anywhere or consumed. 
There
exists a hashmap (pool) that returns a list of contexts for existing poolkey
(address/username) that is used instead.

This fixes the issue by removing the arraylist registry, and limiting the
length of the context list for a given poolkey.

Signed-off-by: Rohit Yadav 




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1451: CLOUDSTACK-9319: Use timeout when applying co...

2016-10-25 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1451: CLOUDSTACK-9319: Use timeout when applying config to...

2016-10-25 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1451
  
Code LGTM.

BVTs are good(iso failures are URL access issues and not related to this 
PR).

merging this now


---
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 #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1615
  
@karuturi there was a capacity issue at the backend, the tests have been 
re-fired.


---
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 #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-10-25 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1724
  
I saw there seems to be an issue with the fwd-merge, I'll fix that 
@murali-reddy for now.


---
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 #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

2016-10-25 Thread yvsubhash
Github user yvsubhash commented on the issue:

https://github.com/apache/cloudstack/pull/1726
  
@koushik-das Added the null check


---
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 #1726: CLOUDSTACK-9560 Root volume of deleted VM lef...

2016-10-25 Thread koushik-das
Github user koushik-das commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1726#discussion_r84862649
  
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -2199,15 +2199,20 @@ public void cleanupDownloadUrls(){
 if(downloadUrlCurrentAgeInSecs < 
_downloadUrlExpirationInterval){  // URL hasnt expired yet
 continue;
 }
-
-s_logger.debug("Removing download url " + 
volumeOnImageStore.getExtractUrl() + " for volume id " + 
volumeOnImageStore.getVolumeId());
+long volumeId = volumeOnImageStore.getVolumeId();
+s_logger.debug("Removing download url " + 
volumeOnImageStore.getExtractUrl() + " for volume id " + volumeId);
 
 // Remove it from image store
 ImageStoreEntity secStore = (ImageStoreEntity) 
_dataStoreMgr.getDataStore(volumeOnImageStore.getDataStoreId(), 
DataStoreRole.Image);
 
secStore.deleteExtractUrl(volumeOnImageStore.getInstallPath(), 
volumeOnImageStore.getExtractUrl(), Upload.Type.VOLUME);
 
 // Now expunge it from DB since this entry was created 
only for download purpose
 _volumeStoreDao.expunge(volumeOnImageStore.getId());
+Volume volume = _volumeDao.findById(volumeId);
+if (volume.getState() == Volume.State.Expunged)
--- End diff --

@yvsubhash It is not about the DB relationship, is it possible that some 
other thread went ahead and deleted the volume entry after the store_ref 
entries are queried?


---
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 #1726: CLOUDSTACK-9560 Root volume of deleted VM lef...

2016-10-25 Thread yvsubhash
Github user yvsubhash commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1726#discussion_r84859378
  
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -2199,15 +2199,20 @@ public void cleanupDownloadUrls(){
 if(downloadUrlCurrentAgeInSecs < 
_downloadUrlExpirationInterval){  // URL hasnt expired yet
 continue;
 }
-
-s_logger.debug("Removing download url " + 
volumeOnImageStore.getExtractUrl() + " for volume id " + 
volumeOnImageStore.getVolumeId());
+long volumeId = volumeOnImageStore.getVolumeId();
+s_logger.debug("Removing download url " + 
volumeOnImageStore.getExtractUrl() + " for volume id " + volumeId);
 
 // Remove it from image store
 ImageStoreEntity secStore = (ImageStoreEntity) 
_dataStoreMgr.getDataStore(volumeOnImageStore.getDataStoreId(), 
DataStoreRole.Image);
 
secStore.deleteExtractUrl(volumeOnImageStore.getInstallPath(), 
volumeOnImageStore.getExtractUrl(), Upload.Type.VOLUME);
 
 // Now expunge it from DB since this entry was created 
only for download purpose
 _volumeStoreDao.expunge(volumeOnImageStore.getId());
+Volume volume = _volumeDao.findById(volumeId);
+if (volume.getState() == Volume.State.Expunged)
--- End diff --

@koushik-das Having a null for volume is not a possible scenario as there 
is db level relation ship between volume and volume_store_ref and the id is 
picked up from volume_store_ref. Would you still recommend having a null check 
here?


---
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 #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-10-25 Thread murali-reddy
GitHub user murali-reddy reopened a pull request:

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

CLOUDSTACK-9511: fix test_privategw_acl.py to handle multiple physical 
network


fix to ensure only physical network with guest traffic is picked up for
creating a private network for vpc private gateway

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

$ git pull https://github.com/murali-reddy/cloudstack test_privategw_acl

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

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


commit 5728ad03caf0580970f2c8226cae4440e12f4d92
Author: Murali Reddy 
Date:   2016-10-24T09:45:35Z

CLOUDSTACK-9511: fix test_privategw_acl.py to handle multiple physical 
network

fix to ensure only physical network with guest traffic is picked up for
creating a private network for vpc private gateway




---
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 #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-10-25 Thread murali-reddy
Github user murali-reddy closed the pull request at:

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


---
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 #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-25 Thread koushik-das
Github user koushik-das commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1727#discussion_r84839433
  
--- Diff: core/src/com/cloud/agent/api/VMSnapshotTO.java ---
@@ -123,4 +125,12 @@ public boolean getQuiescevm() {
 public void setQuiescevm(boolean quiescevm) {
 this.quiescevm = quiescevm;
 }
+
+public Long getServiceOfferingId() {
--- End diff --

See previous comment, please remove as not 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 #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-25 Thread koushik-das
Github user koushik-das commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1727#discussion_r84841681
  
--- Diff: server/src/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java ---
@@ -610,7 +615,11 @@ public UserVm revertToSnapshot(Long vmSnapshotId) 
throws InsufficientCapacityExc
 VmWorkJobVO placeHolder = null;
 placeHolder = createPlaceHolderWork(vmSnapshotVo.getVmId());
 try {
-return orchestrateRevertToVMSnapshot(vmSnapshotId);
+UserVm revertedVM = 
orchestrateRevertToVMSnapshot(vmSnapshotId);
+
+updateUserVmServiceOffering(revertedVM, vmSnapshotVo);
--- End diff --

Better to put updateUserVmServiceOffering() call inside 
orchestrateRevertToVMSnapshot() as orchestrate is called from multiple places. 


---
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 #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-25 Thread koushik-das
Github user koushik-das commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1727#discussion_r84841375
  
--- Diff: server/src/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java ---
@@ -639,10 +648,57 @@ else if (jobResult instanceof Throwable)
 throw new RuntimeException("Unexpected exception", 
(Throwable)jobResult);
 }
 
+updateUserVmServiceOffering(userVm, vmSnapshotVo);
--- End diff --

No need to update service offering here, the else part is just to create an 
entry in the DB for the revert to snapshot job.


---
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 #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-25 Thread koushik-das
Github user koushik-das commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1727#discussion_r84839190
  
--- Diff: core/src/com/cloud/agent/api/VMSnapshotTO.java ---
@@ -35,6 +35,7 @@
 private VMSnapshotTO parent;
 private List volumes;
 private boolean quiescevm;
+private Long serviceOfferingId;
--- End diff --

Don't see any use of service offering stored here, is it needed? *TO 
objects are used to pass data from orchestration layer to agent layer.


---
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 #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-25 Thread koushik-das
Github user koushik-das commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1727#discussion_r84839712
  
--- Diff: 
engine/storage/snapshot/src/org/apache/cloudstack/storage/vmsnapshot/DefaultVMSnapshotStrategy.java
 ---
@@ -128,7 +128,7 @@ public VMSnapshot takeVMSnapshot(VMSnapshot vmSnapshot) 
{
 if (options != null)
 quiescevm = options.needQuiesceVM();
 VMSnapshotTO target =
-new VMSnapshotTO(vmSnapshot.getId(), vmSnapshot.getName(), 
vmSnapshot.getType(), null, vmSnapshot.getDescription(), false, current, 
quiescevm);
+new VMSnapshotTO(vmSnapshot.getId(), vmSnapshot.getName(), 
vmSnapshot.getType(), null, vmSnapshot.getDescription(), false, current, 
quiescevm, userVm.getServiceOfferingId());
--- End diff --

Storing the service offering in DB along with vm snapshot entry should be 
sufficient, don't see any reason to put it in TO object.


---
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 #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-25 Thread koushik-das
Github user koushik-das commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1727#discussion_r84839559
  
--- Diff: engine/schema/src/com/cloud/vm/snapshot/VMSnapshotVO.java ---
@@ -248,4 +252,9 @@ public void setRemoved(Date removed) {
 public Class getEntityType() {
 return VMSnapshot.class;
 }
+
+public long getServiceOfferingId() {
--- End diff --

Please put the getter in the VMSnapshot interface and provide 
implementation here.


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