Skip to content
This repository was archived by the owner on Mar 29, 2021. It is now read-only.

Conversation

@ujhgj
Copy link
Owner

@ujhgj ujhgj commented Jan 17, 2018

Без задания со *
Без логирования

ujhgj added 3 commits January 17, 2018 13:00
с единой секцией для всех endpoint`ов всех компонент
с отдельными секциями для endpoint`ов компонент
дашборды с параметрами
@ujhgj ujhgj requested review from Nklya, chromko and vitkhab January 17, 2018 16:03
забыл добавить сам чарт в гит
Copy link
Collaborator

@chromko chromko left a comment

Choose a reason for hiding this comment

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

Основная работа выполнена.
Также оставил пару коментов

{
"name": "DS_PROMETHEUS_SERVER",
"label": "Prometheus Server",
"name": "DS_HTTP://35.198.163.2",
Copy link
Collaborator

Choose a reason for hiding this comment

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

А почему IP-адресом на датасорс ссылался?

action: keep
regex: comment

- job_name: 'reddit-post'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Хотел бы на будущее отметить, что в текущем конфиге будут повторяться таргеты. Одни и те же таргеты попадут в группы post,ui,comment и в группу reddit-production. Тогда и метрики будут задублированы. Соотвественно лучше делать конфиги такими, чтобы таргеты были уникальны.
Это просто, чтобы помнить об этом. Ошибки в текущем конфиге в соотвествии с дз нет

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants