Re: [ovs-dev] [PATCH ovn v2] vagrant: Use centos-8 box from official location
Bleep bloop. Greetings Flavio Fernandes, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Author Flavio Fernandes needs to sign off. WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers: Flavio Fernandes Lines checked: 61, Warnings: 1, Errors: 1 Please check this out. If you feel there has been an error, please email acon...@redhat.com Thanks, 0-day Robot ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
[ovs-dev] [PATCH ovn v2] vagrant: Use centos-8 box from official location
From: Flavio Fernandes Prefer the usage of "vagrant box update" instead of "dnf update" to save provisioning time. Also, as part of provisioning centos-8, use pip to install packages twisted, zope-interface and six. Signed-off-by: Flavio Fernandes --- v2: Nit: Make dnf update line consistent with other images. --- Vagrantfile | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/Vagrantfile b/Vagrantfile index 88c981c71..f0526ec48 100644 --- a/Vagrantfile +++ b/Vagrantfile @@ -37,7 +37,7 @@ apt-get -y install build-essential fakeroot graphviz autoconf automake bzip2 \ SCRIPT $bootstrap_ovs_centos7 =