epics-containers / edge-containers-cli

command line shortcuts for epics containers developers
Apache License 2.0
3 stars 1 forks source link

Overhaul of ps command #104

Closed marcelldls closed 6 months ago

marcelldls commented 7 months ago

This PR tries to do the following for ec ps:

I had to change the podman/docker ps -o to be json rather than customised table as docker does not support getting the restart information - which had the benefit of removing the need for a regex for version information and makes it easier to get labels.

In order to get consistency with the status I decided to just have a bool for "running" since this differs between k8s and podman/docker

gilesknap commented 7 months ago

This looks great. Many thanks. Merging.

gilesknap commented 6 months ago

This looks great. Merging (this time I really mean it)