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?

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?

[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

Reply via email to