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

[Task] Optimization suggestions #1509

Open
3 tasks
gmij opened this issue Jan 23, 2024 · 13 comments
Open
3 tasks

[Task] Optimization suggestions #1509

gmij opened this issue Jan 23, 2024 · 13 comments
Assignees

Comments

@gmij
Copy link

gmij commented Jan 23, 2024

Description

新手使用过程中,碰到的一些问题,及自己的建议。请查收。感谢@tom 的支持。

Task List

  • 系统设置的位置,增加存储层配置。不要让用户去改配置文件
  • 操作系统监控模板,把流量的显示,调整成按Mb显示,不要用Bytes显示
  • Nginx的配置,只开启stub_status时应允许添加,当前报502错误。
@hertzbeat
Copy link
Contributor

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


Description

Some problems encountered by novices during use and their own suggestions. please check. Thanks @tom for your support.

Task List

  • The location of system settings, adding storage layer configuration. Don’t let users change configuration files
  • Operating system monitoring template, adjust the display of traffic to display in Mb, do not display in Bytes
  • Nginx configuration, it should be allowed to add when only stub_status is enabled. Currently, a 502 error is reported.

@hertzbeat hertzbeat changed the title [Task] 优化建议 [Task] Optimization suggestions Jan 23, 2024
@tomsun28 tomsun28 added the good first issue Good for newcomers label Jan 23, 2024
@tomsun28
Copy link
Contributor

👍

@Calvin979
Copy link
Contributor

I'd like to help

@tomsun28
Copy link
Contributor

I'd like to help

👍👍 welcome, we can consider this two points first.

  • 操作系统监控模板,把流量的显示,调整成按Mb显示,不要用Bytes显示
  • Nginx的配置,只开启stub_status时应允许添加,当前报502错误。

@Calvin979
Copy link
Contributor

Calvin979 commented Jan 29, 2024

@hertzbeat
Copy link
Contributor

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


@Calvin979
Copy link
Contributor

@gmij hello,关于问题3。我这边没有复现出来,你可以试一下以下两种方式:
1、检查下Nginx的路由配置是否正确
2、删除Nginx的yml模板中关于req_status的监控配置
image

@hertzbeat
Copy link
Contributor

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


@gmij hello, regarding question 3. I haven't reproduced it here. You can try the following two methods:

  1. Check whether the routing configuration of Nginx is correct.
  2. Delete the monitoring configuration about req_status in the Nginx yml template.
    image

@gmij
Copy link
Author

gmij commented Feb 24, 2024

@Calvin979

  1. 确认nginx的路由配置是正确的。
  2. 如果req_status 默认可以不启用的话,是否可以在模板中, 注释掉,需要的人自行启用。这样用户体验上,会不会更好?

@hertzbeat
Copy link
Contributor

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


@Calvin979

  1. Confirm that the routing configuration of nginx is correct.
  2. If req_status can be disabled by default, can it be commented out in the template and enabled by those who need it? Will this provide a better user experience?

@Calvin979
Copy link
Contributor

@tomsun28 是否需要注释Nginx监控模板中req status的相关监控配置

@tomsun28
Copy link
Contributor

hi,不启用 req_status 模块的情况下也是可以添加nginx监控的,建议如果用户不需要req_status的相关指标,可以自己去监控模版注释掉对应配置,我们默认还是保留吧。

@hertzbeat
Copy link
Contributor

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


Hi, nginx monitoring can be added without enabling the req_status module. It is recommended that if the user does not need the relevant indicators of req_status, he can comment out the corresponding configuration in the monitoring template. We will keep it by default.

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