[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20141213152024.GB1849@nanopsycho.orion>
Date: Sat, 13 Dec 2014 16:20:24 +0100
From: Jiri Pirko <jiri@...nulli.us>
To: vadim4j@...il.com
Cc: netdev@...r.kernel.org
Subject: Re: [PATCH iproute2 v4] ip: Simplify executing ip cmd within network
ns
Sat, Dec 13, 2014 at 02:32:10PM CET, vadim4j@...il.com wrote:
>On Sat, Dec 13, 2014 at 10:58:03AM +0200, vadim4j@...il.com wrote:
>> On Sat, Dec 13, 2014 at 10:42:43AM +0200, vadim4j@...il.com wrote:
>> > On Sat, Dec 13, 2014 at 09:29:36AM +0100, Jiri Pirko wrote:
>> > > Fri, Dec 12, 2014 at 11:15:07PM CET, vadim4j@...il.com wrote:
>> > > >From: Vadim Kochan <vadim4j@...il.com>
>> > > >
>> > > >Added new '-netns' option to simplify executing following cmd:
>> > > >
>> > > > ip netns exec NETNS ip OPTIONS COMMAND OBJECT
>> > > >
>> > > > to
>> > > >
>> > > > ip -n[etns] NETNS OPTIONS COMMAND OBJECT
>> > > >
>> > > >e.g.:
>> > > >
>> > > > ip -net vnet0 link add br0 type bridge
>> > > > ip -n vnet0 link
>> > > >
>> > > >Signed-off-by: Vadim Kochan <vadim4j@...il.com>
>> > >
>> > >
>> > > This looks good. I'm still missing support in tc, bridge, etc. I think
>> > > it would be great to do this in the same patch/patchset.
>> > >
>> > I planned to do this in the future patches after this main
>> > changes will be accepted. Actually adding this option to other
>> > tools is trivial.
>> >
>> > Anyway may be I will re-send v5 with supporting of these tools if I will have time.
>> >
>> > Regards,
>>
>> BTW, some tools already have '-n' option, so I think only '-net' can be
>> used in such cases.
Yep, that is my point. I would like to have the same option for all.
>>
>> Regards,
>
>OK, I am going to split changes into series of patches and bring new
>option to : ip, tc, and bridge tools.
>Regarding other misc tools - will do it later as I am not very familiar with them.
>Are you OK with this Jiri ?
Yep. Thank you!
>
>Regards,
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists