Skip to content

Random MAC address button in the clone view#474

Merged
retspen merged 5 commits intoretspen:masterfrom
daniviga:random-mac
Nov 18, 2014
Merged

Random MAC address button in the clone view#474
retspen merged 5 commits intoretspen:masterfrom
daniviga:random-mac

Conversation

@daniviga
Copy link
Copy Markdown
Contributor

This PR adds a button (for each interface) to generate a new, random MAC address in the clone view.

screenshot from 2014-11-14 16 05 49

@daniviga daniviga mentioned this pull request Nov 14, 2014
@aafanasyev
Copy link
Copy Markdown

Yes it looks nice! Can I complain a bit more about generation order? :)

For management propose it would be nice if Mac addresses could be generated in some kind of numeric order. E.q.

00:00:00:00:00:00
00:00:00:00:00:01

etc.

However it should be done only by a user request. Just with a check mark. Otherwise it can be fully random. IMHO. But some times I miss such future in VBox when I am checking my DHCP server.

@daniviga
Copy link
Copy Markdown
Contributor Author

Yes :) what do you mean with "generation order"?

@aafanasyev
Copy link
Copy Markdown

Oh sorry I just edited my previous comment here please check it.

@daniviga
Copy link
Copy Markdown
Contributor Author

I replicated the same behavior virt-manager has: the first 3 octets are 52:54:00 which is the OUI assigned to KVM, the last three are purely random.

The numeric order you want should be relative only to that VM interfaces I guess, not sequential with all the other VMs, right?

@aafanasyev
Copy link
Copy Markdown

Yes, but it is can u give me a couple of hours I will draw the interface for you. So I will be much more clear.

@aafanasyev
Copy link
Copy Markdown

Looks like I am really lazy :D However it looks useful(IMHO):

ordered mac

@retspen
Copy link
Copy Markdown
Owner

retspen commented Nov 18, 2014

@daniviga greate job, thanks!

retspen added a commit that referenced this pull request Nov 18, 2014
Random MAC address button in the clone view
@retspen retspen merged commit c87e3d7 into retspen:master Nov 18, 2014
@daniviga daniviga deleted the random-mac branch November 27, 2014 13:30
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.

3 participants