Addition of PORT(s) in the output for application list command | Feature Request

[shubhamwaghe]

On running the following command for listing appinstances, various ports (especially HTTP/DEBUG/OTHER) also would be a good addition to the output

drove -c stage appinstances list my-application-01

Example Output (PORT(s) column requirement) -

Instance ID    Executor Host            State       Error Message  Created                Last Updated
------------  ------------------------  ----------  -------------  ---------------------  --------------------
AI-INSTANCE   stg-instance.website.com  HEALTHY                    18/09/2024, 11:00:00   18/09/2024, 18:00:00

Proposal:

1. Addition of Ports column
2. Addition of comma separated name, port as values Eg =
   admin: 23959,http: 27271

Implementation:

In list_instances -

Addition of "Ports" in headers list & following snippet -

try:
    instance_row.append(", ".join("{}: {}".format(key, value['hostPort']) for key, value in instance["localInfo"]["ports"].items()))
except KeyError:
    instance_row.append("")

Sample ports raw output -

"ports": {
    "admin": {
        "containerPort": 10001,
        "hostPort": 23959,
        "portType": "HTTP"
    },
    "http": {
        "containerPort": 10000,
        "hostPort": 27271,
        "portType": "HTTP"
    }
}

[santanusinha]

Sorry missed this. Can be done, need to figure out a way to show this in a consistent manner. It would be an issue since the ports list can change in length. Will address in next release.