Skip to content

Ignore site to site vpn status check on internallbvm - #3864

Merged
yadvr merged 1 commit into
apache:4.13from
ravening:ignore_s2s_status_ilb
Feb 18, 2020
Merged

Ignore site to site vpn status check on internallbvm#3864
yadvr merged 1 commit into
apache:4.13from
ravening:ignore_s2s_status_ilb

Conversation

@ravening

Copy link
Copy Markdown
Member

Description

When the state of the site to site vpn changes, the check
is done on all the virtual routers including the internal
load balancing vm as well. It is not needed to check the
state for internal load balancing vm

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):

How Has This Been Tested?

Steps to reprodcue the issue

1 . Create two VPC's. Configure VPN customer gateway and VPN connection. The steps are clearly explained here
2 . Now navigate to networks section in main cloudstack ui. In the drop down menu select VPC. Click configure on one of the VPC
3 . Click on "Create Network" and create a tier with "DefaultisolatedNetworkOfferingForVPCNetworksWithInternalLB" network offering
4 . Once the tier is created click on "Internal LB" tab.
5 . In the right side, click on "Add Internal LB" button
6 . Enter the input according to your needs and press OK
7 . Once the Internal LB is created, click on it and select the "Assigned VM's" tab.
8 . Click on "Assign VM's" button and select the VM's which are present in this tier. If none are present then create vm's.
9 . This will create a new type of router "internallb" which can be listed using the cloudmonkey command "list internalloadbalancervms" whose name starts with b-
10 . Change the state of any VPN connection in this VPC and observe the logs for the keyword "Site-to-site Vpn Connection to on router b-<>-VM"

Expected result:

A log mesasge mentioned in the step 10 will be displayed in /var/log/cloudstack/management/management-server.log

Actual result:

The above log should not be displayed.

@yadvryadvr added this to the 4.14.0.0 milestone Feb 6, 2020
@andrijapanicsb
andrijapanicsb self-requested a review February 7, 2020 12:39
@andrijapanicsb

Copy link
Copy Markdown
Contributor

@ravening since this is a bug fix, do you mind rebasing against 4.13 so we can merge it there and merge forward to master afterwards?

@yadvr

Copy link
Copy Markdown
Member

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@ravening
raveningforce-pushed the ignore_s2s_status_ilb branch from 164f984 to 4630e85CompareFebruary 12, 2020 09:30
@blueorangutan

Copy link
Copy Markdown

Packaging result: ✖centos6 ✖centos7 ✔debian. JID-817

When the state of the site to site vpn changes, the check
is done on all the virtual routers including the internal
load balancing vm as well. It is not needed to check the
state for internal load balancing vm
@ravening
raveningforce-pushed the ignore_s2s_status_ilb branch from 4630e85 to 2215f8bCompareFebruary 12, 2020 10:04
@ravening
ravening changed the base branch from master to 4.13February 12, 2020 10:08
@ravening

Copy link
Copy Markdown
MemberAuthor

@ravening since this is a bug fix, do you mind rebasing against 4.13 so we can merge it there and merge forward to master afterwards?

Rebased to 4.13

@DaanHooglandDaanHoogland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code lgtm

@yadvryadvr closed this Feb 12, 2020
@yadvryadvr reopened this Feb 12, 2020
@yadvr

Copy link
Copy Markdown
Member

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-839

@yadvr

Copy link
Copy Markdown
Member

@blueorangutan test

@blueorangutan

Copy link
Copy Markdown

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan

Copy link
Copy Markdown

Trillian test result (tid-963)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31658 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3864-t963-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_domain_service_offerings.py
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_iso.py
Intermittent failure detected: /marvin/tests/smoke/test_list_ids_parameter.py
Intermittent failure detected: /marvin/tests/smoke/test_loadbalance.py
Intermittent failure detected: /marvin/tests/smoke/test_metrics_api.py
Intermittent failure detected: /marvin/tests/smoke/test_migration.py
Intermittent failure detected: /marvin/tests/smoke/test_multipleips_per_nic.py
Intermittent failure detected: /marvin/tests/smoke/test_nested_virtualization.py
Intermittent failure detected: /marvin/tests/smoke/test_network_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_network.py
Intermittent failure detected: /marvin/tests/smoke/test_nic.py
Intermittent failure detected: /marvin/tests/smoke/test_password_server.py
Intermittent failure detected: /marvin/tests/smoke/test_portforwardingrules.py
Intermittent failure detected: /marvin/tests/smoke/test_primary_storage.py
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_projects.py
Intermittent failure detected: /marvin/tests/smoke/test_public_ip_range.py
Intermittent failure detected: /marvin/tests/smoke/test_reset_vm_on_reboot.py
Intermittent failure detected: /marvin/tests/smoke/test_resource_accounting.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dhcphosts.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dns.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dnsservice.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_iptables_default_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermittent failure detected: /marvin/tests/smoke/test_routers.py
Intermittent failure detected: /marvin/tests/smoke/test_secondary_storage.py
Intermittent failure detected: /marvin/tests/smoke/test_service_offerings.py
Intermittent failure detected: /marvin/tests/smoke/test_snapshots.py
Intermittent failure detected: /marvin/tests/smoke/test_ssvm.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Intermittent failure detected: /marvin/tests/smoke/test_usage.py
Intermittent failure detected: /marvin/tests/smoke/test_vm_deployment_planner.py
Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Intermittent failure detected: /marvin/tests/smoke/test_vm_snapshots.py
Intermittent failure detected: /marvin/tests/smoke/test_volumes.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Intermittent failure detected: /marvin/tests/smoke/test_host_maintenance.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 34 look OK, 39 have error(s)
Only failed tests results shown below:

TestResultTime (s)Test File
ContextSuite context=TestLoadBalance>:setupError0.00test_loadbalance.py
test_list_vms_metricsError0.16test_metrics_api.py
test_01_native_to_native_network_migrationError4.32test_migration.py
test_02_native_to_native_vpc_migrationError6.04test_migration.py
ContextSuite context=TestInternalLb>:setupError0.00test_internal_lb.py
ContextSuite context=TestNetworkACL>:setupError0.00test_network_acl.py
test_01_1_create_iso_with_checksum_sha1_negativeError66.58test_iso.py
test_01_create_iso_with_checksum_sha1Error65.50test_iso.py
test_01_create_iso_with_checksum_sha1Error66.59test_iso.py
test_02_1_create_iso_with_checksum_sha256_negativeError66.58test_iso.py
test_02_create_iso_with_checksum_sha256Error65.50test_iso.py
test_02_create_iso_with_checksum_sha256Error66.58test_iso.py
test_03_1_create_iso_with_checksum_md5_negativeError66.58test_iso.py
test_03_create_iso_with_checksum_md5Error65.49test_iso.py
test_03_create_iso_with_checksum_md5Error66.57test_iso.py
test_04_create_iso_with_no_checksumError65.50test_iso.py
test_04_create_iso_with_no_checksumError66.59test_iso.py
test_01_create_isoFailure1514.61test_iso.py
ContextSuite context=TestISO>:setupError3030.32test_iso.py
ContextSuite context=TestListIdsParams>:setupError0.00test_list_ids_parameter.py
test_nic_secondaryip_add_removeError0.08test_multipleips_per_nic.py
ContextSuite context=TestNestedVirtualization>:setupError0.00test_nested_virtualization.py
test_delete_accountError0.78test_network.py
test_delete_network_while_vm_on_itError0.08test_network.py
test_deploy_vm_l2networkError0.09test_network.py
test_l2network_restartError1.29test_network.py
ContextSuite context=TestPortForwarding>:setupError3.60test_network.py
ContextSuite context=TestPublicIP>:setupError5.38test_network.py
test_reboot_routerError0.07test_network.py
test_releaseIPError0.76test_network.py
ContextSuite context=TestRouterRules>:setupError0.86test_network.py
test_01_nicError2.46test_nic.py
ContextSuite context=TestIsolatedNetworksPasswdServer>:setupError0.00test_password_server.py
ContextSuite context=TestPortForwardingRules>:setupError0.00test_portforwardingrules.py
test_01_add_primary_storage_disabled_hostError0.22test_primary_storage.py
test_01_primary_storage_nfsError0.17test_primary_storage.py
ContextSuite context=TestStorageTags>:setupError0.31test_primary_storage.py
ContextSuite context=TestPrivateGwACL>:setupError0.00test_privategw_acl.py
test_09_project_suspendError1.12test_projects.py
test_10_project_activationError1.09test_projects.py
ContextSuite context=TestResetVmOnReboot>:setupError0.00test_reset_vm_on_reboot.py
ContextSuite context=TestRAMCPUResourceAccounting>:setupError0.00test_resource_accounting.py
test_01_sys_vm_startFailure0.10test_secondary_storage.py
ContextSuite context=TestRouterDHCPHosts>:setupError0.00test_router_dhcphosts.py
ContextSuite context=TestRouterDHCPOpts>:setupError0.00test_router_dhcphosts.py
ContextSuite context=TestRouterDns>:setupError0.00test_router_dns.py
ContextSuite context=TestCpuCapServiceOfferings>:setupError0.00test_service_offerings.py
ContextSuite context=TestServiceOfferings>:setupError0.24test_service_offerings.py
ContextSuite context=TestRouterDnsService>:setupError0.00test_router_dnsservice.py
ContextSuite context=TestRouterIpTablesPolicies>:setupError0.00test_routers_iptables_default_policy.py
ContextSuite context=TestVPCIpTablesPolicies>:setupError0.00test_routers_iptables_default_policy.py
test_01_isolate_network_FW_PF_default_routes_egress_trueError0.24test_routers_network_ops.py
test_02_isolate_network_FW_PF_default_routes_egress_falseError0.24test_routers_network_ops.py
ContextSuite context=TestRedundantIsolateNetworks>:setupError1.47test_routers_network_ops.py
ContextSuite context=TestRouterServices>:setupError0.00test_routers.py
ContextSuite context=TestSnapshotRootDisk>:setupError0.00test_snapshots.py
test_03_ssvm_internalsError935.56test_ssvm.py
test_04_cpvm_internalsError940.10test_ssvm.py
test_05_stop_ssvmError1.23test_ssvm.py
test_06_stop_cpvmError2.20test_ssvm.py
test_07_reboot_ssvmError911.41test_ssvm.py
test_08_reboot_cpvmError911.53test_ssvm.py
test_09_destroy_ssvmError2.17test_ssvm.py
test_10_destroy_cpvmError2.19test_ssvm.py
test_02_create_template_with_checksum_sha1Error65.51test_templates.py
test_03_create_template_with_checksum_sha256Error65.52test_templates.py
test_04_create_template_with_checksum_md5Error65.50test_templates.py
test_05_create_template_with_no_checksumError65.51test_templates.py
test_01_register_template_direct_download_flagError0.06test_templates.py
test_02_deploy_vm_from_direct_download_templateError0.00test_templates.py
test_03_deploy_vm_wrong_checksumError0.05test_templates.py
ContextSuite context=TestTemplates>:setupError5.05test_templates.py
ContextSuite context=TestISOUsage>:setupError0.00test_usage.py
ContextSuite context=TestLBRuleUsage>:setupError0.00test_usage.py
ContextSuite context=TestNatRuleUsage>:setupError0.00test_usage.py
ContextSuite context=TestPublicIPUsage>:setupError0.00test_usage.py
ContextSuite context=TestSnapshotUsage>:setupError0.00test_usage.py
ContextSuite context=TestVmUsage>:setupError0.00test_usage.py
ContextSuite context=TestVolumeUsage>:setupError0.00test_usage.py
ContextSuite context=TestVpnUsage>:setupError0.00test_usage.py
test_01_deploy_vm_on_specific_hostError2.39test_vm_deployment_planner.py
test_02_deploy_vm_on_specific_clusterError1.24test_vm_deployment_planner.py
test_03_deploy_vm_on_specific_podError1.29test_vm_deployment_planner.py
test_04_deploy_vm_on_host_override_pod_and_clusterError1.32test_vm_deployment_planner.py
test_05_deploy_vm_on_cluster_override_podError1.26test_vm_deployment_planner.py
ContextSuite context=Test01DeployVM>:setupError0.00test_vm_life_cycle.py
ContextSuite context=Test02VMLifeCycle>:setupError0.00test_vm_life_cycle.py
test_14_secure_to_secure_vm_migrationError0.02test_vm_life_cycle.py
test_15_secured_to_nonsecured_vm_migrationError0.02test_vm_life_cycle.py
test_16_nonsecured_to_secured_vm_migrationError0.01test_vm_life_cycle.py
ContextSuite context=TestVmSnapshot>:setupError2.10test_vm_snapshots.py
ContextSuite context=TestCreateVolume>:setupError0.00test_volumes.py
ContextSuite context=TestVolumes>:setupError0.00test_volumes.py
ContextSuite context=TestVPCRedundancy>:setupError0.00test_vpc_redundant.py
ContextSuite context=TestVPCNics>:setupError0.00test_vpc_router_nics.py
ContextSuite context=TestRVPCSite2SiteVpn>:setupError0.00test_vpc_vpn.py
ContextSuite context=TestVPCSite2SiteVPNMultipleOptions>:setupError0.00test_vpc_vpn.py
ContextSuite context=TestVpcRemoteAccessVpn>:setupError0.00test_vpc_vpn.py
ContextSuite context=TestVpcSite2SiteVpn>:setupError0.00test_vpc_vpn.py
test_01_cancel_host_maintenace_with_no_migration_jobsFailure0.16test_host_maintenance.py
test_02_cancel_host_maintenace_with_migration_jobsError2.64test_host_maintenance.py
test_01_cancel_host_maintenance_ssh_enabled_agent_connectedError0.03test_host_maintenance.py
test_03_cancel_host_maintenance_ssh_disabled_agent_connectedError0.02test_host_maintenance.py
test_04_cancel_host_maintenance_ssh_disabled_agent_disconnectedError0.02test_host_maintenance.py
test_hostha_enable_ha_when_host_disconectedError937.05test_hostha_kvm.py
test_hostha_enable_ha_when_host_in_maintenanceError302.79test_hostha_kvm.py

@yadvr

Copy link
Copy Markdown
Member

@blueorangutan test

@blueorangutan

Copy link
Copy Markdown

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan

Copy link
Copy Markdown

Trillian test result (tid-983)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36042 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3864-t983-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 76 look OK, 1 have error(s)
Only failed tests results shown below:

TestResultTime (s)Test File
test_02_vpc_privategw_static_routesFailure206.47test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanupFailure205.04test_privategw_acl.py
test_04_rvpc_privategw_static_routesFailure307.08test_privategw_acl.py

@DaanHoogland

Copy link
Copy Markdown
Contributor

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-855

@yadvryadvr closed this Feb 14, 2020
@yadvryadvr reopened this Feb 14, 2020
@yadvr

Copy link
Copy Markdown
Member

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-870

@apacheapache deleted a comment from blueorangutanFeb 14, 2020
@DaanHooglandDaanHoogland modified the milestones: 4.14.0.0, 4.13.1.0Feb 14, 2020
@yadvr

Copy link
Copy Markdown
Member

@blueorangutan test

@blueorangutan

Copy link
Copy Markdown

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@yadvr

Copy link
Copy Markdown
Member

@blueorangutan test

@blueorangutan

Copy link
Copy Markdown

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@yadvr

Copy link
Copy Markdown
Member

@blueorangutan test

@blueorangutan

Copy link
Copy Markdown

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan

Copy link
Copy Markdown

Trillian test result (tid-1013)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 56142 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3864-t1013-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_network.py
Intermittent failure detected: /marvin/tests/smoke/test_password_server.py
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dhcphosts.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dns.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dnsservice.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_iptables_default_policy.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermittent failure detected: /marvin/tests/smoke/test_service_offerings.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Intermittent failure detected: /marvin/tests/smoke/test_usage.py
Smoke tests completed. 66 look OK, 11 have error(s)
Only failed tests results shown below:

TestResultTime (s)Test File
test_01_port_fwd_on_src_natFailure813.96test_network.py
test_02_port_fwd_on_non_src_natFailure816.08test_network.py
test_reboot_routerFailure397.13test_network.py
test_network_rules_acquired_public_ip_1_static_nat_ruleFailure815.04test_network.py
test_network_rules_acquired_public_ip_2_nat_ruleFailure816.61test_network.py
test_network_rules_acquired_public_ip_3_Load_Balancer_RuleFailure819.73test_network.py
test_isolate_network_password_serverFailure216.62test_password_server.py
test_02_vpc_privategw_static_routesFailure942.59test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanupFailure941.82test_privategw_acl.py
test_04_rvpc_privategw_static_routesFailure996.95test_privategw_acl.py
test_router_dhcphostsFailure216.57test_router_dhcphosts.py
ContextSuite context=TestRouterDHCPHosts>:teardownError225.82test_router_dhcphosts.py
test_router_dns_guestipqueryFailure613.54test_router_dns.py
test_router_dns_guestipqueryFailure613.79test_router_dnsservice.py
test_01_single_VPC_iptables_policiesError916.62test_routers_iptables_default_policy.py
test_01_isolate_network_FW_PF_default_routes_egress_trueFailure261.04test_routers_network_ops.py
test_02_isolate_network_FW_PF_default_routes_egress_falseFailure261.01test_routers_network_ops.py
test_01_RVR_Network_FW_PF_SSH_default_routes_egress_trueFailure285.13test_routers_network_ops.py
test_02_RVR_Network_FW_PF_SSH_default_routes_egress_falseFailure287.15test_routers_network_ops.py
test_01_service_offering_cpu_limit_useError811.23test_service_offerings.py
test_04_change_offering_smallFailure910.99test_service_offerings.py
test_04_extract_templateFailure128.27test_templates.py
ContextSuite context=TestISOUsage>:setupError0.00test_usage.py

@yadvr

Copy link
Copy Markdown
Member

I checked some issue at the backend. Considering the test on #3864 (comment) LGTM.

@yadvr
yadvr merged commit bbe2bf1 into apache:4.13Feb 18, 2020
nvazquez pushed a commit to shapeblue/cloudstack that referenced this pull request Feb 21, 2020
…3864)
When the state of the site to site vpn changes, the check
is done on all the virtual routers including the internal
load balancing vm as well. It is not needed to check the
state for internal load balancing vm
nvazquez pushed a commit to shapeblue/cloudstack that referenced this pull request Feb 21, 2020
…3864)
When the state of the site to site vpn changes, the check
is done on all the virtual routers including the internal
load balancing vm as well. It is not needed to check the
state for internal load balancing vm
ggoodrich-ipp pushed a commit to ippathways/cloudstack that referenced this pull request Feb 24, 2020
…3864)
When the state of the site to site vpn changes, the check
is done on all the virtual routers including the internal
load balancing vm as well. It is not needed to check the
state for internal load balancing vm
ustcweizhou pushed a commit to ustcweizhou/cloudstack that referenced this pull request Feb 28, 2020
…3864)
When the state of the site to site vpn changes, the check
is done on all the virtual routers including the internal
load balancing vm as well. It is not needed to check the
state for internal load balancing vm
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@ravening@andrijapanicsb@yadvr@blueorangutan@DaanHoogland@weizhouapache@svenvogel