Skip to content

Conversation

@yuqiu123
Copy link
Contributor

需要 review 的章节:introduction, basic concepts, monitoring iomesh, iomesh metrics


A user request for a snapshot of a volume, which is similar to a persistent volume request.

[**Volume Snapshot Class**](https://kubernetes.io/docs/concepts/storage/volume-snapshot-classes/)
Copy link
Contributor

Choose a reason for hiding this comment

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

建议和 Volume Snapshot 的位置对调下

Copy link
Contributor Author

Choose a reason for hiding this comment

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

volumesnapshotclass 的定义提到了 volumesnapshot, 感觉得先说一下 volumesnapshot?


Before deploying and using IOMesh, familiarity with the following concepts is suggested.

[**Kubernetes**](https://kubernetes.io/)
Copy link
Contributor

@yixin-chen-23 yixin-chen-23 Mar 28, 2023

Choose a reason for hiding this comment

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

这一章可不可以采取 有链接的标题带下划虚线,没有链接的不带下划虚线的形式呀;感觉现在这样,页面里有的标题紫有的标题黑,不太和谐

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这个得请教下 @gxglls 能不能实现?

# Configure ServiceMonitor for Prometheus.
serviceMonitor:
create: true # Set it to true to create a serviceMonitor object, which defaults to false.
namespace: "iomesh-system" # Create a Namespace for serviceMonitor object, which defaults to iomesh-system.
Copy link
Contributor

Choose a reason for hiding this comment

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

@cospotato 是不是这一部分所有的 Namespace 都是应该保持一致的?

Copy link
Contributor

Choose a reason for hiding this comment

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

是的

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cospotato 那现在文档是对的吗

Copy link
Contributor

Choose a reason for hiding this comment

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

@yuqiu123 指的是哪份文档?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yixin 说的【是不是这一部分所有的 Namespace 都是应该保持一致的】

|------|------|--------|
| Name | The node name. | "node-1" |
| Namespace | The namespace where the node is located. | "iomesh-system" |
| Health Status | Shows the node health status, including:<p>Initializing</p><p>Healthy</p><p>Error</p><p>Disconnected</p> | "Healthy"|
Copy link
Contributor

Choose a reason for hiding this comment

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

这里的格式不太对,在线文档里没有以 bullet list 形式展现

Copy link
Contributor

Choose a reason for hiding this comment

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

或者用灰色底色块标注也行

Co-authored-by: yixin-chen-23 <124547192+yixin-chen-23@users.noreply.github.com>
| Name | The node name. | "node-1" |
| Namespace | The namespace where the node is located. | "iomesh-system" |
| Health Status | Shows the node health status, including:<p>Initializing</p><p>Healthy</p><p>Error</p><p>Disconnected</p> | "Healthy"|
|Capacity | The total capacity of the node.| "2 TiB" |
Copy link
Contributor

@nanana-w nanana-w Mar 30, 2023

Choose a reason for hiding this comment

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

Capacity 和 space usage 和 tower 上的不一致,另外这里表示的是存储容量,用 storage capacity 和 storage capacity usage 是不是更合适?
@yixin-chen-23

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@yixin-chen-23 yixin 看看这里哈

# Configure ServiceMonitor for Prometheus.
serviceMonitor:
create: true # Set it to true to create a serviceMonitor object, which defaults to false.
namespace: "iomesh-system" # Create a Namespace for serviceMonitor object, which defaults to iomesh-system.
Copy link
Contributor

Choose a reason for hiding this comment

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

是的

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.

6 participants