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

[proposal] The CRI of "Status" returns more information of system #2568

Closed
zjumoon01 opened this issue Dec 18, 2018 · 3 comments
Closed

[proposal] The CRI of "Status" returns more information of system #2568

zjumoon01 opened this issue Dec 18, 2018 · 3 comments
Assignees

Comments

@zjumoon01
Copy link
Contributor

Why you need it?

The response of "Status" CRI contains extra information of the Runtime. However, there is only "golang version" in the extra information returned. It will be useful if more information is returned, such as "pouch root dir". In my scenario, I intend to limit rootfs iops/bps for pods. So I need get pouch root dir first and translate into its device. Then I can set device and rate limit to pods.

How it could be?

StatusResponse could contains more extra information,

Other related information

What other extra information can be returned ? That can be discussed. I think we can refer to "pouch info"

@starnop
Copy link
Contributor

starnop commented Dec 18, 2018

Yes, you are absolutely right. In fact, we don't know what the end users really need. Thanks for your input, and I hope that you can have a list here. 👍

@zhuangqh
Copy link
Contributor

zhuangqh commented Dec 18, 2018

RFC:

key value
config marshal cri config

@zjumoon01 @starnop @allencloud

@zhuangqh
Copy link
Contributor

compared with directly put each kv to the map.

  1. json value support not only string value
  2. It is unnecessary to know the exactly spell of each key.

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

No branches or pull requests

4 participants