Re: Review Request: Added fix for default route issue in system vms

2013-03-25 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10060/#review18348 --- Ship it! applied to master and 4.1 - Chip Childers On March 25,

Re: Review Request: Added fix for default route issue in system vms

2013-03-25 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10060/#review18340 --- Ship it! Ship It! - Marcus Sorensen On March 25, 2013, 10:27 a.m

Re: Review Request: Added fix for default route issue in system vms

2013-03-25 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10060/ --- (Updated March 25, 2013, 10:27 a.m.) Review request for cloudstack, Marcus Sore

Re: Review Request: Added fix for default route issue in system vms

2013-03-22 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10060/ --- (Updated March 22, 2013, 9:04 a.m.) Review request for cloudstack, Marcus Soren

Re: Review Request: Added fix for default route issue in system vms

2013-03-21 Thread Marcus Sorensen
My concern is that it will only fix the problem 90% of the time, because we didn't wait 4 seconds. Then we bump it to 5, or 10, and needlessly delay the boot up. Its far better to take a few lines of code to verify you're ready before continuing than assume that some number of seconds is always eno

Re: Review Request: Added fix for default route issue in system vms

2013-03-21 Thread Anthony Urso
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10060/#review18223 --- Ship it! If this solves the problem, it's better to do it in this s

Re: Review Request: Added fix for default route issue in system vms

2013-03-21 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10060/#review18204 --- patches/systemvm/debian/config/etc/init.d/cloud-early-config

Review Request: Added fix for default route issue in system vms

2013-03-21 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10060/ --- Review request for cloudstack, Anthony Urso, Abhinandan Prateek, edison su, Rohi