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

curvefs/monitor: promethus #1237

Merged
merged 1 commit into from
Apr 15, 2022
Merged

Conversation

Cyber-SiKu
Copy link
Contributor

What problem does this PR solve?

Issue Number: close #1236

Problem Summary:

  1. add curvefs monitor
  2. curvefs_tool status-mds show dummy port for dummy mds

What is changed and how it works?

What's Changed:

How it Works:

Side effects(Breaking backward compatibility? Performance regression?):

Check List

  • Relevant documentation/comments is changed or added
  • I acknowledge that all my contributions will be made under the project's license

@Cyber-SiKu
Copy link
Contributor Author

recheck

@@ -0,0 +1,159 @@
## 目录结构介绍

Copy link
Contributor

Choose a reason for hiding this comment

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

Improve the document format like output of command tree.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Improve the document format like output of command tree.

fix


##### target_json.py

用于生成prometheus监控对象的python脚本,每隔一段时间用curve_ops_tool拉取监控目标并更新。
Copy link
Contributor

Choose a reason for hiding this comment

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

curve_ops_tool or curvefs_tool ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

curve_ops_tool or curvefs_tool ?

fix


grafana的启动配置文件,将映射到容器的 `/etc/grafana/grafana.ini` 上

##### grafana/report:
Copy link
Contributor

Choose a reason for hiding this comment

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

It is an empty dir, delete it if don't need now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is an empty dir, delete it if don't need now.

it use for daily report.


```curve-monitor.sh start ```

* 部署grafana每日报表
Copy link
Contributor

Choose a reason for hiding this comment

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

The file report.py not exist. grafana-report.py?

Kepp this README file and monitor dir consistent.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The file report.py not exist. grafana-report.py?

Kepp this README file and monitor dir consistent.

From the project point of view, it should be grafana-report.py

]
},
"time": {
"from": "now-6h",
Copy link
Contributor

Choose a reason for hiding this comment

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

Default 30min is ok.

@Cyber-SiKu Cyber-SiKu force-pushed the curvefs/monitor branch 4 times, most recently from 32a3579 to 8cd649e Compare April 13, 2022 11:59
├── grafana # grafana相关目录
│ ├── dashboards # grafana所有dashboards的json文件存放目录,grafana将从该目录加载文件来创建dashboards;
| | | # 通过update_dashboard.sh脚本来更新最新的dashboards。
│ │ ├── etcd.json
Copy link
Contributor

Choose a reason for hiding this comment

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

forget client.json

Copy link
Contributor Author

Choose a reason for hiding this comment

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

forget client.json

ok

#!/bin/sh

#
# Copyright (c) 2020 NetEase Inc.
Copy link
Contributor

Choose a reason for hiding this comment

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

2022

Copy link
Contributor Author

Choose a reason for hiding this comment

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

2022

fix

1. add curvefs monitor
2. curvefs_tool status-mds show dummy port for dummy mds
3. add port in mountpoints, change mountpoint from {hostname}:{mountpoint} to {hostname}:{port}:{mountpath}
@Cyber-SiKu Cyber-SiKu merged commit d17b8de into opencurve:master Apr 15, 2022
@Cyber-SiKu Cyber-SiKu deleted the curvefs/monitor branch April 19, 2022 05:54
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.

Curvefs: monitor
3 participants