Juniper / vqfx10k-vagrant

Vagrant projects for vQFX10k
Apache License 2.0
136 stars 62 forks source link

Direct COM1 to file to improve boot time. #35

Closed edwinpjacques closed 5 years ago

edwinpjacques commented 5 years ago

Improve Boot Time on VirtualBox 6 (maybe others too?)

Boot time in VirtualBox 6 was taking 30m or so for the routing engine. Found that the problem was related to COM1 not being connected to anything. Connected it to a file, and now the light qfx boots in about a minute on my machine. Made the change in all Vagrantfiles and tested to make sure it comes up.

This change only helps, hopefully a quick one to approve.

edwinpjacques commented 5 years ago

Hi @grigorenkoae was hoping you could take a look at this? Thanks!

edwinpjacques commented 5 years ago

Closes #36

edwinpjacques commented 5 years ago

@trmccart would you mind to approve this? It make Vagrant usable again for many OS X users. Thank you.

edwinpjacques commented 5 years ago

@aburston can you try this on Windows please to be safe? I tested all the Vagrantfiles on OSX.

aburston commented 5 years ago

@aburston can you try this on Windows please to be safe? I tested all the Vagrantfiles on OSX.

Will do, could you please assign this issue to me.

edwinpjacques commented 5 years ago

@aburston I don't have permission to do that. Not sure who maintains this. Activity seems a bit light.

aburston commented 5 years ago

@edwinpjacques looks like @trmccart is the person who last merged code, so we could ask him once I have tested this.

edwinpjacques commented 5 years ago

@aburston hold on, let me try another tweak to avoid having to write a log file at all. The hard part is supporting Windows since I have to set the log file to NUL instead of /dev/null. Should be doable though.

aburston commented 5 years ago

@edwinpjacques Don't try create files for the "National University of Lesotho", tried that doesn't work out well in windows.

edwinpjacques commented 5 years ago

@aburston I replaced the assignment of re_log with a conditional that works on OS X, Linux & should work fine on Windows. And yes I tested all the files (on OS X).