feat: add printer supporting tree, table and yaml format

[ycyaoxdu]

Signed-off-by: ycyaoxdu [email protected]

[ycyaoxdu]

#133
some command may do not need yaml output? such as hub-info, klusterlet-info? so i just added -o flag for which may need yaml output.
i think i need more suggestions on this...

[ycyaoxdu]

/hold

[itdove]

@ycyaoxdu Can't you use something like this:

clusteradm/pkg/cmd/get/cluster/exec.go

Line 54 in cae3fa5

return o.printer.PrintObj(table, o.Streams.Out)

which gives you all the -o options instead only the YAML.

[ycyaoxdu]

@itdove i've added a field to genericclioptions called Printer c90782a
and then the usage is like f50b50f
, how about this?

---

return o.printer.PrintObj(table, o.Streams.Out)

and also used this resourcePrinter to do this.

[itdove]

I don't think a genericoption is the good choice as I think -o doesn't apply to all commands but only on get commands...
as you can see here https://github.com/stolostron/cm-cli/blob/5bbe69bf8312ef615d31d5b5037ae0c0174cb0c7/pkg/cmd/get/clusterclaim/cmd.go#L52-L54 I added the -o to each command I think they can use it. I didn't add it to all gets because some of the get cmd are not supposed to have a -o like cm get config cluster.

[ycyaoxdu]

I don't think a genericoption is the good choice as I think -o doesn't apply to all commands but only on get commands... as you can see here https://github.com/stolostron/cm-cli/blob/5bbe69bf8312ef615d31d5b5037ae0c0174cb0c7/pkg/cmd/get/clusterclaim/cmd.go#L52-L54 I added the -o to each command I think they can use it. I didn't add it to all gets because some of the get cmd are not supposed to have a -o like cm get config cluster.

hi @itdove I've noticed that we need to add apis to generalize the --output ? could you please help to contribute this feature? or give me some advice on this? thanks a lot.

[ycyaoxdu]

@qiujian16 PTAL
add a printer supporting tree, table, yaml format

➜  clusteradm git:(feat/general-printers) clusteradm get clusters
<ManagedCluster> 
└── <cluster1> 
│   ├── <KubernetesVersion> v1.24.0
│   ├── <Capacity> 
│   │   ├── <Cpu> 16
│   │   ├── <Memory> 32763764Ki
│   ├── <Accepted> true
│   ├── <Available> True
│   ├── <ClusterSet> default
└── <cluster2> 
    └── <Capacity> 
    │   ├── <Cpu> 16
    │   ├── <Memory> 32763764Ki
    └── <Accepted> true
    └── <Available> True
    └── <ClusterSet> default
    └── <KubernetesVersion> v1.24.0
➜  clusteradm git:(feat/general-printers) clusteradm get clustersets
<ManagedClusterSet> 
└── <default> 
│   ├── <Status> 2 ManagedClusters selected
│   ├── <BoundNamespace> default
└── <global> 
    └── <BoundNamespace> 
    └── <Status> 2 ManagedClusters selected
➜  clusteradm git:(feat/general-printers) clusteradm get works --cluster cluster1
<ManifestWork> 
└── <my-first-work> 
│   ├── <Cluster> cluster1
│   ├── <Number of Manifests> 2
│   ├── <Applied> False
│   ├── <Available> True
└── <example-manifestwork> 
    └── <Cluster> cluster1
    └── <Number of Manifests> 2
    └── <Applied> True
    └── <Available> True

default format has set to tree.

[ycyaoxdu]

/assign @qiujian16

[qiujian16]

/approve
/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: qiujian16, ycyaoxdu

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [qiujian16]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[ycyaoxdu]

/unhold