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

Add standard EC setting to admin info #4918

Merged
merged 1 commit into from
May 3, 2024

Conversation

klauspost
Copy link
Contributor

Description

Discretely adds the standard EC setting to regular mc admin info.

λ mc admin info play
●  play.min.io
   Uptime: 1 hour
   Version: 2024-04-15T10:03:01Z
   Network: 1/1 OK
   Drives: 4/4 OK
   Pool: 1

Pools:
   1st, Erasure sets: 1, Drives per erasure set: 4

945 MiB Used, 409 Buckets, 5,498 Objects, 52 Versions, 4 Delete Markers
4 drives online, 0 drives offline, EC:2

Will save many deep dives into JSON.

How to test this PR?

mc admin info play

Types of changes

  • New feature (non-breaking change which adds functionality)

Discretely adds the standard EC setting to regular `mc admin info`.

```
λ mc admin info play
●  play.min.io
   Uptime: 1 hour
   Version: 2024-04-15T10:03:01Z
   Network: 1/1 OK
   Drives: 4/4 OK
   Pool: 1

Pools:
   1st, Erasure sets: 1, Drives per erasure set: 4

945 MiB Used, 409 Buckets, 5,498 Objects, 52 Versions, 4 Delete Markers
4 drives online, 0 drives offline, EC:2
```

Will save many deep dives into JSON.
Copy link
Contributor

@shtripat shtripat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Member

@vadmeste vadmeste left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@harshavardhana harshavardhana merged commit b471de8 into minio:master May 3, 2024
7 checks passed
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.

4 participants