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

vm wizard: add MAC Address validator - #657

Merged
yadvr merged 5 commits into
apache:masterfrom
utchoang:feature/add-mac-address-validator
Sep 9, 2020
Merged

yadvr merged 5 commits into
apache:masterfrom
utchoang:feature/add-mac-address-validator

Conversation

@utchoang

@utchoang utchoang commented Sep 1, 2020

Copy link
Copy Markdown

Fixes #648
@rhtyd cc @svenvogel

  • Add MAC Address Validator:
    Screenshot_1
    Screenshot_2

@utchoang

utchoang commented Sep 1, 2020

Copy link
Copy Markdown
Author

@rhtyd Also on the issue, @andrijapanicsb wants to see the IP range of the network, to be able to KNOW from which range to use an IP when specifying it explicitly down for the NIC attached to that network. Do you have any ideas?

@yadvr yadvr added this to the 1.0-GA milestone Sep 2, 2020
@yadvr

yadvr commented Sep 2, 2020

Copy link
Copy Markdown
Member

@utchoang in the network record if that's an isolated/shared network (or != L2 network) then you can check the record.cidr and you can check if the input IP is within the cidr [1][2]

[1] https://tech.mybuilder.com/determining-if-an-ipv4-address-is-within-a-cidr-range-in-javascript/
[2] https://stackoverflow.com/questions/46200086/how-to-check-if-a-given-ip-falls-between-a-given-ip-range-using-node-js

@yadvr

yadvr commented Sep 2, 2020

Copy link
Copy Markdown
Member

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

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

@utchoang

utchoang commented Sep 3, 2020

Copy link
Copy Markdown
Author
  • Show list IP range and check valid IP range:
    Screenshot_1
    Screenshot_2
    Screenshot_3

@utchoang

utchoang commented Sep 4, 2020

Copy link
Copy Markdown
Author

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

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

@yadvr

yadvr commented Sep 4, 2020

Copy link
Copy Markdown
Member

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

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

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔️centos ✔️debian ✔️archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/657 (JID-3426)

@yadvr

yadvr commented Sep 7, 2020

Copy link
Copy Markdown
Member

LGTM
@davidjumani can you review and test this?

Comment thread src/views/compute/wizard/NetworkConfiguration.vue
@utchoang

utchoang commented Sep 8, 2020

Copy link
Copy Markdown
Author

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

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

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔️centos ✔️debian ✔️archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/657 (JID-3441)

@yadvr

yadvr commented Sep 8, 2020

Copy link
Copy Markdown
Member

@davidjumani please review this

Comment thread src/views/compute/wizard/NetworkConfiguration.vue Outdated
Comment thread src/views/compute/wizard/NetworkConfiguration.vue
@utchoang

utchoang commented Sep 9, 2020

Copy link
Copy Markdown
Author

@davidjumani I've fixed it. Please review it again. Thanks.
Screenshot_1

@davidjumani

Copy link
Copy Markdown
Contributor

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

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

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔️centos ✔️debian ✔️archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/657 (JID-3447)

@davidjumani davidjumani 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! Thanks @utchoang
@rhtyd Good to go!

@yadvr
yadvr merged commit d03e559 into apache:master Sep 9, 2020
@utchoang
utchoang deleted the feature/add-mac-address-validator branch September 10, 2020 07:39
weizhouapache pushed a commit that referenced this pull request Jan 19, 2021
* vm wizard: add MAC Address validator

* show IP range and check valid IP range

* add CIDR into expanding of network selection & configuration

* hide element when network type !== L2
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] List network's range with other details during VM deployment & MAC format explanation

4 participants