Skip to content

Conversation

Dorthu
Copy link
Collaborator

@Dorthu Dorthu commented Apr 23, 2021

After looking through Events with the CLI, it seemed that many of the
columns were not super useful; for instance, seen and read are hard
to change via the CLI and are more relevant to Cloud, and message
isn't present at all. entity.label is also absent, although it looks
like there were intentions to show it.

This change does the following:

  • Removes seen and read from default column display list
  • Adds entity.label and message to default column display list

New output looks like this:

┌───────────┬──────────┬─────────────────┬────────────────────────────────┬─────────────────────┬──────────┬──────────────┬───────────────────────┐
│ id        │ username │ action          │ label                          │ created             │ duration │ status       │ message               │
├───────────┼──────────┼─────────────────┼────────────────────────────────┼─────────────────────┼──────────┼──────────────┼───────────────────────┤
│ 162178765 │ wsmith   │ image_upload    │ test-image-2                   │ 2021-04-21T17:18:10 │ 4967     │ finished     │                       │
│ 162176559 │ wsmith   │ image_upload    │ test-image                     │ 2021-04-21T17:07:56 │ 59       │ failed       │ Upload window expired │

@Dorthu Dorthu force-pushed the feature/cli-show-event-message branch from 3377163 to cce1502 Compare April 23, 2021 11:40
After looking through Events with the CLI, it seemed that many of the
columns were not super useful; for instance, `seen` and `read` are hard
to change via the CLI and are more relevant to Cloud, and `message`
isn't present at all.  `entity.label` is also absent, although it looks
like there were intentions to show it.

This change does the following:

 * Removes `seen` and `read` from default column display list
 * Adds `entity.label` and `message` to default column display list

New output looks like this:

```
┌───────────┬──────────┬─────────────────┬────────────────────────────────┬─────────────────────┬──────────┬──────────────┬───────────────────────┐
│ id        │ username │ action          │ label                          │ created             │ duration │ status       │ message               │
├───────────┼──────────┼─────────────────┼────────────────────────────────┼─────────────────────┼──────────┼──────────────┼───────────────────────┤
│ 162178765 │ wsmith   │ image_upload    │ test-image-2                   │ 2021-04-21T17:18:10 │ 4967     │ finished     │                       │
│ 162176559 │ wsmith   │ image_upload    │ test-image                     │ 2021-04-21T17:07:56 │ 59       │ failed       │ Upload window expired │
```
@Dorthu Dorthu force-pushed the feature/cli-show-event-message branch from cce1502 to 5d37f1c Compare April 23, 2021 11:41
@Dorthu Dorthu changed the title feature/cli show event message Changed columns for linode-cli events list Apr 23, 2021
@bbiggerr bbiggerr merged commit ad7adcd into linode:development Apr 28, 2021
LBGarber pushed a commit to LBGarber/linode-api-docs that referenced this pull request Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants