Podman pod stats -- fix GO template output

[baude]

Go templates were not being processed or printed correctly for podman
pod stats. Added the ability to do templates as well as honor the
table identifier.

Fixes #2258

Signed-off-by: baude bbaude@redhat.com

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baude

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 [baude]

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

[rhatdan]

LGTM

[mheon]

I'm a bit concerned that we're adding tabwriter stuff back into the main code path, which could have the same perf implications that podman ps had that we needed a rewrite for.

On the other hand, I doubt we see as many pods as we did containers, so let's cross that bridge when we come to it.

/lgtm

[mheon]

Hey, bot, LGTM label please?

[baude]

I'm a bit concerned that we're adding tabwriter stuff back into the main code path, which could have the same perf implications that podman ps had that we needed a rewrite for.

On the other hand, I doubt we see as many pods as we did containers, so let's cross that bridge when we come to it.

/lgtm

@mheon it wasnt tabwriter that had the perf implications, it was the use of the go templating for regular output and the multiple serializations needed for that.