[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] virsh: add command net-info

On 11/08/2010 07:40 PM, Osier Yang wrote:
> +    {"desc", "Returns basic information about the network"},

Probably better to make it clear that it's about a "virtual
network", rather than any other kind of network.

Maybe "Returns basic information about a virtual network"?
(or something similar)

> @@ -9924,6 +9982,7 @@ static const vshCmdDef commands[] = {
>      {"net-start", cmdNetworkStart, opts_network_start, info_network_start},
>      {"net-undefine", cmdNetworkUndefine, opts_network_undefine, info_network_undefine},
>      {"net-uuid", cmdNetworkUuid, opts_network_uuid, info_network_uuid},
> +    {"net-info", cmdNetworkInfo, opts_network_info, info_network_info},

Instead of adding net-info to the end of the net-* commands, it's
probably better to have it positioned alphabetically like the others.
(between net-edit and net-list seems like the right place)

With those two nits fixed, ACK. :)

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]