Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(Draft) Add a few instance commands #223

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

dexhorthy
Copy link
Member

@dexhorthy dexhorthy commented Dec 28, 2022

Outstanding work to make this ready

  • pact test for usage of /instances V4 API with customerSelector
  • add more k8s info to instances list view

Right now you need a customer to make this work, because I couldn't get the appSelector to work with the v4 api. There's a few things we should add, including listing all instances, and a few k8s cluster details we can add in.

$ replicated customer instance ls Paypal
ID         STATUS     VERSION    VERSIONS-BEHIND    VERSION-AGE    VERSION-AGE-RELATIVE    LAST-CHECKIN        TTI-LICENSE    TTI-INSTANCE    FIRST-READY
2JByR1V    ready      1.0.2      4                  3d8h           3d8h                    2022-12-23 00:00    9m44s          0m4s            2022-12-20 00:00
2JK93av    ready      1.2.0      1                  1d2h           1d2h                    2022-12-28 15:05    2d22h          7m19s           2022-12-23 00:00
2JK9CVv    missing    1.0.2      4                  3d8h           3d8h                    2022-12-28 15:02    2d22h          1m50s           2022-12-23 00:00
2JMsKqo    ready      1.2.0      1                  1d2h           1d2h                    2022-12-25 00:00    4d1h           3h56m           2022-12-24 00:00
2JMsQiT    ready      1.2.0      1                  1d2h           1d2h                    2022-12-25 00:00    4d1h           1m31s           2022-12-24 00:00

Using the -o wide flag will give the following columns:

ID STATUS VERSION VERSIONS-BEHIND VERSION-AGE VERSION-AGE-RELATIVE LAST-CHECKIN TTI-LICENSE TTI-INSTANCE FIRST-SEEN FIRST-READY LICENSE-CREATED LATEST-CHANNEL-RELEASE FULL-ID

instance inspect will show a mini terminal version of the instances page. This is a little goofy but I don't hate it. It gives a good way to test multiple consumers of the backend data.

replicated customer instance inspect Paypal 2Jvas53

Screen Shot 2022-12-28 at 9 26 35 AM

Right now you need a customer to make this work, because I couldn't get the `appSelector` to work with the v4 api

```
replicated customer instance ls Boeing
```

```
replicated customer instance inspect Boeing 2Jvas53
```
@dexhorthy dexhorthy changed the title Add a few instance commands (Draft) Add a few instance commands Dec 28, 2022
@dexhorthy
Copy link
Member Author

I held off on this because there was testing infrastructure missing at the time, but if anyone wants to resolve conflics and run with it, they should feel welcome to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant