[ovs-dev] [PATCH] rhel: Fix creation of symlink for ocf script

Russell Bryant russell at ovn.org
Fri Jul 21 00:13:18 UTC 2017


On Thu, Jul 20, 2017 at 3:41 AM, Numan Siddique <nusiddiq at redhat.com> wrote:
> On Wed, Jul 19, 2017 at 7:00 PM, Aaron Conole <aconole at redhat.com> wrote:
>
>> Timothy Redaelli <tredaelli at redhat.com> 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 <tredaelli at redhat.com>
>> > ---
>>
>> LGTM
>>
>
> LGTM.
> Acked-by: Numan Siddique <nusiddiq at redhat.com>
>
>
>
>>
>> Reviewed-by: Aaron Conole <aconole at redhat.com>

Thanks!  I applied this to master and branch-2.7.

Aaron, I changed this to Acked-by to match usual OVS convention.


Russell Bryant


More information about the dev mailing list