Skip to content

[pkg/ottl] Implement MarshalLogObject for the ottlmetric context #38103

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

Closed
edmocosta opened this issue Feb 21, 2025 · 2 comments · Fixed by #38293
Closed

[pkg/ottl] Implement MarshalLogObject for the ottlmetric context #38103

edmocosta opened this issue Feb 21, 2025 · 2 comments · Fixed by #38293
Labels
enhancement New feature or request pkg/ottl priority:p1 High

Comments

@edmocosta
Copy link
Contributor

Component(s)

pkg/ottl

Is your feature request related to a problem? Please describe.

The MarshalLogObject(encoder zapcore.ObjectEncoder) function is used by the logger library to print out the TransformContext value. It's currently implemented for all other contexts, but not for metrics, leading debug logs to print empty values:

initial TransformContext before executing StatementSequence	{"TransformContext": {}}

Describe the solution you'd like

Similar to other contexts implementations, so the debug logs will properly print it out.

Describe alternatives you've considered

No response

Additional context

No response

@edmocosta edmocosta added enhancement New feature or request needs triage New item requiring triage labels Feb 21, 2025
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@odubajDT
Copy link
Contributor

Hi, I can take a look at this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pkg/ottl priority:p1 High
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants