Re: [ovs-dev] [ovs-dev, v1] datapath-windows: Do not send out nbls when cloned nbls are being accessed

2019-04-11 Thread Aaron Conole
Stephen Finucane  writes:

> On Wed, 2019-04-10 at 13:53 -0400, Aaron Conole wrote:
>> Aaron Conole  writes:
>> 
>> > 0-day Robot  writes:
>> > 
>> > > Bleep bloop.  Greetings Anand Kumar via dev, I am a robot and I have 
>> > > tried out your patch.
>> > > Thanks for your contribution.
>> > > 
>> > > I encountered some error that I wasn't expecting.  See the details below.
>> > > 
>> > > 
>> > > checkpatch:
>> > > ERROR: Author should not be mailing list.
>> > > Lines checked: 31, Warnings: 0, Errors: 1
>> > 
>> > I looked into this - it seems that your patch doesn't have a Reply-To
>> > field.
>> 
>> Clarification, when fetched from Patchwork it doesn't seem that the
>> Reply-To field is included.  Perhaps this is something useful to address
>> in Patchwork as well?
>
> This was resolved in Patchwork 2.1 [1]. Unfortunately the ozlabs.org
> instance hasn't been updated yet and I've no influence over that.
> Hopefully they'll bump to 2.2 when that's released in the coming
> months.
>
> Stephen
>
> [1] 
> https://github.com/getpatchwork/patchwork/commit/01b9cbb9ce9f44387be22b5d036dde0fff4bb14b

Thanks for the information, Stephen!

>> > I plan to catch cases like this and duplicate the 'Signed-off-by:' as a
>> > 'From:'.
>> > 
>> > Sorry for the noise, again.
>> > > Please check this out.  If you feel there has been an error,
>> > > please email acon...@bytheb.org
>> > > 
>> > > Thanks,
>> > > 0-day Robot
>> > > ___
>> > > dev mailing list
>> > > d...@openvswitch.org
>> > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>> > ___
>> > dev mailing list
>> > d...@openvswitch.org
>> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [ovs-dev, v1] datapath-windows: Do not send out nbls when cloned nbls are being accessed

2019-04-11 Thread Stephen Finucane
On Wed, 2019-04-10 at 13:53 -0400, Aaron Conole wrote:
> Aaron Conole  writes:
> 
> > 0-day Robot  writes:
> > 
> > > Bleep bloop.  Greetings Anand Kumar via dev, I am a robot and I have 
> > > tried out your patch.
> > > Thanks for your contribution.
> > > 
> > > I encountered some error that I wasn't expecting.  See the details below.
> > > 
> > > 
> > > checkpatch:
> > > ERROR: Author should not be mailing list.
> > > Lines checked: 31, Warnings: 0, Errors: 1
> > 
> > I looked into this - it seems that your patch doesn't have a Reply-To
> > field.
> 
> Clarification, when fetched from Patchwork it doesn't seem that the
> Reply-To field is included.  Perhaps this is something useful to address
> in Patchwork as well?

This was resolved in Patchwork 2.1 [1]. Unfortunately the ozlabs.org
instance hasn't been updated yet and I've no influence over that.
Hopefully they'll bump to 2.2 when that's released in the coming
months.

Stephen

[1] 
https://github.com/getpatchwork/patchwork/commit/01b9cbb9ce9f44387be22b5d036dde0fff4bb14b

> > I plan to catch cases like this and duplicate the 'Signed-off-by:' as a
> > 'From:'.
> > 
> > Sorry for the noise, again.
> > > Please check this out.  If you feel there has been an error, please email 
> > > acon...@bytheb.org
> > > 
> > > Thanks,
> > > 0-day Robot
> > > ___
> > > dev mailing list
> > > d...@openvswitch.org
> > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> > ___
> > dev mailing list
> > d...@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev


___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [ovs-dev, v1] datapath-windows: Do not send out nbls when cloned nbls are being accessed

2019-04-10 Thread Aaron Conole
Aaron Conole  writes:

> 0-day Robot  writes:
>
>> Bleep bloop.  Greetings Anand Kumar via dev, I am a robot and I have tried 
>> out your patch.
>> Thanks for your contribution.
>>
>> I encountered some error that I wasn't expecting.  See the details below.
>>
>>
>> checkpatch:
>> ERROR: Author should not be mailing list.
>> Lines checked: 31, Warnings: 0, Errors: 1
>
> I looked into this - it seems that your patch doesn't have a Reply-To
> field.

Clarification, when fetched from Patchwork it doesn't seem that the
Reply-To field is included.  Perhaps this is something useful to address
in Patchwork as well?

> I plan to catch cases like this and duplicate the 'Signed-off-by:' as a
> 'From:'.
>
> Sorry for the noise, again.

>>
>> Please check this out.  If you feel there has been an error, please email 
>> acon...@bytheb.org
>>
>> Thanks,
>> 0-day Robot
>> ___
>> dev mailing list
>> d...@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> ___
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [ovs-dev, v1] datapath-windows: Do not send out nbls when cloned nbls are being accessed

2019-04-10 Thread Aaron Conole
0-day Robot  writes:

> Bleep bloop.  Greetings Anand Kumar via dev, I am a robot and I have tried 
> out your patch.
> Thanks for your contribution.
>
> I encountered some error that I wasn't expecting.  See the details below.
>
>
> checkpatch:
> ERROR: Author should not be mailing list.
> Lines checked: 31, Warnings: 0, Errors: 1

I looked into this - it seems that your patch doesn't have a Reply-To
field.

I plan to catch cases like this and duplicate the 'Signed-off-by:' as a
'From:'.

Sorry for the noise, again.

>
> Please check this out.  If you feel there has been an error, please email 
> acon...@bytheb.org
>
> Thanks,
> 0-day Robot
> ___
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [ovs-dev, v1] datapath-windows: Do not send out nbls when cloned nbls are being accessed

2019-04-09 Thread 0-day Robot
Bleep bloop.  Greetings Anand Kumar via dev, I am a robot and I have tried out 
your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
ERROR: Author should not be mailing list.
Lines checked: 31, Warnings: 0, Errors: 1


Please check this out.  If you feel there has been an error, please email 
acon...@bytheb.org

Thanks,
0-day Robot
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev