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(otelcol): allow overriding debug metric level for all otelcol
components
#825
base: main
Are you sure you want to change the base?
feat(otelcol): allow overriding debug metric level for all otelcol
components
#825
Conversation
docs/sources/shared/reference/components/otelcol-debug-metrics-block.md
Outdated
Show resolved
Hide resolved
docs/sources/shared/reference/components/otelcol-debug-metrics-block.md
Outdated
Show resolved
Hide resolved
docs/sources/shared/reference/components/otelcol-debug-metrics-block.md
Outdated
Show resolved
Hide resolved
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.
Wow, such a big PR! 😅 Thank you very much! It looks good and I only left a few minor comments.
docs/sources/shared/reference/components/otelcol-debug-metrics-block.md
Outdated
Show resolved
Hide resolved
… except `auth.* due to cyclic imports Signed-off-by: hainenber <dotronghai96@gmail.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
…auth` components Signed-off-by: hainenber <dotronghai96@gmail.com>
…level Signed-off-by: hainenber <dotronghai96@gmail.com>
…ert` Signed-off-by: hainenber <dotronghai96@gmail.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Co-authored-by: Paulin Todev <paulin.todev@gmail.com> Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
…of arg table Signed-off-by: hainenber <dotronghai96@gmail.com>
… components Signed-off-by: hainenber <dotronghai96@gmail.com>
…ation Signed-off-by: hainenber <dotronghai96@gmail.com>
45e98d7
to
f81140c
Compare
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.
Thank you! I left a comment with what I believe is the only thing left to polish. I'd be happy to merge it afterwards :)
|
||
Hierarchy | Block | Description | Required | ||
----------|------------|--------------------------------------|--------- | ||
debug_metrics | [debug_metrics][] | Configures the metrics that this component generates to monitor its state. | no |
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.
debug_metrics | [debug_metrics][] | Configures the metrics that this component generates to monitor its state. | no | |
debug_metrics | [debug_metrics][] | Configures the metrics that this component generates to monitor its state. | no |
Would you mind doing a search and replace for this please? It's ok to put one space in every table, even if it's not consistent with the other rows in the table.
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.
Done! :D
Signed-off-by: hainenber <dotronghai96@gmail.com>
PR Description
otelcol.*
componentsDebugMetrics
as imethod for allotelcol.*
Argument interfacedebug_metrics
block for allotelcol.*
componentsDebugMetricsArg
for missing OtelCol componentsWhich issue(s) this PR fixes
Closes #792
Notes to the Reviewer
PR Checklist