Bedrock CLI `deployment get` interface improvements

[samiyaakhtar]

As a: developer/manager/operator

I want: to run spk deployment get

So that: I can understand the output of the table more easily

Describe the solution you'd like:

• [x] Have a flag to override having no separators in the table view, but show separators by default in spk deployment get command - https://github.com/microsoft/bedrock-cli/pull/38
• [x] Hide the deployment Id column unless in dev mode (dashboard and cli)
• [x] Rename the columns to make them more meaningful - https://github.com/microsoft/bedrock/issues/1404
• [ ] Support better resizing #1200
• [ ] Introduce an output format csv which outputs the current table but in csv format
• [ ] Flag for manual hld edits, disabled by default, so that the default display is just deployments that are bedrock initiated
• [ ] Duration column values should be more succinct. (e.g. remove *.XX for minutes)
• [ ] Explore emoji support and colored text support in this cli table, which can help replace the check marks under "result" columns, and make the table more user friendly

A few other minor improvements:

• [x] Remove deployment Id column https://github.com/microsoft/bedrock-cli/pull/11
• [x] Rename Env to ring column https://github.com/microsoft/bedrock-cli/pull/11
• [x] Introduce state column to normal mode but remove start time column, and also add duration column - https://github.com/microsoft/bedrock-cli/pull/38
• [x] Consider truncating some fields on normal mode to adjust the addition of new columns, but there should be no truncation on wide mode - https://github.com/microsoft/bedrock-cli/pull/38
• [x] There should be some wording on deployments awaiting on PR such as "Awaiting Approval" under status to make it clear that the deployment is waiting on human intervention and not stuck or failed - https://github.com/microsoft/bedrock-cli/pull/38
• [x] Introduce author to be shown in wide output - https://github.com/microsoft/bedrock-cli/pull/38

Acceptance Criteria: The table is more readable

Additional context:

Does this require updates to documentation?: No

[timfpark]

Saving this from our slack discussion before Slack erases it. One embodiment of this for the CLI might look like this:

                       |                | Source Code to Container | Container to HLD  | HLD to Manifest  |
Start Time             | Service        | Commit   Build  Tag   OK | Release Env    OK | Commit  Build OK |
-----------------------|----------------|--------------------------|-------------------|------------------|
10/30/2019, 4:24:24 PM | userService    | 87e0738  3175   3175  ✓  | 324     STABLE ✓  | cf267d1 3193  ✓  |
10/30/2019, 5:05:47 PM | mapService     | bc3ef55  3184   3184  ✓  | 326     STABLE ✓  | 43c6921 3194  ✓  |
10/30/2019, 4:27:50 PM | featureService | 06dcd6d  3178   3178  ✓  | 325     STABLE ✓  | be5c224 3192  ✓  |
10/30/2019, 5:45:37 PM | storageService | 7031973  3186   3186  ✓  | 327     STABLE ✓  | b131d7a 3191  ✓  |
10/30/2019, 5:45:37 PM | storageService | 7031973  3186   3186  ✓  | 327     CANARY ✓  | 701ced1 3189  ✓  |
10/30/2019, 5:40:37 PM |  (HLD change)  |        -- n/a --         |     -- n/a --     | 0f768fc 3188  ✓  |
10/30/2019, 5:05:47 PM | mapService     | bc3ef55  3184   3184  ✓  | 326     CANARY ✓  | b7415b3 3187  ✓  |
10/30/2019, 4:27:50 PM | featureService | 06dcd6d  3178   3178  ✓  | 325     CANARY ✓  | 0a688c9 3183  ✓  |
10/30/2019, 4:24:24 PM | userService    | 87e0738  3175   3175  ✓  | 324     CANARY ✓  | 3816f2c 3180  ✓  |

[andrebriggs]

@samiyaakhtar This seems to have dependencies on talking to a designer. Sounds like feedback is need before we mark this as a small task.

[andrebriggs]

@samiyaakhtar let's set up time to get feedback with the designer and re-visit.

[andrebriggs]

@samiyaakhtar any update on the items in the description task list?

cc @gemorris

[samiyaakhtar]

@andrebriggs Not yet, let's prioritize this for this sprint or next. @gemorris do you have feedback around this task?

[samiyaakhtar]

Adding the following improvements based on our discussion today:

• Remove deployment Id column
• Rename Env to ring column
• Introduce state column to normal mode but remove start time column, and also perhaps add duration column
• Consider truncating some fields on normal mode to adjust the addition of new columns, but there should be no truncation on wide mode
• There should be some wording on deployments awaiting on PR such as "Awaiting Approval" under status to make it clear that the deployment is waiting on human intervention and not stuck or failed

[samiyaakhtar]

Added the following improvements based on today's discussion:

• Introduce an output format csv which outputs the current table but in csv format
• Flag for manual HLD edits, disabled by default, so that the default display is just deployments that are bedrock initiated
• Explore emoji support and colored text support in this cli table, which can help replace the check marks under "result" columns, and make the table more user friendly

cc @andrebriggs @gemorris