You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by rhtyd <gi...@git.apache.org> on 2016/09/27 07:50:22 UTC

[GitHub] cloudstack pull request #1694: CLOUDSTACK-9509: Host Connects Without Storag...

GitHub user rhtyd opened a pull request:

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

    CLOUDSTACK-9509: Host Connects Without Storage

    KVM hosts on shared storage failure was accepted by mgmt server with the
    host state as Up, even though there was no primary/shared storage available on
    it. This patch offers a quick fix by throwing an exception in the storage monitor
    which connects storage pool on host. The failure is trapped by agent manager
    that disconnects the agent without any investigation.
    
    Based on Lab tests, KVM agent may take upto 2 minutes to attempt NFS mount when
    the storage is inaccessible (firewalled, or shutdown) before returning back with
    an error. It is safe to assume that this won't add pressure on mgmt server due to
    several reconnection attempts, and KVM agent would retry reconnection every 2
    minutes.
    
    For such KVM hosts, where failure happens due to storage issues; they will be
    briefly put in Alert state but will be mostly be in Connecting state during which
    the KVM host attempts to mount/reconfigure NFS storage pool.
    
    /cc @jburwell @karuturi 
    @blueorangutan package

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

    $ git pull https://github.com/shapeblue/cloudstack kvm-no-storage-failfast

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

    https://github.com/apache/cloudstack/pull/1694.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 #1694
    
----
commit e13e40ee9cc664ec9d326c8b6fae0c76f6adc01a
Author: Rohit Yadav <ro...@shapeblue.com>
Date:   2016-06-07T06:11:16Z

    CLOUDSTACK-9509: Host Connects Without Storage
    
    KVM hosts on shared storage failure was accepted by mgmt server with the
    host state as Up, even though there was no primary/shared storage available on
    it. This patch offers a quick fix by throwing an exception in the storage monitor
    which connects storage pool on host. The failure is trapped by agent manager
    that disconnects the agent without any investigation.
    
    Based on Lab tests, KVM agent may take upto 2 minutes to attempt NFS mount when
    the storage is inaccessible (firewalled, or shutdown) before returning back with
    an error. It is safe to assume that this won't add pressure on mgmt server due to
    several reconnection attempts, and KVM agent would retry reconnection every 2
    minutes.
    
    For such KVM hosts, where failure happens due to storage issues; they will be
    briefly put in Alert state but will be mostly be in Connecting state during which
    the KVM host attempts to mount/reconfigure NFS storage pool.
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>

----


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    <b>Trillian test result (tid-165)</b>
    Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6
    Total time taken: 29324 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1694-t165-xenserver-65sp1.zip
    Test completed. 41 look ok, 2 have error(s)
    
    
    Test | Result | Time (s) | Test File
    --- | --- | --- | ---
    test_05_rvpc_multi_tiers | `Failure` | 445.58 | test_vpc_redundant.py
    test_04_rvpc_network_garbage_collector_nics | `Failure` | 418.55 | test_vpc_redundant.py
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 505.57 | test_vpc_redundant.py
    test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 422.99 | test_privategw_acl.py
    test_01_vpc_site2site_vpn | Success | 306.53 | test_vpc_vpn.py
    test_01_vpc_remote_access_vpn | Success | 126.74 | test_vpc_vpn.py
    test_01_redundant_vpc_site2site_vpn | Success | 457.94 | test_vpc_vpn.py
    test_02_VPC_default_routes | Success | 366.79 | test_vpc_router_nics.py
    test_01_VPC_nics_after_destroy | Success | 675.34 | test_vpc_router_nics.py
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 836.97 | test_vpc_redundant.py
    test_02_redundant_VPC_default_routes | Success | 973.92 | test_vpc_redundant.py
    test_09_delete_detached_volume | Success | 15.70 | test_volumes.py
    test_08_resize_volume | Success | 91.07 | test_volumes.py
    test_07_resize_fail | Success | 96.07 | test_volumes.py
    test_06_download_detached_volume | Success | 25.47 | test_volumes.py
    test_05_detach_volume | Success | 100.39 | test_volumes.py
    test_04_delete_attached_volume | Success | 10.24 | test_volumes.py
    test_03_download_attached_volume | Success | 15.37 | test_volumes.py
    test_02_attach_volume | Success | 10.73 | test_volumes.py
    test_01_create_volume | Success | 402.57 | test_volumes.py
    test_03_delete_vm_snapshots | Success | 280.24 | test_vm_snapshots.py
    test_02_revert_vm_snapshots | Success | 186.45 | test_vm_snapshots.py
    test_01_create_vm_snapshots | Success | 134.07 | test_vm_snapshots.py
    test_deploy_vm_multiple | Success | 304.21 | 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 | 36.98 | test_vm_life_cycle.py
    test_09_expunge_vm | Success | 125.19 | test_vm_life_cycle.py
    test_08_migrate_vm | Success | 61.42 | test_vm_life_cycle.py
    test_07_restore_vm | Success | 0.15 | test_vm_life_cycle.py
    test_06_destroy_vm | Success | 10.18 | test_vm_life_cycle.py
    test_03_reboot_vm | Success | 15.24 | test_vm_life_cycle.py
    test_02_start_vm | Success | 25.29 | test_vm_life_cycle.py
    test_01_stop_vm | Success | 30.30 | test_vm_life_cycle.py
    test_CreateTemplateWithDuplicateName | Success | 126.23 | 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.06 | test_templates.py
    test_04_extract_template | Success | 5.24 | test_templates.py
    test_03_delete_template | Success | 5.12 | test_templates.py
    test_02_edit_template | Success | 90.13 | test_templates.py
    test_01_create_template | Success | 75.87 | test_templates.py
    test_10_destroy_cpvm | Success | 201.78 | test_ssvm.py
    test_09_destroy_ssvm | Success | 198.80 | test_ssvm.py
    test_08_reboot_cpvm | Success | 146.63 | test_ssvm.py
    test_07_reboot_ssvm | Success | 148.88 | test_ssvm.py
    test_06_stop_cpvm | Success | 166.81 | test_ssvm.py
    test_05_stop_ssvm | Success | 138.88 | test_ssvm.py
    test_04_cpvm_internals | Success | 1.13 | test_ssvm.py
    test_03_ssvm_internals | Success | 3.68 | test_ssvm.py
    test_02_list_cpvm_vm | Success | 0.13 | test_ssvm.py
    test_01_list_sec_storage_vm | Success | 0.14 | test_ssvm.py
    test_01_snapshot_root_disk | Success | 26.69 | test_snapshots.py
    test_04_change_offering_small | Success | 76.02 | test_service_offerings.py
    test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
    test_02_edit_service_offering | Success | 0.12 | test_service_offerings.py
    test_01_create_service_offering | Success | 0.11 | test_service_offerings.py
    test_02_sys_template_ready | Success | 0.15 | test_secondary_storage.py
    test_01_sys_vm_start | Success | 0.18 | test_secondary_storage.py
    test_01_scale_vm | Success | 5.27 | test_scale_vm.py
    test_09_reboot_router | Success | 65.57 | test_routers.py
    test_08_start_router | Success | 50.62 | test_routers.py
    test_07_stop_router | Success | 15.23 | test_routers.py
    test_06_router_advanced | Success | 0.07 | test_routers.py
    test_05_router_basic | Success | 0.04 | test_routers.py
    test_04_restart_network_wo_cleanup | Success | 5.60 | test_routers.py
    test_03_restart_network_cleanup | Success | 126.22 | test_routers.py
    test_02_router_internal_adv | Success | 0.81 | test_routers.py
    test_01_router_internal_basic | Success | 0.41 | test_routers.py
    test_router_dhcphosts | Success | 108.25 | test_router_dhcphosts.py
    test_01_updatevolumedetail | Success | 5.14 | test_resource_detail.py
    test_01_reset_vm_on_reboot | Success | 35.45 | test_reset_vm_on_reboot.py
    test_createRegion | Success | 0.04 | test_regions.py
    test_create_pvlan_network | Success | 5.25 | test_pvlan.py
    test_dedicatePublicIpRange | Success | 0.56 | test_public_ip_range.py
    test_04_rvpc_privategw_static_routes | Success | 922.77 | test_privategw_acl.py
    test_02_vpc_privategw_static_routes | Success | 508.78 | test_privategw_acl.py
    test_01_vpc_privategw_acl | Success | 174.09 | test_privategw_acl.py
    test_01_primary_storage_nfs | Success | 38.65 | test_primary_storage.py
    test_01_primary_storage_iscsi | Success | 78.61 | test_primary_storage.py
    test_createPortablePublicIPRange | Success | 15.24 | test_portable_publicip.py
    test_createPortablePublicIPAcquire | Success | 15.82 | test_portable_publicip.py
    test_isolate_network_password_server | Success | 61.40 | test_password_server.py
    test_UpdateStorageOverProvisioningFactor | Success | 0.19 | test_over_provisioning.py
    test_extendPhysicalNetworkVlan | Success | 15.36 | test_non_contigiousvlan.py
    test_01_nic | Success | 868.91 | test_nic.py
    test_releaseIP | Success | 279.32 | test_network.py
    test_reboot_router | Success | 475.42 | test_network.py
    test_public_ip_user_account | Success | 10.31 | test_network.py
    test_public_ip_admin_account | Success | 40.34 | test_network.py
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 71.75 | test_network.py
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 61.57 | test_network.py
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 93.13 | test_network.py
    test_delete_account | Success | 279.44 | test_network.py
    test_02_port_fwd_on_non_src_nat | Success | 55.76 | test_network.py
    test_01_port_fwd_on_src_nat | Success | 78.95 | test_network.py
    test_nic_secondaryip_add_remove | Success | 234.01 | test_multipleips_per_nic.py
    login_test_saml_user | Success | 24.86 | test_login.py
    test_assign_and_removal_lb | Success | 139.01 | test_loadbalance.py
    test_02_create_lb_rule_non_nat | Success | 187.53 | test_loadbalance.py
    test_01_create_lb_rule_src_nat | Success | 194.85 | test_loadbalance.py
    test_07_list_default_iso | Success | 0.07 | test_iso.py
    test_05_iso_permissions | Success | 0.07 | test_iso.py
    test_04_extract_Iso | Success | 5.13 | test_iso.py
    test_03_delete_iso | Success | 95.23 | test_iso.py
    test_02_edit_iso | Success | 0.07 | test_iso.py
    test_01_create_iso | Success | 22.26 | test_iso.py
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Success | 447.61 | test_internal_lb.py
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | Success | 324.47 | test_internal_lb.py
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | Success | 950.36 | test_internal_lb.py
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Success | 695.69 | test_internal_lb.py
    test_dedicateGuestVlanRange | Success | 10.42 | test_guest_vlan_range.py
    test_UpdateConfigParamWithScope | Success | 0.22 | test_global_settings.py
    test_04_create_fat_type_disk_offering | Success | 0.08 | test_disk_offerings.py
    test_03_delete_disk_offering | Success | 0.04 | test_disk_offerings.py
    test_02_edit_disk_offering | Success | 0.06 | test_disk_offerings.py
    test_02_create_sparse_type_disk_offering | Success | 0.07 | test_disk_offerings.py
    test_01_create_disk_offering | Success | 0.13 | test_disk_offerings.py
    test_deployvm_userdispersing | Success | 96.19 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userconcentrated | Success | 40.86 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_firstfit | Success | 222.00 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userdata_post | Success | 91.18 | test_deploy_vm_with_userdata.py
    test_deployvm_userdata | Success | 212.12 | test_deploy_vm_with_userdata.py
    test_02_deploy_vm_root_resize | Success | 7.59 | test_deploy_vm_root_resize.py
    test_01_deploy_vm_root_resize | Success | 6.80 | test_deploy_vm_root_resize.py
    test_00_deploy_vm_root_resize | Success | 6.98 | test_deploy_vm_root_resize.py
    test_deploy_vm_from_iso | Success | 173.63 | test_deploy_vm_iso.py
    test_DeployVmAntiAffinityGroup | Success | 126.49 | test_affinity_groups.py
    test_01_test_vm_volume_snapshot | Skipped | 0.00 | test_vm_snapshots.py
    test_06_copy_template | Skipped | 0.00 | test_templates.py
    test_06_copy_iso | Skipped | 0.00 | test_iso.py
    test_deploy_vgpu_enabled_vm | Skipped | 0.01 | test_deploy_vgpu_enabled_vm.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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    Packaging result: \u2714centos6 \u2714centos7 \u2714debian. JID-136


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by nathanejohnson <gi...@git.apache.org>.
Github user nathanejohnson commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @rhtyd It appears that rolling this forward to 4.9 et al has broken builds.  The signature of the CreateStoragePool constructor changed between 4.8 and 4.9.
    
    https://github.com/apache/cloudstack/blob/4.8/server/src/com/cloud/storage/listener/StoragePoolMonitor.java#L52
    
    That's the 4.8 version
    
    https://github.com/apache/cloudstack/blob/4.9/server/src/com/cloud/storage/listener/StoragePoolMonitor.java#L57
    
    That's the 4.9 version, which takes 3 parameters.  When building I get the following error:
    
    [ERROR] <snip> /server/test/com/cloud/storage/listener/StoragePoolMonitorTest.java:[49,29] error: constructor StoragePoolMonitor in class StoragePoolMonitor cannot be applied to given types;
    



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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @nathanejohnson I'm not sure what you are doing, the build failure was due to a unit test introduced in this PR. In 4.9+ branches, the unit test did not consider the additional parameter added to the constructor signature in 4.9+ branches. The simplest fix was to simply pass null, as the unit test does not depend/use that additional parameter. Thanks for bringing this to my attention, the build is fixed on 4.9, master branches 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by nathanejohnson <gi...@git.apache.org>.
Github user nathanejohnson commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @rhtyd I have an issue with circular dependencies trying to make the test work with the new signature.  In order to initialize the StoragePoolMonitor, I need a DataStoreManagerImpl type, and this is defined in cloud-engine-storage .  I cannot add cloud-engine-storage as a maven dependency of cloud-server because this in turn depends on cloud-secondary-storage , which in turn depends on cloud-server.  I'm not sure what the best path forward is, but unfortunately I don't think I can get this resolved today (no pun intended).


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

[GitHub] cloudstack pull request #1694: CLOUDSTACK-9509: Host Connects Without Storag...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1694#discussion_r80714655
  
    --- Diff: server/test/com/cloud/storage/listener/StoragePoolMonitorTest.java ---
    @@ -0,0 +1,80 @@
    +// Licensed to the Apache Software Foundation (ASF) under one
    +// or more contributor license agreements.  See the NOTICE file
    +// distributed with this work for additional information
    +// regarding copyright ownership.  The ASF licenses this file
    +// to you under the Apache License, Version 2.0 (the
    +// "License"); you may not use this file except in compliance
    +// with the License.  You may obtain a copy of the License at
    +//
    +//   http://www.apache.org/licenses/LICENSE-2.0
    +//
    +// Unless required by applicable law or agreed to in writing,
    +// software distributed under the License is distributed on an
    +// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    +// KIND, either express or implied.  See the License for the
    +// specific language governing permissions and limitations
    +// under the License.
    +package com.cloud.storage.listener;
    +
    +import com.cloud.agent.api.StartupRoutingCommand;
    +import com.cloud.exception.ConnectionException;
    +import com.cloud.exception.StorageUnavailableException;
    +import com.cloud.host.HostVO;
    +import com.cloud.hypervisor.Hypervisor;
    +import com.cloud.storage.ScopeType;
    +import com.cloud.storage.StorageManagerImpl;
    +import com.cloud.storage.StoragePoolStatus;
    +import org.apache.cloudstack.storage.datastore.db.PrimaryDataStoreDao;
    +import org.apache.cloudstack.storage.datastore.db.StoragePoolVO;
    +import org.junit.Before;
    +import org.junit.Test;
    +import org.mockito.Mockito;
    +
    +import java.util.Collections;
    +
    +public class StoragePoolMonitorTest {
    +
    +    private StorageManagerImpl storageManager;
    +    private PrimaryDataStoreDao poolDao;
    +    private StoragePoolMonitor storagePoolMonitor;
    +    private HostVO host;
    +    private StoragePoolVO pool;
    +    private StartupRoutingCommand cmd;
    +
    +    @Before
    +    public void setUp() throws Exception {
    +        storageManager = Mockito.mock(StorageManagerImpl.class);
    +        poolDao = Mockito.mock(PrimaryDataStoreDao.class);
    +
    +        storagePoolMonitor = new StoragePoolMonitor(storageManager, poolDao);
    +        host = new HostVO("some-uuid");
    --- End diff --
    
    Why not pass in `UUID.randomUUID().toString()`?


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    <b>Trillian test result (tid-79)</b>
    Environment: vmware-55u3 (x2), Advanced Networking
    Total time taken: 30443 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1694-t79-vmware-55u3.zip
    Test completed. 34 look ok, 15 have errors
    
    
    Test | Result | Time (s)
    --- | --- | ---
    test_04_rvpc_privategw_static_routes | `Failure` | 398.735
    test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 207.986
    test_02_vpc_privategw_static_routes | `Failure` | 257.854
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `Failure` | 501.051
    test_02_VPC_default_routes | `Failure` | 221.414
    test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | `Failure` | 377.254
    test_01_vpc_privategw_acl | `Failure` | 107.494
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | `Failure` | 405.698
    test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | `Failure` | 414.494
    test_router_dhcphosts | `Error` | 820.586
    test_isolate_network_password_server | `Error` | 827.198
    test_extendPhysicalNetworkVlan | `Error` | 0.025
    test_dedicatePublicIpRange | `Error` | 0.235
    test_CreateTemplateWithDuplicateName | `Error` | 5.175
    test_07_quota | `Error` | 0.029
    test_06_quota | `Error` | 0.028
    test_05_quota | `Error` | 0.032
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 446.367
    test_04_quota | `Error` | 0.030
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 375.908
    test_03_quota | `Error` | 0.028
    test_02_routervm_iptables_policies | `Error` | 909.879
    test_02_quota | `Error` | 0.033
    test_01_vpc_site2site_vpn | `Error` | 452.068
    test_01_vpc_remote_access_vpn | `Error` | 0.083
    test_01_test_vm_volume_snapshot | `Error` | 181.540
    test_01_snapshot_root_disk | `Error` | 35.310
    test_01_single_VPC_iptables_policies | `Error` | 1003.815
    test_01_redundant_vpc_site2site_vpn | `Error` | 643.114
    test_01_quota | `Error` | 0.038
    test_01_create_template | `Error` | 40.325
    ContextSuite context=TestTemplates>:setup | `Error` | 299.833
    ContextSuite context=TestRouterDHCPHosts>:teardown | `Error` | 841.655
    ContextSuite context=TestDedicateGuestVlanRange>:setup | `Error` | 0.000
    test_releaseIP | Success | 228.681
    test_reboot_router | Success | 605.432
    test_public_ip_user_account | Success | 10.266
    test_public_ip_admin_account | Success | 40.791
    test_nic_secondaryip_add_remove | Success | 293.906
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 67.347
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 62.142
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 125.051
    test_network_acl | Success | 162.044
    test_deployvm_userdispersing | Success | 55.621
    test_deployvm_userdata_post | Success | 30.532
    test_deployvm_userdata | Success | 136.273
    test_deployvm_userconcentrated | Success | 50.639
    test_deployvm_firstfit | Success | 186.294
    test_deploy_vm_multiple | Success | 218.556
    test_deploy_vm_from_iso | Success | 393.054
    test_deploy_vm | Success | 0.056
    test_delete_account | Success | 308.119
    test_create_pvlan_network | Success | 5.171
    test_createRegion | Success | 0.028
    test_createPortablePublicIPRange | Success | 15.264
    test_createPortablePublicIPAcquire | Success | 15.488
    test_assign_and_removal_lb | Success | 134.082
    test_advZoneVirtualRouter | Success | 0.032
    test_UpdateStorageOverProvisioningFactor | Success | 0.100
    test_UpdateConfigParamWithScope | Success | 0.114
    test_DeployVmAntiAffinityGroup_in_project | Success | 242.078
    test_DeployVmAntiAffinityGroup | Success | 177.027
    test_10_destroy_cpvm | Success | 271.825
    test_10_attachAndDetach_iso | Success | 72.246
    test_09_reboot_router | Success | 145.923
    test_09_expunge_vm | Success | 125.144
    test_09_destroy_ssvm | Success | 238.942
    test_09_delete_detached_volume | Success | 31.169
    test_08_start_router | Success | 151.003
    test_08_reboot_cpvm | Success | 156.507
    test_08_migrate_vm | Success | 70.877
    test_07_stop_router | Success | 25.271
    test_07_restore_vm | Success | 0.061
    test_07_reboot_ssvm | Success | 158.295
    test_07_list_default_iso | Success | 0.067
    test_06_stop_cpvm | Success | 201.800
    test_06_router_advanced | Success | 0.040
    test_06_download_detached_volume | Success | 65.539
    test_06_destroy_vm | Success | 10.148
    test_05_stop_ssvm | Success | 213.934
    test_05_rvpc_multi_tiers | Success | 605.495
    test_05_router_basic | Success | 0.026
    test_05_iso_permissions | Success | 0.077
    test_05_detach_volume | Success | 110.299
    test_04_rvpc_network_garbage_collector_nics | Success | 844.685
    test_04_restart_network_wo_cleanup | Success | 5.608
    test_04_extract_Iso | Success | 5.697
    test_04_delete_attached_volume | Success | 10.191
    test_04_create_fat_type_disk_offering | Success | 0.096
    test_04_cpvm_internals | Success | 1.183
    test_04_change_offering_small | Success | 88.002
    test_03_ssvm_internals | Success | 4.149
    test_03_restart_network_cleanup | Success | 146.017
    test_03_reboot_vm | Success | 5.101
    test_03_download_attached_volume | Success | 15.313
    test_03_delete_vm_snapshots | Success | 275.195
    test_03_delete_service_offering | Success | 0.034
    test_03_delete_iso | Success | 95.142
    test_03_delete_disk_offering | Success | 0.067
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 658.450
    test_03_RVR_Network_check_router_state | Success | 333.238
    test_02_sys_template_ready | Success | 0.113
    test_02_start_vm | Success | 20.209
    test_02_router_internal_adv | Success | 0.932
    test_02_revert_vm_snapshots | Success | 194.146
    test_02_redundant_VPC_default_routes | Success | 593.909
    test_02_port_fwd_on_non_src_nat | Success | 55.717
    test_02_list_cpvm_vm | Success | 0.092
    test_02_isolate_network_FW_PF_default_routes_egress_false | Success | 286.817
    test_02_edit_service_offering | Success | 0.051
    test_02_edit_iso | Success | 0.086
    test_02_edit_disk_offering | Success | 0.094
    test_02_deploy_vm_root_resize | Success | 6.219
    test_02_create_sparse_type_disk_offering | Success | 0.089
    test_02_create_lb_rule_non_nat | Success | 188.067
    test_02_attach_volume | Success | 58.794
    test_01_updatevolumedetail | Success | 5.141
    test_01_sys_vm_start | Success | 0.131
    test_01_stop_vm | Success | 5.085
    test_01_router_internal_basic | Success | 0.496
    test_01_reset_vm_on_reboot | Success | 30.310
    test_01_primary_storage_nfs | Success | 38.374
    test_01_port_fwd_on_src_nat | Success | 111.849
    test_01_nic | Success | 827.325
    test_01_list_sec_storage_vm | Success | 0.089
    test_01_isolate_network_FW_PF_default_routes_egress_true | Success | 266.909
    test_01_deploy_vm_root_resize | Success | 6.194
    test_01_create_volume | Success | 519.838
    test_01_create_vm_snapshots | Success | 161.633
    test_01_create_service_offering | Success | 0.074
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1237.733
    test_01_create_lb_rule_src_nat | Success | 188.869
    test_01_create_iso | Success | 66.440
    test_01_create_disk_offering | Success | 0.102
    test_01_VPC_nics_after_destroy | Success | 606.932
    test_00_deploy_vm_root_resize | Success | 6.361
    login_test_saml_user | Success | 23.450
    test_vm_nic_adapter_vmxnet3 | Skipped | 0.001
    test_deploy_vgpu_enabled_vm | Skipped | 0.005
    test_08_resize_volume | Skipped | 5.134
    test_07_resize_fail | Skipped | 10.209
    test_06_copy_iso | Skipped | 0.000
    test_01_scale_vm | Skipped | 66.426
    test_01_primary_storage_iscsi | Skipped | 0.026



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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    <b>Trillian test result (tid-69)</b>
    Environment: kvm-centos7 (x2), Advanced Networking
    Total time taken: 25642 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1694-t69-kvm-centos7.zip
    Test completed. 40 look ok, 13 have errors
    
    
    Test | Result | Time (s)
    --- | --- | ---
    test_04_rvpc_privategw_static_routes | `Failure` | 148.190
    test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 67.483
    test_02_vpc_privategw_static_routes | `Failure` | 82.661
    test_02_list_templates | `Failure` | 0.039
    test_01_vpc_privategw_acl | `Failure` | 42.148
    test_router_dhcphosts | `Error` | 775.938
    test_oobm_zchange_password | `Error` | 0.055
    test_oobm_zchange_password | `Error` | 0.050
    test_oobm_multiple_mgmt_server_ownership | `Error` | 0.051
    test_oobm_multiple_mgmt_server_ownership | `Error` | 0.045
    test_oobm_issue_power_status | `Error` | 0.053
    test_oobm_issue_power_status | `Error` | 0.049
    test_oobm_issue_power_soft | `Error` | 0.053
    test_oobm_issue_power_soft | `Error` | 0.049
    test_oobm_issue_power_reset | `Error` | 0.051
    test_oobm_issue_power_reset | `Error` | 0.046
    test_oobm_issue_power_on | `Error` | 0.048
    test_oobm_issue_power_on | `Error` | 0.044
    test_oobm_issue_power_off | `Error` | 0.051
    test_oobm_issue_power_off | `Error` | 0.046
    test_oobm_issue_power_cycle | `Error` | 0.052
    test_oobm_issue_power_cycle | `Error` | 0.048
    test_oobm_enabledisable_across_clusterzones | `Error` | 0.048
    test_oobm_enabledisable_across_clusterzones | `Error` | 0.044
    test_oobm_enable_feature_valid | `Error` | 0.052
    test_oobm_enable_feature_valid | `Error` | 0.048
    test_oobm_enable_feature_invalid | `Error` | 0.051
    test_oobm_disable_feature_valid | `Error` | 0.052
    test_oobm_disable_feature_valid | `Error` | 0.048
    test_oobm_disable_feature_invalid | `Error` | 0.050
    test_oobm_configure_invalid_driver | `Error` | 0.051
    test_oobm_configure_default_driver | `Error` | 0.063
    test_oobm_configure_default_driver | `Error` | 0.058
    test_oobm_background_powerstate_sync | `Error` | 0.041
    test_oobm_background_powerstate_sync | `Error` | 0.034
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | `Error` | 743.280
    test_network_rules_acquired_public_ip_2_nat_rule | `Error` | 745.475
    test_network_rules_acquired_public_ip_1_static_nat_rule | `Error` | 729.606
    test_isolate_network_password_server | `Error` | 809.759
    test_extendPhysicalNetworkVlan | `Error` | 0.035
    test_deploy_vm_from_iso | `Error` | 181.906
    test_dedicatePublicIpRange | `Error` | 0.328
    test_03_delete_template | `Error` | 5.098
    test_02_routervm_iptables_policies | `Error` | 776.140
    test_01_snapshot_root_disk | `Error` | 0.099
    test_01_single_VPC_iptables_policies | `Error` | 865.694
    ContextSuite context=TestVpcSite2SiteVpn>:setup | `Error` | 0.000
    ContextSuite context=TestVpcRemoteAccessVpn>:setup | `Error` | 0.000
    ContextSuite context=TestRouterDHCPHosts>:teardown | `Error` | 822.380
    ContextSuite context=TestRVPCSite2SiteVpn>:setup | `Error` | 0.000
    ContextSuite context=TestDedicateGuestVlanRange>:setup | `Error` | 0.000
    test_router_dns_guestipquery | Success | 76.805
    test_router_dns_externalipquery | Success | 0.081
    test_releaseIP | Success | 239.178
    test_reboot_router | Success | 380.124
    test_public_ip_user_account | Success | 10.332
    test_public_ip_admin_account | Success | 40.354
    test_nic_secondaryip_add_remove | Success | 218.867
    test_network_acl | Success | 56.248
    test_deployvm_userdispersing | Success | 20.775
    test_deployvm_userdata_post | Success | 5.404
    test_deployvm_userdata | Success | 50.898
    test_deployvm_userconcentrated | Success | 20.726
    test_deployvm_firstfit | Success | 60.826
    test_deploy_vm_multiple | Success | 233.798
    test_deploy_vm | Success | 0.034
    test_delete_account | Success | 269.111
    test_create_pvlan_network | Success | 5.275
    test_createRegion | Success | 0.046
    test_createPortablePublicIPRange | Success | 15.482
    test_createPortablePublicIPAcquire | Success | 15.729
    test_assign_and_removal_lb | Success | 133.827
    test_advZoneVirtualRouter | Success | 0.027
    test_UpdateStorageOverProvisioningFactor | Success | 0.161
    test_UpdateConfigParamWithScope | Success | 0.235
    test_DeployVmAntiAffinityGroup_in_project | Success | 132.014
    test_DeployVmAntiAffinityGroup | Success | 66.284
    test_CreateTemplateWithDuplicateName | Success | 40.555
    test_10_destroy_cpvm | Success | 131.455
    test_10_attachAndDetach_iso | Success | 71.706
    test_09_reboot_router | Success | 35.380
    test_09_expunge_vm | Success | 125.211
    test_09_destroy_ssvm | Success | 163.514
    test_09_delete_detached_volume | Success | 15.701
    test_08_start_router | Success | 25.326
    test_08_resize_volume | Success | 15.464
    test_08_reboot_cpvm | Success | 101.591
    test_08_migrate_vm | Success | 41.092
    test_08_list_system_templates | Success | 0.029
    test_07_stop_router | Success | 10.190
    test_07_restore_vm | Success | 0.137
    test_07_resize_fail | Success | 20.605
    test_07_reboot_ssvm | Success | 103.279
    test_07_list_public_templates | Success | 0.035
    test_07_list_default_iso | Success | 0.067
    test_06_stop_cpvm | Success | 136.742
    test_06_router_advanced | Success | 0.085
    test_06_download_detached_volume | Success | 15.348
    test_06_destroy_vm | Success | 125.967
    test_05_template_permissions | Success | 0.070
    test_05_stop_ssvm | Success | 133.522
    test_05_rvpc_multi_tiers | Success | 414.497
    test_05_router_basic | Success | 0.041
    test_05_iso_permissions | Success | 0.080
    test_05_detach_volume | Success | 100.311
    test_04_rvpc_network_garbage_collector_nics | Success | 820.674
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Success | 213.537
    test_04_restart_network_wo_cleanup | Success | 5.679
    test_04_extract_template | Success | 5.174
    test_04_extract_Iso | Success | 5.186
    test_04_delete_attached_volume | Success | 10.258
    test_04_create_fat_type_disk_offering | Success | 0.079
    test_04_cpvm_internals | Success | 1.212
    test_04_change_offering_small | Success | 209.853
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | Success | 178.277
    test_03_ssvm_internals | Success | 3.083
    test_03_restart_network_cleanup | Success | 55.619
    test_03_reboot_vm | Success | 125.949
    test_03_list_snapshots | Success | 0.084
    test_03_download_attached_volume | Success | 15.391
    test_03_delete_service_offering | Success | 0.049
    test_03_delete_iso | Success | 95.237
    test_03_delete_disk_offering | Success | 0.063
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 432.178
    test_03_RVR_Network_check_router_state | Success | 257.206
    test_02_sys_template_ready | Success | 0.124
    test_02_start_vm | Success | 10.202
    test_02_router_internal_adv | Success | 0.976
    test_02_redundant_VPC_default_routes | Success | 439.610
    test_02_port_fwd_on_non_src_nat | Success | 55.807
    test_02_list_cpvm_vm | Success | 0.137
    test_02_isolate_network_FW_PF_default_routes_egress_false | Success | 188.450
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | Success | 541.607
    test_02_edit_template | Success | 90.131
    test_02_edit_service_offering | Success | 0.068
    test_02_edit_iso | Success | 0.091
    test_02_edit_disk_offering | Success | 0.061
    test_02_deploy_vm_root_resize | Success | 7.028
    test_02_create_sparse_type_disk_offering | Success | 0.088
    test_02_create_lb_rule_non_nat | Success | 187.532
    test_02_attach_volume | Success | 44.317
    test_02_VPC_default_routes | Success | 260.523
    test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | Success | 271.552
    test_01_updatevolumedetail | Success | 0.108
    test_01_sys_vm_start | Success | 0.181
    test_01_stop_vm | Success | 35.312
    test_01_router_internal_basic | Success | 0.504
    test_01_reset_vm_on_reboot | Success | 131.035
    test_01_primary_storage_nfs | Success | 35.830
    test_01_port_fwd_on_src_nat | Success | 109.860
    test_01_nic | Success | 753.174
    test_01_list_volumes | Success | 0.025
    test_01_list_sec_storage_vm | Success | 0.153
    test_01_isolate_network_FW_PF_default_routes_egress_true | Success | 192.037
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Success | 487.103
    test_01_deploy_vm_root_resize | Success | 7.024
    test_01_create_volume | Success | 621.370
    test_01_create_template | Success | 35.463
    test_01_create_service_offering | Success | 0.140
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 979.739
    test_01_create_lb_rule_src_nat | Success | 208.595
    test_01_create_iso | Success | 66.911
    test_01_create_disk_offering | Success | 0.133
    test_01_VPC_nics_after_destroy | Success | 456.835
    test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | Success | 256.421
    test_00_deploy_vm_root_resize | Success | 224.827
    login_test_saml_user | Success | 24.906
    test_vm_nic_adapter_vmxnet3 | Skipped | 0.001
    test_static_role_account_acls | Skipped | 0.493
    test_rolepermission_lifecycle_update | Skipped | 0.028
    test_rolepermission_lifecycle_list | Skipped | 0.021
    test_rolepermission_lifecycle_delete | Skipped | 0.023
    test_rolepermission_lifecycle_create | Skipped | 0.022
    test_rolepermission_lifecycle_concurrent_updates | Skipped | 0.025
    test_role_lifecycle_update_role_inuse | Skipped | 0.020
    test_role_lifecycle_update | Skipped | 0.022
    test_role_lifecycle_list | Skipped | 0.025
    test_role_lifecycle_delete | Skipped | 0.025
    test_role_lifecycle_create | Skipped | 0.026
    test_role_inuse_deletion | Skipped | 0.018
    test_role_account_acls_multiple_mgmt_servers | Skipped | 0.017
    test_role_account_acls | Skipped | 0.020
    test_deploy_vgpu_enabled_vm | Skipped | 0.005
    test_default_role_deletion | Skipped | 0.029
    test_06_copy_template | Skipped | 0.000
    test_06_copy_iso | Skipped | 0.000
    test_03_delete_vm_snapshots | Skipped | 0.000
    test_02_revert_vm_snapshots | Skipped | 0.000
    test_01_test_vm_volume_snapshot | Skipped | 0.001
    test_01_scale_vm | Skipped | 0.001
    test_01_primary_storage_iscsi | Skipped | 0.046
    test_01_create_vm_snapshots | Skipped | 0.000



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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    The previous test run was incorrect, several tests were run from master the features that don't exist on 4.8.
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1694: CLOUDSTACK-9509: Host Connects Without Storag...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1694#discussion_r80714972
  
    --- Diff: server/src/com/cloud/storage/listener/StoragePoolMonitor.java ---
    @@ -99,12 +99,12 @@ public void processConnect(Host host, StartupCommand cmd, boolean forRebalance)
                         }
     
                         Long hostId = host.getId();
    -                    s_logger.debug("Host " + hostId + " connected, sending down storage pool information ...");
    +                    s_logger.debug("Host " + hostId + " connected, connecting host to shared pool id " + pool.getId() + " and sending storage pool information ...");
    --- End diff --
    
    Please wrap in a `if (s_logger.isDebugEnabled)` to check to avoid unnecessary string concatenation when the `DEBUG` level is disabled.


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @murali-reddy @jburwell can we have lgtm/review on this, 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by abhinandanprateek <gi...@git.apache.org>.
Github user abhinandanprateek commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @rhtyd LGTM on code review


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    <b>Trillian test result (tid-166)</b>
    Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
    Total time taken: 25743 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1694-t166-kvm-centos7.zip
    Test completed. 42 look ok, 1 have error(s)
    
    
    Test | Result | Time (s) | Test File
    --- | --- | --- | ---
    test_02_redundant_VPC_default_routes | `Failure` | 871.35 | test_vpc_redundant.py
    test_01_vpc_site2site_vpn | Success | 165.39 | test_vpc_vpn.py
    test_01_vpc_remote_access_vpn | Success | 56.76 | test_vpc_vpn.py
    test_01_redundant_vpc_site2site_vpn | Success | 231.90 | test_vpc_vpn.py
    test_02_VPC_default_routes | Success | 295.84 | test_vpc_router_nics.py
    test_01_VPC_nics_after_destroy | Success | 537.10 | test_vpc_router_nics.py
    test_05_rvpc_multi_tiers | Success | 515.78 | test_vpc_redundant.py
    test_04_rvpc_network_garbage_collector_nics | Success | 1455.31 | test_vpc_redundant.py
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 561.50 | test_vpc_redundant.py
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1310.96 | test_vpc_redundant.py
    test_09_delete_detached_volume | Success | 16.23 | test_volumes.py
    test_08_resize_volume | Success | 15.48 | test_volumes.py
    test_07_resize_fail | Success | 20.57 | test_volumes.py
    test_06_download_detached_volume | Success | 15.46 | 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.35 | test_volumes.py
    test_02_attach_volume | Success | 74.17 | test_volumes.py
    test_01_create_volume | Success | 714.87 | test_volumes.py
    test_deploy_vm_multiple | Success | 249.49 | test_vm_life_cycle.py
    test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
    test_advZoneVirtualRouter | Success | 0.04 | test_vm_life_cycle.py
    test_10_attachAndDetach_iso | Success | 26.98 | test_vm_life_cycle.py
    test_09_expunge_vm | Success | 125.21 | test_vm_life_cycle.py
    test_08_migrate_vm | Success | 41.00 | test_vm_life_cycle.py
    test_07_restore_vm | Success | 0.12 | test_vm_life_cycle.py
    test_06_destroy_vm | Success | 130.95 | test_vm_life_cycle.py
    test_03_reboot_vm | Success | 126.14 | test_vm_life_cycle.py
    test_02_start_vm | Success | 10.20 | test_vm_life_cycle.py
    test_01_stop_vm | Success | 35.32 | test_vm_life_cycle.py
    test_CreateTemplateWithDuplicateName | Success | 65.74 | test_templates.py
    test_08_list_system_templates | Success | 0.06 | test_templates.py
    test_07_list_public_templates | Success | 0.16 | test_templates.py
    test_05_template_permissions | Success | 0.06 | test_templates.py
    test_04_extract_template | Success | 5.38 | test_templates.py
    test_03_delete_template | Success | 5.10 | test_templates.py
    test_02_edit_template | Success | 90.18 | test_templates.py
    test_01_create_template | Success | 66.06 | test_templates.py
    test_10_destroy_cpvm | Success | 166.61 | test_ssvm.py
    test_09_destroy_ssvm | Success | 193.87 | test_ssvm.py
    test_08_reboot_cpvm | Success | 101.39 | test_ssvm.py
    test_07_reboot_ssvm | Success | 133.27 | test_ssvm.py
    test_06_stop_cpvm | Success | 131.85 | test_ssvm.py
    test_05_stop_ssvm | Success | 163.67 | test_ssvm.py
    test_04_cpvm_internals | Success | 1.25 | test_ssvm.py
    test_03_ssvm_internals | Success | 4.23 | test_ssvm.py
    test_02_list_cpvm_vm | Success | 0.11 | test_ssvm.py
    test_01_list_sec_storage_vm | Success | 0.12 | test_ssvm.py
    test_01_snapshot_root_disk | Success | 21.44 | test_snapshots.py
    test_04_change_offering_small | Success | 210.04 | 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.15 | test_secondary_storage.py
    test_01_sys_vm_start | Success | 0.23 | test_secondary_storage.py
    test_09_reboot_router | Success | 35.37 | test_routers.py
    test_08_start_router | Success | 35.38 | test_routers.py
    test_07_stop_router | Success | 10.19 | 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.61 | test_routers.py
    test_03_restart_network_cleanup | Success | 55.70 | test_routers.py
    test_02_router_internal_adv | Success | 1.08 | test_routers.py
    test_01_router_internal_basic | Success | 0.57 | test_routers.py
    test_router_dhcphosts | Success | 279.41 | test_router_dhcphosts.py
    test_01_updatevolumedetail | Success | 5.21 | test_resource_detail.py
    test_01_reset_vm_on_reboot | Success | 176.58 | test_reset_vm_on_reboot.py
    test_createRegion | Success | 0.04 | test_regions.py
    test_create_pvlan_network | Success | 5.29 | test_pvlan.py
    test_dedicatePublicIpRange | Success | 0.41 | test_public_ip_range.py
    test_04_rvpc_privategw_static_routes | Success | 534.66 | test_privategw_acl.py
    test_03_vpc_privategw_restart_vpc_cleanup | Success | 483.23 | test_privategw_acl.py
    test_02_vpc_privategw_static_routes | Success | 424.18 | test_privategw_acl.py
    test_01_vpc_privategw_acl | Success | 88.83 | test_privategw_acl.py
    test_01_primary_storage_nfs | Success | 35.88 | test_primary_storage.py
    test_createPortablePublicIPRange | Success | 10.25 | test_portable_publicip.py
    test_createPortablePublicIPAcquire | Success | 15.52 | test_portable_publicip.py
    test_isolate_network_password_server | Success | 90.11 | test_password_server.py
    test_UpdateStorageOverProvisioningFactor | Success | 0.18 | test_over_provisioning.py
    test_extendPhysicalNetworkVlan | Success | 15.52 | test_non_contigiousvlan.py
    test_01_nic | Success | 789.01 | test_nic.py
    test_releaseIP | Success | 183.67 | test_network.py
    test_reboot_router | Success | 445.92 | test_network.py
    test_public_ip_user_account | Success | 10.41 | test_network.py
    test_public_ip_admin_account | Success | 40.32 | test_network.py
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 67.00 | test_network.py
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 61.85 | test_network.py
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 121.85 | test_network.py
    test_delete_account | Success | 339.30 | test_network.py
    test_02_port_fwd_on_non_src_nat | Success | 55.82 | test_network.py
    test_01_port_fwd_on_src_nat | Success | 109.78 | test_network.py
    test_nic_secondaryip_add_remove | Success | 224.04 | test_multipleips_per_nic.py
    login_test_saml_user | Success | 25.60 | test_login.py
    test_assign_and_removal_lb | Success | 133.65 | test_loadbalance.py
    test_02_create_lb_rule_non_nat | Success | 187.42 | test_loadbalance.py
    test_01_create_lb_rule_src_nat | Success | 187.79 | test_loadbalance.py
    test_07_list_default_iso | Success | 0.10 | test_iso.py
    test_05_iso_permissions | Success | 0.09 | test_iso.py
    test_04_extract_Iso | Success | 5.40 | test_iso.py
    test_03_delete_iso | Success | 95.23 | test_iso.py
    test_02_edit_iso | Success | 0.07 | test_iso.py
    test_01_create_iso | Success | 21.92 | test_iso.py
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Success | 284.76 | test_internal_lb.py
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | Success | 183.92 | test_internal_lb.py
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | Success | 567.47 | test_internal_lb.py
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Success | 462.28 | test_internal_lb.py
    test_dedicateGuestVlanRange | Success | 10.57 | test_guest_vlan_range.py
    test_UpdateConfigParamWithScope | Success | 0.29 | test_global_settings.py
    test_04_create_fat_type_disk_offering | Success | 0.07 | test_disk_offerings.py
    test_03_delete_disk_offering | Success | 0.03 | test_disk_offerings.py
    test_02_edit_disk_offering | Success | 0.05 | test_disk_offerings.py
    test_02_create_sparse_type_disk_offering | Success | 0.06 | test_disk_offerings.py
    test_01_create_disk_offering | Success | 0.10 | test_disk_offerings.py
    test_deployvm_userdispersing | Success | 21.01 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userconcentrated | Success | 86.13 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_firstfit | Success | 71.00 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userdata_post | Success | 10.82 | test_deploy_vm_with_userdata.py
    test_deployvm_userdata | Success | 55.78 | test_deploy_vm_with_userdata.py
    test_02_deploy_vm_root_resize | Success | 6.94 | test_deploy_vm_root_resize.py
    test_01_deploy_vm_root_resize | Success | 7.00 | test_deploy_vm_root_resize.py
    test_00_deploy_vm_root_resize | Success | 279.12 | test_deploy_vm_root_resize.py
    test_deploy_vm_from_iso | Success | 215.27 | test_deploy_vm_iso.py
    test_DeployVmAntiAffinityGroup | Success | 96.25 | test_affinity_groups.py
    test_03_delete_vm_snapshots | Skipped | 0.00 | test_vm_snapshots.py
    test_02_revert_vm_snapshots | Skipped | 0.00 | test_vm_snapshots.py
    test_01_test_vm_volume_snapshot | Skipped | 0.00 | test_vm_snapshots.py
    test_01_create_vm_snapshots | Skipped | 0.00 | test_vm_snapshots.py
    test_06_copy_template | Skipped | 0.00 | test_templates.py
    test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
    test_01_primary_storage_iscsi | Skipped | 0.07 | test_primary_storage.py
    test_06_copy_iso | Skipped | 0.00 | test_iso.py
    test_deploy_vgpu_enabled_vm | Skipped | 0.00 | test_deploy_vgpu_enabled_vm.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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1694: CLOUDSTACK-9509: Host Connects Without Storag...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1694#discussion_r80853405
  
    --- Diff: server/test/com/cloud/storage/listener/StoragePoolMonitorTest.java ---
    @@ -0,0 +1,80 @@
    +// Licensed to the Apache Software Foundation (ASF) under one
    +// or more contributor license agreements.  See the NOTICE file
    +// distributed with this work for additional information
    +// regarding copyright ownership.  The ASF licenses this file
    +// to you under the Apache License, Version 2.0 (the
    +// "License"); you may not use this file except in compliance
    +// with the License.  You may obtain a copy of the License at
    +//
    +//   http://www.apache.org/licenses/LICENSE-2.0
    +//
    +// Unless required by applicable law or agreed to in writing,
    +// software distributed under the License is distributed on an
    +// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    +// KIND, either express or implied.  See the License for the
    +// specific language governing permissions and limitations
    +// under the License.
    +package com.cloud.storage.listener;
    +
    +import com.cloud.agent.api.StartupRoutingCommand;
    +import com.cloud.exception.ConnectionException;
    +import com.cloud.exception.StorageUnavailableException;
    +import com.cloud.host.HostVO;
    +import com.cloud.hypervisor.Hypervisor;
    +import com.cloud.storage.ScopeType;
    +import com.cloud.storage.StorageManagerImpl;
    +import com.cloud.storage.StoragePoolStatus;
    +import org.apache.cloudstack.storage.datastore.db.PrimaryDataStoreDao;
    +import org.apache.cloudstack.storage.datastore.db.StoragePoolVO;
    +import org.junit.Before;
    +import org.junit.Test;
    +import org.mockito.Mockito;
    +
    +import java.util.Collections;
    +
    +public class StoragePoolMonitorTest {
    +
    +    private StorageManagerImpl storageManager;
    +    private PrimaryDataStoreDao poolDao;
    +    private StoragePoolMonitor storagePoolMonitor;
    +    private HostVO host;
    +    private StoragePoolVO pool;
    +    private StartupRoutingCommand cmd;
    +
    +    @Before
    +    public void setUp() throws Exception {
    +        storageManager = Mockito.mock(StorageManagerImpl.class);
    +        poolDao = Mockito.mock(PrimaryDataStoreDao.class);
    +
    +        storagePoolMonitor = new StoragePoolMonitor(storageManager, poolDao);
    +        host = new HostVO("some-uuid");
    --- End diff --
    
    This a unit test, uniqueness of the VO object is not necessary. We just want to create a hostVO object for testing that could have any uuid.


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    Thanks for sharing @nathanejohnson in case you've already fixed it, can you send a PR? Otherwise, it's late for me, I'll have a look first in the morning.


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @blueorangutan test centos6 kvm-centos7


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    Packaging result: \u2714centos6 \u2714centos7 \u2714debian. JID-89


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    Packaging result: \u2714centos6 \u2714centos7 \u2714debian. JID-30


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

[GitHub] cloudstack pull request #1694: CLOUDSTACK-9509: Host Connects Without Storag...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    Merging this based on tests results and review, the failures caused in VPC/VR related tests are known intermittent issues.


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by nathanejohnson <gi...@git.apache.org>.
Github user nathanejohnson commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @rthyd ahh, it didn't occur to me to just pass null.  I was trying to initialize that third parameter with a mock (like the other two).
    
    Thanks again!


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

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    <b>Trillian test result (tid-167)</b>
    Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
    Total time taken: 32856 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1694-t167-vmware-55u3.zip
    Test completed. 41 look ok, 2 have error(s)
    
    
    Test | Result | Time (s) | Test File
    --- | --- | --- | ---
    test_01_vpc_site2site_vpn | `Error` | 502.52 | test_vpc_vpn.py
    ContextSuite context=TestRVPCSite2SiteVpn>:setup | `Error` | 0.00 | test_vpc_vpn.py
    test_06_download_detached_volume | `Error` | 25.52 | test_volumes.py
    test_01_vpc_remote_access_vpn | Success | 147.16 | test_vpc_vpn.py
    test_02_VPC_default_routes | Success | 324.92 | test_vpc_router_nics.py
    test_01_VPC_nics_after_destroy | Success | 740.48 | test_vpc_router_nics.py
    test_05_rvpc_multi_tiers | Success | 573.72 | test_vpc_redundant.py
    test_04_rvpc_network_garbage_collector_nics | Success | 1551.90 | test_vpc_redundant.py
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 618.12 | test_vpc_redundant.py
    test_02_redundant_VPC_default_routes | Success | 544.01 | test_vpc_redundant.py
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1161.38 | test_vpc_redundant.py
    test_09_delete_detached_volume | Success | 30.88 | test_volumes.py
    test_05_detach_volume | Success | 100.29 | test_volumes.py
    test_04_delete_attached_volume | Success | 10.20 | test_volumes.py
    test_03_download_attached_volume | Success | 20.60 | test_volumes.py
    test_02_attach_volume | Success | 58.95 | test_volumes.py
    test_01_create_volume | Success | 445.61 | test_volumes.py
    test_03_delete_vm_snapshots | Success | 275.23 | test_vm_snapshots.py
    test_02_revert_vm_snapshots | Success | 199.49 | test_vm_snapshots.py
    test_01_test_vm_volume_snapshot | Success | 227.10 | test_vm_snapshots.py
    test_01_create_vm_snapshots | Success | 161.65 | test_vm_snapshots.py
    test_deploy_vm_multiple | Success | 299.42 | 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 | 27.03 | test_vm_life_cycle.py
    test_09_expunge_vm | Success | 125.21 | test_vm_life_cycle.py
    test_08_migrate_vm | Success | 71.31 | test_vm_life_cycle.py
    test_07_restore_vm | Success | 0.10 | test_vm_life_cycle.py
    test_06_destroy_vm | Success | 5.11 | test_vm_life_cycle.py
    test_03_reboot_vm | Success | 5.24 | test_vm_life_cycle.py
    test_02_start_vm | Success | 20.35 | test_vm_life_cycle.py
    test_01_stop_vm | Success | 5.11 | test_vm_life_cycle.py
    test_CreateTemplateWithDuplicateName | Success | 226.98 | 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.05 | test_templates.py
    test_04_extract_template | Success | 16.32 | test_templates.py
    test_03_delete_template | Success | 5.21 | test_templates.py
    test_02_edit_template | Success | 90.17 | test_templates.py
    test_01_create_template | Success | 106.06 | test_templates.py
    test_10_destroy_cpvm | Success | 237.07 | test_ssvm.py
    test_09_destroy_ssvm | Success | 269.20 | test_ssvm.py
    test_08_reboot_cpvm | Success | 186.89 | test_ssvm.py
    test_07_reboot_ssvm | Success | 158.79 | test_ssvm.py
    test_06_stop_cpvm | Success | 208.19 | test_ssvm.py
    test_05_stop_ssvm | Success | 215.19 | test_ssvm.py
    test_04_cpvm_internals | Success | 2.04 | test_ssvm.py
    test_03_ssvm_internals | Success | 5.14 | test_ssvm.py
    test_02_list_cpvm_vm | Success | 0.12 | test_ssvm.py
    test_01_list_sec_storage_vm | Success | 0.12 | test_ssvm.py
    test_01_snapshot_root_disk | Success | 66.74 | test_snapshots.py
    test_04_change_offering_small | Success | 127.20 | test_service_offerings.py
    test_03_delete_service_offering | Success | 0.13 | test_service_offerings.py
    test_02_edit_service_offering | Success | 0.26 | test_service_offerings.py
    test_01_create_service_offering | Success | 0.13 | test_service_offerings.py
    test_02_sys_template_ready | Success | 0.19 | test_secondary_storage.py
    test_01_sys_vm_start | Success | 0.31 | test_secondary_storage.py
    test_09_reboot_router | Success | 136.24 | test_routers.py
    test_08_start_router | Success | 131.20 | test_routers.py
    test_07_stop_router | Success | 20.21 | 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.93 | test_routers.py
    test_03_restart_network_cleanup | Success | 186.45 | test_routers.py
    test_02_router_internal_adv | Success | 1.02 | test_routers.py
    test_01_router_internal_basic | Success | 0.56 | test_routers.py
    test_router_dhcphosts | Success | 149.32 | test_router_dhcphosts.py
    test_01_updatevolumedetail | Success | 0.08 | test_resource_detail.py
    test_01_reset_vm_on_reboot | Success | 30.37 | test_reset_vm_on_reboot.py
    test_createRegion | Success | 0.04 | test_regions.py
    test_create_pvlan_network | Success | 5.23 | test_pvlan.py
    test_dedicatePublicIpRange | Success | 0.46 | test_public_ip_range.py
    test_04_rvpc_privategw_static_routes | Success | 942.49 | test_privategw_acl.py
    test_03_vpc_privategw_restart_vpc_cleanup | Success | 922.44 | test_privategw_acl.py
    test_02_vpc_privategw_static_routes | Success | 595.39 | test_privategw_acl.py
    test_01_vpc_privategw_acl | Success | 181.37 | test_privategw_acl.py
    test_01_primary_storage_nfs | Success | 37.58 | test_primary_storage.py
    test_createPortablePublicIPRange | Success | 15.23 | test_portable_publicip.py
    test_createPortablePublicIPAcquire | Success | 16.02 | test_portable_publicip.py
    test_isolate_network_password_server | Success | 93.44 | test_password_server.py
    test_UpdateStorageOverProvisioningFactor | Success | 0.19 | test_over_provisioning.py
    test_extendPhysicalNetworkVlan | Success | 15.46 | test_non_contigiousvlan.py
    test_01_nic | Success | 805.85 | test_nic.py
    test_releaseIP | Success | 244.57 | test_network.py
    test_reboot_router | Success | 652.08 | test_network.py
    test_public_ip_user_account | Success | 10.26 | test_network.py
    test_public_ip_admin_account | Success | 40.52 | test_network.py
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 67.22 | test_network.py
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 61.81 | test_network.py
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 124.67 | test_network.py
    test_delete_account | Success | 329.81 | test_network.py
    test_02_port_fwd_on_non_src_nat | Success | 55.75 | test_network.py
    test_01_port_fwd_on_src_nat | Success | 111.98 | test_network.py
    test_nic_secondaryip_add_remove | Success | 214.49 | test_multipleips_per_nic.py
    login_test_saml_user | Success | 27.46 | test_login.py
    test_assign_and_removal_lb | Success | 134.22 | test_loadbalance.py
    test_02_create_lb_rule_non_nat | Success | 187.85 | test_loadbalance.py
    test_01_create_lb_rule_src_nat | Success | 187.93 | test_loadbalance.py
    test_07_list_default_iso | Success | 0.06 | test_iso.py
    test_05_iso_permissions | Success | 0.06 | test_iso.py
    test_04_extract_Iso | Success | 5.17 | test_iso.py
    test_03_delete_iso | Success | 95.20 | test_iso.py
    test_02_edit_iso | Success | 0.06 | test_iso.py
    test_01_create_iso | Success | 22.08 | test_iso.py
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Success | 561.02 | test_internal_lb.py
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | Success | 406.00 | test_internal_lb.py
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | Success | 920.72 | test_internal_lb.py
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Success | 779.82 | test_internal_lb.py
    test_dedicateGuestVlanRange | Success | 10.49 | test_guest_vlan_range.py
    test_UpdateConfigParamWithScope | Success | 0.31 | test_global_settings.py
    test_04_create_fat_type_disk_offering | Success | 0.06 | test_disk_offerings.py
    test_03_delete_disk_offering | Success | 0.04 | test_disk_offerings.py
    test_02_edit_disk_offering | Success | 0.05 | test_disk_offerings.py
    test_02_create_sparse_type_disk_offering | Success | 0.06 | test_disk_offerings.py
    test_01_create_disk_offering | Success | 0.10 | test_disk_offerings.py
    test_deployvm_userdispersing | Success | 56.41 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userconcentrated | Success | 132.22 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_firstfit | Success | 206.85 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userdata_post | Success | 61.06 | test_deploy_vm_with_userdata.py
    test_deployvm_userdata | Success | 277.70 | test_deploy_vm_with_userdata.py
    test_02_deploy_vm_root_resize | Success | 6.79 | test_deploy_vm_root_resize.py
    test_01_deploy_vm_root_resize | Success | 7.72 | test_deploy_vm_root_resize.py
    test_00_deploy_vm_root_resize | Success | 7.12 | test_deploy_vm_root_resize.py
    test_deploy_vm_from_iso | Success | 184.19 | test_deploy_vm_iso.py
    test_DeployVmAntiAffinityGroup | Success | 192.11 | test_affinity_groups.py
    test_08_resize_volume | Skipped | 10.17 | test_volumes.py
    test_07_resize_fail | Skipped | 15.34 | test_volumes.py
    test_06_copy_template | Skipped | 0.00 | test_templates.py
    test_01_scale_vm | Skipped | 33.68 | test_scale_vm.py
    test_01_primary_storage_iscsi | Skipped | 0.06 | test_primary_storage.py
    test_06_copy_iso | Skipped | 0.00 | test_iso.py
    test_deploy_vgpu_enabled_vm | Skipped | 0.01 | test_deploy_vgpu_enabled_vm.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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1694
  
    @rhtyd a Trillian-Jenkins test job (centos6 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1694: CLOUDSTACK-9509: Host Connects Without Storag...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1694#discussion_r80853668
  
    --- Diff: server/src/com/cloud/storage/listener/StoragePoolMonitor.java ---
    @@ -99,12 +99,12 @@ public void processConnect(Host host, StartupCommand cmd, boolean forRebalance)
                         }
     
                         Long hostId = host.getId();
    -                    s_logger.debug("Host " + hostId + " connected, sending down storage pool information ...");
    +                    s_logger.debug("Host " + hostId + " connected, connecting host to shared pool id " + pool.getId() + " and sending storage pool information ...");
    --- End diff --
    
    Fixed.


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