On Tue, Oct 4, 2022 at 5:41 AM Dumitru Ceara <[email protected]> wrote: > > On 9/21/22 15:36, Vladislav Odintsov wrote: > > Signed-off-by: Vladislav Odintsov <[email protected]> > > --- > > Hi Vladislav, > > > rhel/ovn-fedora.spec.in | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/rhel/ovn-fedora.spec.in b/rhel/ovn-fedora.spec.in > > index 821eb03cc..57dc977c1 100644 > > --- a/rhel/ovn-fedora.spec.in > > +++ b/rhel/ovn-fedora.spec.in > > @@ -65,6 +65,7 @@ BuildRequires: tcpdump > > BuildRequires: unbound unbound-devel > > > > Requires: openssl hostname iproute module-init-tools openvswitch > > +Requires: python3-openvswitch > > > > Requires(post): systemd-units > > Requires(preun): systemd-units > > This change looks good to me, I tested it locally and it works fine: > > Acked-by: Dumitru Ceara <[email protected]> > > Numan, you have more Fedora packaging experience, do you agree?
LGTM Thanks. I applied this patch to the main branch and backported till branch-22.03. > > This is not something this patch should fix but it reminded me of: OVS > builds Debian packages in CI; there's a patch to do the same thing in > OVN [0]; should we do it for Fedora packages too? +1. That would be beneficial IMO. Numan > > [0] > https://patchwork.ozlabs.org/project/ovn/patch/[email protected]/ > > Thanks, > Dumitru > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
