Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VXLAN isolation method does not pass the correct broadcast domain type when creating the vlan public IP range #9920

Open
Pearl1594 opened this issue Nov 12, 2024 · 4 comments

Comments

@Pearl1594
Copy link
Contributor

ISSUE TYPE
  • Bug Report
COMPONENT NAME
Zone Creation, Physical network
CLOUDSTACK VERSION
Probably all versions - observed on 4.19.1.2 and 4.20
CONFIGURATION
OS / ENVIRONMENT
SUMMARY

During zone creation, if the isolation method for public traffic is selected as VXLAN, when adding the public IP range and VNI, the broadcast domain type gets passed as VLAN and not VXLAN. Same holds true when creating vlan public IP range (after zone creation) unless 'vxlan://' is explicitly passed in the vni field.

STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

@weizhouapache
Copy link
Member

as far as I know, vlan still works in vxlan zone, right ?
if so, I think we can keep current behavior (if vlan/vxlan is not passed then use vlan by default).
otherwise, we need to change all possible form which requires vlan id/vni (create shared network, isolated/l2 with vlanid, private gateway with vlanid) on UI.

@Pearl1594
Copy link
Contributor Author

Pearl1594 commented Nov 12, 2024

Ok.. then we need to atleast add a doc PR to explain that one needs to explicitly pass vxlan:// before the VNI if they want to use VLANs .
Can we not add a dropdown menu to the form and then before sending the request to the backend create the vlan field to have <isolation_method>:// ?
image

@weizhouapache
Copy link
Member

Ok.. then we need to atleast add a doc PR to explain that one needs to explicitly pass vxlan:// before the VNI if they want to use VLANs . Can we not add a dropdown menu to the form and then before sending the request to the backend create the vlan field to have <isolation_method>:// ?

good idea !
@Pearl1594

@rajujith
Copy link
Collaborator

rajujith commented Nov 13, 2024

Ok.. then we need to atleast add a doc PR to explain that one needs to explicitly pass vxlan:// before the VNI if they want to use VLANs . Can we not add a dropdown menu to the form and then before sending the request to the backend create the vlan field to have <isolation_method>:// ?

That's a good idea @Pearl1594 . We could have the isolation method selected in the physical network by default, allowing the user to change it from the drop-down list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants