Skip to content
This repository has been archived by the owner on Jan 20, 2021. It is now read-only.

Missing of Add IP Range in GuestNetwork, It has in Legacy Interface #909

Open
hstan77 opened this issue Jan 1, 2021 · 6 comments · May be fixed by #910
Open

Missing of Add IP Range in GuestNetwork, It has in Legacy Interface #909

hstan77 opened this issue Jan 1, 2021 · 6 comments · May be fixed by #910
Assignees
Labels
feature New Feature
Milestone

Comments

@hstan77
Copy link

hstan77 commented Jan 1, 2021

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

In Legacy Interface , GuestNetwork Range can be add :
Infrastructure -> zone -> physical network ->Guest Network-> Guest IP range
and there is link of "Add IP Range" in top right to add IP Range

However in Primate , there is no such function, after guest network added, there is no place view the range (especially the start and end ip )

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@hstan77 hstan77 added the feature New Feature label Jan 1, 2021
@rohityadavcloud
Copy link
Member

@hstan77 do you mean public IP range? There's no such thing as guest IP range in legacy UI as well. In new UI, it's under same path but the difference is that the information is laid out in tabs of different traffic types.

@hstan77
Copy link
Author

hstan77 commented Jan 1, 2021

Hi @rhtyd

In Legacy interface, go to:

Infrastructure > ZonesAdvSG > Physical Network > GuestNet > "Click View IP Range"

And can add / IP there.

If go to Navigation > Guest Network , there is no such function of View IP range and add IP, but browse through Infrastructure zone Physical Network, there is the function there.

This function is useful , especially for Advance WIth SG, where Public IP is the GuestNetwork. For example, when define Network, with x.x.x.x/24 and wish to change / add start and end IP. Also it need to add new prefix where all is in the same VLAN.

@davidjumani
Copy link
Contributor

@hstan77 Do you mean this ?

Screenshot from 2021-01-04 09-51-46

@rhtyd I can look into this

@davidjumani davidjumani linked a pull request Jan 4, 2021 that will close this issue
@hstan77
Copy link
Author

hstan77 commented Jan 4, 2021 via email

@rohityadavcloud
Copy link
Member

rohityadavcloud commented Jan 4, 2021

Thanks for explaining @hstan77 this looks like security groups only feature that isn't supported in new UI, you can use legacy UI or API for the use-case until we address this. Since this is infra-related feature accessible only to the root admin it may not be considered a blocker cc @DaanHoogland (you agree?)

@davidjumani since you've already started a PR, let me give you some pointers -- for SG the guest network uses the public IPs, where the IP range is consumed directly by the end VMs than a virtual network; we probably want to add a tab in the guest networks (SG guest networks only) which are treated as shared network with vlan ip/ranges. This tab should list the range with pagination, for ex:

command: listVlanIpRanges
zoneid: f4bca7d3-81ce-4bfe-9c50-56acf057844b
networkid: 8932951f-4ea2-49e2-9c83-65c16ef4fd04
page: 1
pagesize: 20
response: json

On top of the tab we can give a button to open a modal/form that takes input to create new range, for example: (note, the parameter forVirtualNetwork is false)

command: createVlanIpRange
forVirtualNetwork: false
networkid: 8932951f-4ea2-49e2-9c83-65c16ef4fd04
gateway: 192.168.1.1
netmask: 255.255.255.0
startip: 192.168.1.100
endip: 192.168.1.200
response: json

@davidjumani the tab should check if the APIs are available to the logged-in user/account (createVlanIpRange and listVlanIpRanges)

@rohityadavcloud rohityadavcloud added this to the 1.1 milestone Jan 4, 2021
@hstan77
Copy link
Author

hstan77 commented Jan 4, 2021 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature New Feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants