[Bug 1691022] Review Request: ovn - Virtual networking solution from OpenvSwitch community

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1691022

Numan Siddique <numan.siddique@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |needinfo?(zebob.m@xxxxxxxxx
                   |                            |)



--- Comment #15 from Numan Siddique <numan.siddique@xxxxxxxxx> ---
(In reply to Numan Siddique from comment #14)
> (In reply to Robert-André Mauchin from comment #12)
> >  - Add BR for gcc
> > 
> >  - Update License field
> > 
> > 
> > BSD 3-clause "New" or "Revised" License
> > ---------------------------------------
> > openvswitch-2.11.0/include/sparse/rte_byteorder.h
> > openvswitch-2.11.0/include/sparse/rte_esp.h
> > openvswitch-2.11.0/include/sparse/rte_flow.h
> > openvswitch-2.11.0/include/windows/netinet/icmp6.h
> > openvswitch-2.11.0/include/windows/netinet/ip6.h
> > openvswitch-2.11.0/lib/strsep.c
> > openvswitch-2.11.0/tests/dpdk/ring_client.c
> > 
> > BSD 4-clause "Original" or "Old" License
> > ----------------------------------------
> > openvswitch-2.11.0/include/sparse/rte_icmp.h
> > openvswitch-2.11.0/include/sparse/rte_ip.h
> > openvswitch-2.11.0/include/sparse/rte_sctp.h
> > openvswitch-2.11.0/include/sparse/rte_tcp.h
> > openvswitch-2.11.0/include/sparse/rte_udp.h
> > 
> > Expat License
> > -------------
> > openvswitch-2.11.0/build-aux/install-sh
> > openvswitch-2.11.0/include/openflow/openflow-1.3.h
> > openvswitch-2.11.0/include/openflow/openflow-1.4.h
> > openvswitch-2.11.0/include/openflow/openflow-1.5.h
> > 
> > GPL (v2)
> > --------
> > openvswitch-2.11.0/datapath/actions.c
> > openvswitch-2.11.0/datapath/compat.h
> > openvswitch-2.11.0/datapath/conntrack.c
> > openvswitch-2.11.0/datapath/conntrack.h
> > openvswitch-2.11.0/datapath/datapath.c
> > openvswitch-2.11.0/datapath/datapath.h
> > openvswitch-2.11.0/datapath/dp_notify.c
> > openvswitch-2.11.0/datapath/flow.c
> > openvswitch-2.11.0/datapath/flow.h
> > openvswitch-2.11.0/datapath/flow_netlink.c
> > openvswitch-2.11.0/datapath/flow_netlink.h
> > openvswitch-2.11.0/datapath/flow_table.c
> > openvswitch-2.11.0/datapath/flow_table.h
> > openvswitch-2.11.0/datapath/linux/compat/geneve.c
> > openvswitch-2.11.0/datapath/linux/compat/gre.c
> > openvswitch-2.11.0/datapath/linux/compat/gso.c
> > openvswitch-2.11.0/datapath/linux/compat/include/net/mpls.h
> > openvswitch-2.11.0/datapath/linux/compat/ip_tunnel.c
> > openvswitch-2.11.0/datapath/linux/compat/ip_tunnels_core.c
> > openvswitch-2.11.0/datapath/linux/compat/lisp.c
> > openvswitch-2.11.0/datapath/linux/compat/vxlan.c
> > openvswitch-2.11.0/datapath/meter.c
> > openvswitch-2.11.0/datapath/meter.h
> > openvswitch-2.11.0/datapath/nsh.c
> > openvswitch-2.11.0/datapath/vport-gre.c
> > openvswitch-2.11.0/datapath/vport-internal_dev.c
> > openvswitch-2.11.0/datapath/vport-internal_dev.h
> > openvswitch-2.11.0/datapath/vport-netdev.c
> > openvswitch-2.11.0/datapath/vport-netdev.h
> > openvswitch-2.11.0/datapath/vport-vxlan.c
> > openvswitch-2.11.0/datapath/vport.c
> > openvswitch-2.11.0/datapath/vport.h
> > openvswitch-2.11.0/xenserver/usr_lib_xsconsole_plugins-base_XSFeatureVSwitch.
> > py
> > 
> >  Update license breakdown comment as well.
> > 

OVN is part of OpenvSwitch and the source code is here -
https://github.com/openvswitch/ovs/tree/master/ovn

Compiling OpenvSwitch also compiles OVN.
OVN doesn't make use of the above files listed by you.
You think I still need to update the license ?

Also the openvswitch spec file is here -
https://src.fedoraproject.org/rpms/openvswitch/blob/master/f/openvswitch.spec
and I took that as reference.


> >  - Own these directories:
> > 
> > [!]: Package requires other packages for directories it uses.
> >      Note: No known owner of /usr/share/openvswitch,
> >      /usr/lib/ocf/resource.d/ovn, /usr/share/openvswitch/scripts
> > 
> >  - Issue with Provides/Obsoletes
> > 
> > ovn.x86_64: W: self-obsoletion openvswitch-ovn-common < 2.11.0-3 obsoletes
> > openvswitch-ovn-common = 2.11.0-2.fc31
> > 
> > The version you provide should be > to the version you obsolete, so at least
> > 2.11.0-4
> 
> Ack.
> 
> > 
> >  - Use Fedora crypto default:
> > 
> > ovn.x86_64: W: crypto-policy-non-compliance-openssl /usr/bin/ovn-nbctl
> > SSL_CTX_set_cipher_list
> > ovn.x86_64: W: crypto-policy-non-compliance-openssl /usr/bin/ovn-sbctl
> > SSL_CTX_set_cipher_list
> > ovn.x86_64: W: crypto-policy-non-compliance-openssl /usr/bin/ovn-trace
> > SSL_CTX_set_cipher_list
> > 
> > See https://docs.fedoraproject.org/en-US/packaging-guidelines/CryptoPolicies/
> > 
> > That is: patch lib/stream-ssl.c:1017 to use "PROFILE=SYSTEM"
> 
> This requires changes int the openvswitch code and I will submit a patch to
> the ovs-dev ML
> to use "PROFILE=SYSTEM". I will update if that's acceptable or not.
> 
> 
> > 
> >  - 
> > 
> > # Using {_lib} macro will solve the
> > # rpmlink error, but will install the files in /usr/lib64/.
> > # OVN pacemaker ocf script file is copied in /usr/lib/ocf/resource.d/ovn/
> > # and we are not sure if pacemaker looks into this path to find the
> > # OVN resource agent script.
> > 
> > Can't you check if lib64 actually work or not?
> 
> I actually looked into it. I don't we can address this.
> I checked with the pacemaker folks and they are not sure if the
> ResourceAgent looks into this path for Resource agent scripts.
> It would be too risky to assume that /usr/lib64 will work. So I would prefer
> that the OVN pacemaker script is installed at /usr/lib.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux