You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Jayapal Reddy <ja...@citrix.com> on 2013/04/19 14:06:23 UTC

Review Request: updated cloud-early-config to wait for the interfaces to IP come up

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10647/
-----------------------------------------------------------

Review request for cloudstack, Anthony Urso, Abhinandan Prateek, and edison su.


Description
-------

In VmWare setup, VR failed to get the route for management CIDR. This is happened because when route add command get executed there no eth1 interface set with ip.

So in cloud-early-config setup_interface added some delay to for interface to setup with ip.
 


This addresses bug CLOUDSTACK-2118.


Diffs
-----

  patches/systemvm/debian/config/etc/init.d/cloud-early-config 3faa1bd 

Diff: https://reviews.apache.org/r/10647/diff/


Testing
-------

1. Tested in vmware and xenserver VR.
2. Route for mgmt cidr on vmware and default routes added properly


Thanks,

Jayapal Reddy


Re: Review Request: updated cloud-early-config to wait for the interfaces to IP come up

Posted by Abhinandan Prateek <ap...@apache.org>.

> On April 20, 2013, 4:33 a.m., Anthony Urso wrote:
> > Ship It!

committed


- Abhinandan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10647/#review19502
-----------------------------------------------------------


On April 19, 2013, 12:06 p.m., Jayapal Reddy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10647/
> -----------------------------------------------------------
> 
> (Updated April 19, 2013, 12:06 p.m.)
> 
> 
> Review request for cloudstack, Anthony Urso, Abhinandan Prateek, and edison su.
> 
> 
> Description
> -------
> 
> In VmWare setup, VR failed to get the route for management CIDR. This is happened because when route add command get executed there no eth1 interface set with ip.
> 
> So in cloud-early-config setup_interface added some delay to for interface to setup with ip.
>  
> 
> 
> This addresses bug CLOUDSTACK-2118.
> 
> 
> Diffs
> -----
> 
>   patches/systemvm/debian/config/etc/init.d/cloud-early-config 3faa1bd 
> 
> Diff: https://reviews.apache.org/r/10647/diff/
> 
> 
> Testing
> -------
> 
> 1. Tested in vmware and xenserver VR.
> 2. Route for mgmt cidr on vmware and default routes added properly
> 
> 
> Thanks,
> 
> Jayapal Reddy
> 
>


Re: Review Request: updated cloud-early-config to wait for the interfaces to IP come up

Posted by Anthony Urso <an...@cs.ucla.edu>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10647/#review19502
-----------------------------------------------------------

Ship it!


Ship It!

- Anthony Urso


On April 19, 2013, 12:06 p.m., Jayapal Reddy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10647/
> -----------------------------------------------------------
> 
> (Updated April 19, 2013, 12:06 p.m.)
> 
> 
> Review request for cloudstack, Anthony Urso, Abhinandan Prateek, and edison su.
> 
> 
> Description
> -------
> 
> In VmWare setup, VR failed to get the route for management CIDR. This is happened because when route add command get executed there no eth1 interface set with ip.
> 
> So in cloud-early-config setup_interface added some delay to for interface to setup with ip.
>  
> 
> 
> This addresses bug CLOUDSTACK-2118.
> 
> 
> Diffs
> -----
> 
>   patches/systemvm/debian/config/etc/init.d/cloud-early-config 3faa1bd 
> 
> Diff: https://reviews.apache.org/r/10647/diff/
> 
> 
> Testing
> -------
> 
> 1. Tested in vmware and xenserver VR.
> 2. Route for mgmt cidr on vmware and default routes added properly
> 
> 
> Thanks,
> 
> Jayapal Reddy
> 
>


Re: Review Request: updated cloud-early-config to wait for the interfaces to IP come up

Posted by Abhinandan Prateek <ap...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10647/#review19451
-----------------------------------------------------------

Ship it!


Ship It!

- Abhinandan Prateek


On April 19, 2013, 12:06 p.m., Jayapal Reddy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10647/
> -----------------------------------------------------------
> 
> (Updated April 19, 2013, 12:06 p.m.)
> 
> 
> Review request for cloudstack, Anthony Urso, Abhinandan Prateek, and edison su.
> 
> 
> Description
> -------
> 
> In VmWare setup, VR failed to get the route for management CIDR. This is happened because when route add command get executed there no eth1 interface set with ip.
> 
> So in cloud-early-config setup_interface added some delay to for interface to setup with ip.
>  
> 
> 
> This addresses bug CLOUDSTACK-2118.
> 
> 
> Diffs
> -----
> 
>   patches/systemvm/debian/config/etc/init.d/cloud-early-config 3faa1bd 
> 
> Diff: https://reviews.apache.org/r/10647/diff/
> 
> 
> Testing
> -------
> 
> 1. Tested in vmware and xenserver VR.
> 2. Route for mgmt cidr on vmware and default routes added properly
> 
> 
> Thanks,
> 
> Jayapal Reddy
> 
>