Skip to content

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

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

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

Conversation

@harikrishna-patnala

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

Copy link
Copy Markdown
Member

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

@DaanHoogland DaanHoogland 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, 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

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

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