Russell Bryant <[email protected]> writes: > On Thu, Jul 20, 2017 at 3:41 AM, Numan Siddique <[email protected]> wrote: >> On Wed, Jul 19, 2017 at 7:00 PM, Aaron Conole <[email protected]> wrote: >> >>> Timothy Redaelli <[email protected]> writes: >>> >>> > The policy is to use %files to track installed files. >>> > >>> > If %files is not used the resulting file is not owned by any package. >>> > >>> > Before this commit: >>> > # rpm -qf /usr/lib/ocf/resource.d/ovn/ovndb-servers >>> > file /usr/lib/ocf/resource.d/ovn/ovndb-servers is not owned by any >>> package >>> > >>> > After this commit: >>> > # rpm -qf /usr/lib/ocf/resource.d/ovn/ovndb-servers >>> > openvswitch-ovn-common-2.7.90-1.fc26.x86_64 >>> > >>> > Fixes: a4245b7869c8 ("ovn: Add ovn db servers ocf script in fedora >>> packager") >>> > >>> > Signed-off-by: Timothy Redaelli <[email protected]> >>> > --- >>> >>> LGTM >>> >> >> LGTM. >> Acked-by: Numan Siddique <[email protected]> >> >> >> >>> >>> Reviewed-by: Aaron Conole <[email protected]> > > Thanks! I applied this to master and branch-2.7. > > Aaron, I changed this to Acked-by to match usual OVS convention.
Wow, I didn't notice before that OvS doesn't use a Reviewed-by tag, at least according to the submitting-patches doc. Thanks Russell! > Russell Bryant _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
