Skip to content

With basic zone and VMware hypervisor, VR fails to start#3977

Merged
andrijapanicsb merged 1 commit intoapache:4.13from
shapeblue:VMwareBasicZoneFailsTogetPrivateIPonVR
Mar 27, 2020
Merged

With basic zone and VMware hypervisor, VR fails to start#3977
andrijapanicsb merged 1 commit intoapache:4.13from
shapeblue:VMwareBasicZoneFailsTogetPrivateIPonVR

Conversation

@harikrishna-patnala
Copy link
Copy Markdown
Member

@harikrishna-patnala harikrishna-patnala commented Mar 18, 2020

Description

With basic zone and VMware hypervisor, VR fails to start since eth1 is getting empty instead of a private IP.

Boot Args for VM[DomainRouter|r-4-VM]: template=domP name=r-4-VM eth0ip=10.3.0.89 eth0mask=255.255.224.0 gateway=10.3.31.254 domain=cs1cloud.internal cidrsize=19 dhcprange=10.3.0.1 eth1ip=0.0.0.0 eth1mask=0.0.0.0 mgmtcidr=10.2.0.0/16 localgw=10.2.254.254 sshonguest=true type=dhcpsrvr disable_rp_filter=true extra_pubnics=2 dns1=8.8.8.8

Though VMware does not support security groups, but in a basic zone with VMware and no isolation VMs should be able to deploy.

Fixes: #3031

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?

  1. Created a VMware basic zone
  2. Created a VM and Virtual router
  3. VR is in running state with private IP assigned to it
  4. VM deployment is also successful

@harikrishna-patnala harikrishna-patnala force-pushed the VMwareBasicZoneFailsTogetPrivateIPonVR branch from 8450757 to 9b22133 Compare March 18, 2020 12:54
@harikrishna-patnala
Copy link
Copy Markdown
Member Author

@blueorangutan package

@blueorangutan
Copy link
Copy Markdown

@harikrishna-patnala 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-1073

@harikrishna-patnala harikrishna-patnala changed the title With basic zone and VMware hypervisor, VR fails to start With basic zone and VMware hypervisor, VR fails to start. closes #3031 Mar 25, 2020
@harikrishna-patnala harikrishna-patnala changed the title With basic zone and VMware hypervisor, VR fails to start. closes #3031 With basic zone and VMware hypervisor, VR fails to start Mar 25, 2020
@harikrishna-patnala harikrishna-patnala force-pushed the VMwareBasicZoneFailsTogetPrivateIPonVR branch 3 times, most recently from 95a33c5 to 9811af1 Compare March 26, 2020 08:59
@harikrishna-patnala harikrishna-patnala marked this pull request as ready for review March 26, 2020 09:02
@DaanHoogland
Copy link
Copy Markdown
Contributor

@blueorangutan package

@DaanHoogland DaanHoogland added this to the 4.13.1.0 milestone Mar 26, 2020
@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: ✖centos7 ✖debian. JID-1091

…s getting empty instead of a private IP.

Though VMware does not support security groups, but in a basic zone with VMware and no isolation VMs should be able to deploy.

Root cause:
In case of VMware and basic zone control nic is set to 0.0.0.0 assuming control network will be shared with guest network.
But to have access to VMware instances management/private needs to be assigned to it.

Solution:
Assing a private ip even in case of basic zone VMware.
@harikrishna-patnala harikrishna-patnala force-pushed the VMwareBasicZoneFailsTogetPrivateIPonVR branch from 9811af1 to 22e8fd3 Compare March 26, 2020 09:45
@harikrishna-patnala
Copy link
Copy Markdown
Member Author

@blueorangutan package

@blueorangutan
Copy link
Copy Markdown

@harikrishna-patnala 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: ✔centos7 ✔debian. JID-1093

Copy link
Copy Markdown
Contributor

@DaanHoogland DaanHoogland left a comment

Choose a reason for hiding this comment

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

code lgtm, i'll setup up a vmware basic zone to verify

@DaanHoogland
Copy link
Copy Markdown
Contributor

@blueorangutan test matrix

@blueorangutan
Copy link
Copy Markdown

@DaanHoogland a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware67, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

@blueorangutan
Copy link
Copy Markdown

Trillian test result (tid-1318)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 29456 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3977-t1318-xenserver-71.zip
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
Smoke tests completed. 75 look OK, 2 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_02_vpc_privategw_static_routes Failure 255.15 test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanup Failure 251.82 test_privategw_acl.py
test_04_rvpc_privategw_static_routes Failure 324.78 test_privategw_acl.py
test_01_scale_vm Failure 20.48 test_scale_vm.py

@blueorangutan
Copy link
Copy Markdown

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

Test Result Time (s) Test File
test_02_vpc_privategw_static_routes Failure 250.54 test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanup Failure 277.40 test_privategw_acl.py
test_04_rvpc_privategw_static_routes Failure 381.53 test_privategw_acl.py

@borisstoyanov
Copy link
Copy Markdown
Contributor

@blueorangutan test

@blueorangutan
Copy link
Copy Markdown

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

@DaanHoogland
Copy link
Copy Markdown
Contributor

I started a vmware basic zone with this and started an instance 👍

@andrijapanicsb
Copy link
Copy Markdown
Contributor

Once tests are fine, we are OK to merge.

@blueorangutan
Copy link
Copy Markdown

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

Test Result Time (s) Test File
test_02_vpc_privategw_static_routes Failure 183.92 test_privategw_acl.py
test_03_vpc_privategw_restart_vpc_cleanup Failure 170.73 test_privategw_acl.py
test_04_rvpc_privategw_static_routes Failure 257.16 test_privategw_acl.py

@apache apache deleted a comment from blueorangutan Mar 27, 2020
@apache apache deleted a comment from blueorangutan Mar 27, 2020
@apache apache deleted a comment from DaanHoogland Mar 27, 2020
@andrijapanicsb andrijapanicsb self-requested a review March 27, 2020 18:44
Copy link
Copy Markdown
Contributor

@andrijapanicsb andrijapanicsb left a comment

Choose a reason for hiding this comment

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

LGTM

@andrijapanicsb
Copy link
Copy Markdown
Contributor

2 x approvals, manual testing done OK, automated/regression tests all passing (failed ones are known test issues).

Merging based on ^^^

@andrijapanicsb andrijapanicsb merged commit 78fda2d into apache:4.13 Mar 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants