-
Notifications
You must be signed in to change notification settings - Fork 399
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
feat: add loong collector monitor #1636
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
messixukejia
requested changes
Jul 24, 2024
messixukejia
approved these changes
Jul 26, 2024
linrunqi08
approved these changes
Jul 29, 2024
yyuuttaaoo
reviewed
Jul 29, 2024
contentPtr->set_key(VALUE_PREFIX + gauge->GetName()); | ||
contentPtr->set_value(ToString(gauge->GetValue())); | ||
} | ||
for (auto& item : tmp->GetDoubleGauges()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Double ToString的精度可能比较长,后续看看是否需要控制一下
yyuuttaaoo
approved these changes
Jul 29, 2024
yyuuttaaoo
reviewed
Jul 29, 2024
|
||
using MetricLabels = std::vector<std::pair<std::string, std::string>>; | ||
using MetricLabelsPtr = std::shared_ptr<MetricLabels>; | ||
using DynamicMetricLabels = std::vector<std::pair<std::string, std::function<std::string()>>>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里function的要求是线程安全
yyuuttaaoo
approved these changes
Jul 29, 2024
This was referenced Sep 20, 2024
henryzhx8
pushed a commit
that referenced
this pull request
Jan 20, 2025
本次代码评审主要涉及多文件的更新,包括添加、修改和删除代码,核心目的是为了实现和优化日志处理系统的配置管理、插件系统、自定义指标记录、以及测试用例的改进。具体包括增加对时间戳格式的支持、新增和调整函数以支持插件的动态添加和配置、增强了上下文管理以及日志处理逻辑,同时优化了代码结构和提高了代码的可读性和模块化。 Link: https://code.alibaba-inc.com/sls/ilogtail/codereview/17996585 * fix unittest include and devcontainer user (#1634) * e2e test case * fix checkpoint dump idx in reader array (#1637) * fix: input_kafka consumer wont close due to ctx never done. (#1615) Co-authored-by: licht <[email protected]> * Feat: prometheus relabel lib (#1614) * feat: MetricEvent tags read only iterator * feat: prometheus relabel lib * chore: update code style * chore: update code style * chore: add relabel action test case * chore: update code style * chore: add more test case * feat: labels hash calc * chore: update code style * chore: update enum Action style * chore: use static string reference * fix mssql latest image incompatible (#1643) * fix: GTID Truncation Issue and Improve Consistency in Checkpoint Management (#1644) * fix: cannot get latest GTID when size exceed 1024 * fix license * move log to proper position * feat: add loong collector monitor (#1636) * add LoongCollectorMonitor * add DoubleGauge * add dynamicLabel * upgrade development image (#1645) * Merge remote-tracking branch 'opensource/main' into e2e * suicide immediately when resource hard limit is met (#1649) * Support config server v2 provider (#1633) * fix * refactor flusher runner and http sink (#1639) * Merge remote-tracking branch 'origin/main' into e2e * Merge remote-tracking branch 'opensource/main' into e2e * Support MetricEvent serialize to LogGroup (#1656) * improve Android compatibility (#1518) * improve Android compatibility * support flusher route for pipeline event group (#1655) * fix processor delimiter parse double quote (#1664) * fix delimiter parse double quote * enable unittest coverage * remove useless codes to improve unittest coverage * feat: export go metrics to cpp (#1667) * export go metric to cpp * change metric format * separate feedbackqueue into three distinct interfaces (#1659) * add timer machanism (#1658) * config server implement gorm store mode (#1400) therefore, mysql, sqlite, postgre and sqlserver are supported Co-authored-by: Ruoyu <[email protected]> * add ebpf input & ut & docs, add options for Security & Observer and make below modifications: 1. modify ebpf input appconfig load method 2. ParseJsonToFlags() support recursive 3. add RecurseParseJsonToFlags ut * 1. add ebpf server 2. add source manager Signed-off-by: qianlu.kk <[email protected]> * Add processor_gotime with nanosecond to other time format conversion capability (#1671) * STC scan * fix blacklist verify * feat: prometheus scrape work, scrape manager, input plugin, and inner processor. (#1641) * Merge remote-tracking branch 'origin/main' into e2e * Merge remote-tracking branch 'upstream/main' into e2e * support janus ci run stc * fix ebpf input doc * feat: Upgrade PluginID to PluginMeta (CPP part) (#1676) * cpp plugin meta * feat: add go process-level metric (#1673) * add go process-level metric * feat: Upgrade PluginID to PluginMeta (Golang part) (#1683) * go plugin meta * Merge remote-tracking branch 'external/main' into e2e * Merge remote-tracking branch 'internal/main' into e2e * Merge remote-tracking branch 'origin/nightly' into e2e * fix error use of timestamp (#1692) * support circular process queue for input with pull mode (#1670) * Add k8s meta module to support k8s meta plugin and k8s meta http server. (#1630) * export k8s metadata through http server * fix * add k8s meta code * refine code * refine code * add host and cidr endpoint * add runner to plugin_export * add runner to plugin_export * refactor to use queue to handle meta * fix go.mod * tmp remove get cidr * fix static check * fix * fix * fix * fix * direct send when handle timer event * doc * unit test * fix * fix * fix * fix * fix * fix * fix * fix lint --------- Co-authored-by: abingcbc <[email protected]> * Merge remote-tracking branch 'external/main' into e2e * Merge remote-tracking branch 'internal/main' into e2e * janus ci * Merge branch 'e2e' of gitlab.alibaba-inc.com:sls/ilogtail-nightly into e2e * test trigger janus ci * test * optimize test engine * add e2e test * fix script permission * add enterprice E2E cases * add build scripts
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
主要改动