Thomas,

Thank you for the review and merging. Next I will work on Vagrantfile for
Ubuntu.

Sorry for the inconvenience caused by unintended newlines. I will be more
careful next time.

Regards,

---
Motonori Shindo


2015-01-09 20:11 GMT+09:00 Thomas Graf <tg...@noironetworks.com>:

> On 01/09/15 at 06:31pm, Motonori Shindo wrote:
> > The order of execution in Vagrantfile is "outside-in" meaning that
> > all commands in outer scope are executed first and then the commands
> > in inner scope. Because of this ordering, "configure_ovs" is executed
> > before "bootstrap_fedora", resulting in "configure_ovs" to fail as
> > it is not bootsrapped yet. This change fixes this problem by removing
> > the nested scope in Vagrant.configure block. Multi-machine is not used
> > so we don't need config.vm.define block.
> >
> > Signed-off-by: Motonori Shindo <moton...@shin.do>
>
> Thanks for fixing this, applied to master.
>
> Please note that your mailer has corrupted the patch by adding new
> lines. It was trivial to fix in this case but you might want to check
> before your next submission
>
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to