Make listing servers default command

[ip1981]

[root@adm ~]# /bin/iml server 
+----+------------+--------------+------------------+
| Id | FQDN       | State        | Nids             |
+====+============+==============+==================+
| 1  | mds1.local | monitored    | 10.73.20.11@tcp0 |
+----+------------+--------------+------------------+
| 2  | mds2.local | monitored    | 10.73.20.12@tcp0 |
+----+------------+--------------+------------------+
| 3  | oss1.local | monitored    | 10.73.20.21@tcp0 |
+----+------------+--------------+------------------+
| 4  | oss2.local | monitored    | 10.73.20.22@tcp0 |
+----+------------+--------------+------------------+
| 5  | c1.local   | managed      | 10.0.2.15@tcp0   |
+----+------------+--------------+------------------+
| 6  | c2.local   | unconfigured |                  |
+----+------------+--------------+------------------+

[root@adm ~]# /bin/iml server list
+----+------------+--------------+------------------+
| Id | FQDN       | State        | Nids             |
+====+============+==============+==================+
| 1  | mds1.local | monitored    | 10.73.20.11@tcp0 |
+----+------------+--------------+------------------+
| 2  | mds2.local | monitored    | 10.73.20.12@tcp0 |
+----+------------+--------------+------------------+
| 3  | oss1.local | monitored    | 10.73.20.21@tcp0 |
+----+------------+--------------+------------------+
| 4  | oss2.local | monitored    | 10.73.20.22@tcp0 |
+----+------------+--------------+------------------+
| 5  | c1.local   | managed      | 10.0.2.15@tcp0   |
+----+------------+--------------+------------------+
| 6  | c2.local   | unconfigured |                  |
+----+------------+--------------+------------------+

Closes #2041.
Signed-off-by: Igor Pashev pashev.igor@gmail.com

---

This change is 

[mkpankov]

Reviewed 2 of 2 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @ip1981)

---

iml-manager-cli/src/server.rs, line 544 at r1 (raw file):

pub async fn server_cli(command: Option<ServerCommand>) -> Result<(), ImlManagerCliError> {
    match command {
        None => list_server(DisplayType::Tabular).await?,

As an alternative: pull ServerCommand out of Option and match on it separately, break early if it's None. Avoids duplicated Some on all branches.

[ip1981]

As an alternative: pull ServerCommand out of Option and match on it separately, break early if it's None. Avoids duplicated Some on all branches.

Updated. Thanks!

[mkpankov]

Reviewed 1 of 1 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @ip1981)

[nlinker]

In principle there are examples of such behaviour in the syntax command noun in e.g. git where git branch shows the current branch, git tag shows the list of tags.
Seems to be logical.