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.


Russell Bryant
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to