[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1878 @rafaelweingartner Thanks for reviewing, will work on the changes suggested. --- 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 #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1878 @PaulAngus What was the error shown for duplicate MAC in vCenter. Can you post the log snippet with the error. These changes were tested in vCenter 5.5 and 6.0, didn't observe any issues in vCenter. --- 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 #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1878 @sureshanaparti I see in vCenter any deployment of an RVR with a duplicate MAC, failing with a vCenter error regarding a duplicate MAC. --- 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 #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/1878 @sureshanaparti Thanks for the info! --- 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 #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1878 @remibergsma Same MAC for RVR has been re-introducted as part of [CLOUDSTACK-985](https://issues.apache.org/jira/browse/CLOUDSTACK-985). It confirms that peer NICs of RVRs should have same MAC addresses. Only default public NIC was configured with same MAC. For VMware, there exists additional public NICs which were not configured with same MAC addresses. --- 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 #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/1878 @sureshanaparti why do the mac addresses need to be the same on both routers? We're also executing arpings to update our neighbours. Networking wise there is no need for them to be the same. I've seen it on other parts of the code as well and I really wonder why we do 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 #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1878 Trillian test result (tid-873) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 33766 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1878-t873-kvm-centos7.zip Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py Intermitten failure detected: /marvin/tests/smoke/test_vm_life_cycle.py Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py Test completed. 46 look ok, 3 have error(s) Test | Result | Time (s) | Test File --- | --- | --- | --- test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 362.57 | test_vpc_redundant.py test_04_rvpc_privategw_static_routes | `Failure` | 334.37 | test_privategw_acl.py test_02_list_snapshots_with_removed_data_store | `Error` | 0.03 | test_snapshots.py test_01_vpc_site2site_vpn | Success | 159.36 | test_vpc_vpn.py test_01_vpc_remote_access_vpn | Success | 65.83 | test_vpc_vpn.py test_01_redundant_vpc_site2site_vpn | Success | 239.78 | test_vpc_vpn.py test_02_VPC_default_routes | Success | 274.04 | test_vpc_router_nics.py test_01_VPC_nics_after_destroy | Success | 531.49 | test_vpc_router_nics.py test_05_rvpc_multi_tiers | Success | 509.60 | test_vpc_redundant.py test_04_rvpc_network_garbage_collector_nics | Success | 1410.18 | test_vpc_redundant.py test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 536.46 | test_vpc_redundant.py test_02_redundant_VPC_default_routes | Success | 747.15 | test_vpc_redundant.py test_09_delete_detached_volume | Success | 151.49 | test_volumes.py test_08_resize_volume | Success | 156.71 | test_volumes.py test_07_resize_fail | Success | 156.11 | test_volumes.py test_06_download_detached_volume | Success | 155.99 | test_volumes.py test_05_detach_volume | Success | 150.63 | test_volumes.py test_04_delete_attached_volume | Success | 150.93 | test_volumes.py test_03_download_attached_volume | Success | 156.02 | test_volumes.py test_02_attach_volume | Success | 95.59 | test_volumes.py test_01_create_volume | Success | 711.06 | test_volumes.py test_03_delete_vm_snapshots | Success | 275.18 | test_vm_snapshots.py test_02_revert_vm_snapshots | Success | 95.70 | test_vm_snapshots.py test_01_create_vm_snapshots | Success | 158.66 | test_vm_snapshots.py test_deploy_vm_multiple | Success | 267.19 | test_vm_life_cycle.py test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py test_10_attachAndDetach_iso | Success | 26.64 | test_vm_life_cycle.py test_09_expunge_vm | Success | 125.11 | test_vm_life_cycle.py test_08_migrate_vm | Success | 40.67 | test_vm_life_cycle.py test_07_restore_vm | Success | 0.06 | test_vm_life_cycle.py test_06_destroy_vm | Success | 125.92 | test_vm_life_cycle.py test_03_reboot_vm | Success | 126.20 | test_vm_life_cycle.py test_02_start_vm | Success | 10.13 | test_vm_life_cycle.py test_01_stop_vm | Success | 40.26 | test_vm_life_cycle.py test_CreateTemplateWithDuplicateName | Success | 60.49 | test_templates.py test_08_list_system_templates | Success | 0.02 | test_templates.py test_07_list_public_templates | Success | 0.02 | test_templates.py test_05_template_permissions | Success | 0.04 | test_templates.py test_04_extract_template | Success | 5.14 | test_templates.py test_03_delete_template | Success | 5.08 | test_templates.py test_02_edit_template | Success | 90.16 | test_templates.py test_01_create_template | Success | 30.29 | test_templates.py test_10_destroy_cpvm | Success | 131.42 | test_ssvm.py test_09_destroy_ssvm | Success | 168.49 | test_ssvm.py test_08_reboot_cpvm | Success | 131.51 | test_ssvm.py test_07_reboot_ssvm | Success | 133.41 | test_ssvm.py test_06_stop_cpvm | Success | 131.56 | test_ssvm.py test_05_stop_ssvm | Success | 133.54 | test_ssvm.py test_04_cpvm_internals | Success | 1.14 | test_ssvm.py test_03_ssvm_internals | Success | 3.29 | test_ssvm.py test_02_list_cpvm_vm | Success | 0.08 | test_ssvm.py test_01_list_sec_storage_vm | Success | 0.09 | test_ssvm.py test_01_snapshot_root_disk | Success | 10.94 | test_snapshots.py test_04_change_offering_small | Success | 239.55 | test_service_offerings.py test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py test_02_edit_service_offering | Success | 0.04 | test_service_offerings.py test_01_create_service_offering | Success | 0.07 | test_service_offerings.py test_02_sys_template_ready | Success | 0.12 | test_secondary_storage.py test_01_sys_vm_start | Success | 0
[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1878 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1878 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1878 Packaging result: âcentos6 âcentos7 âdebian. JID-519 --- 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 #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1878 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1878 @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 #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1878 @borisstoyanov Can you please run trillian tests for this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1878 @rafaelweingartner Updated the changes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1878 @rafaelweingartner Sure. I'll move these lines to a method and push the changes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1878 Hi @sureshanaparti Would you mind extracting the code at lines 1967-1972 to a specific method? Then, it allows you to write test cases for the newly created method and proper documentation of the method behaviors? --- 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. ---
