Re: [PATCH v2] iproute2: build nsid-name cache only for commands that need it

On Fri, Sep 16, 2016 at 2:21 PM, Anton Aksola [off-list ref] wrote:

Ok, I will post a new patch version. Should tests be posted in a separate
patch?

2016-09-16 12:44 GMT+03:00 Nicolas Dichtel [off-list ref]:

quoted

Le 16/09/2016 à 11:23, Vadim Kochan a écrit :
[snip]

quoted

Would it be useful to add test for this case into testsuite/ ?

Yes, it's a good idea.

Regards,
Nicolas

Anton, I just looked into tests after when I did post here. I am not
sure it will be trivial,
currently tests are running within separated network namespace by
default (which I did) via
'unshare' tool, and now I see that it is better to call it explicitly
from the each test case. So
I am not sure netns related  tests might be valid if they will be ran
after 'unshare -n', if yes - then there is no problem, otherwise
it needs to be fixed somehow - I will try to do this.

Regards,
Vadim Kochan