Skip to content

Conversation

@SamerJ
Copy link
Contributor

@SamerJ SamerJ commented Aug 14, 2024

When SkipLifecycle & SkipShutdown are set to true, the generated file will have an unused import confmaptest

Description

The Metadata Generator had a bug described above, With this change, that issue has been addressed.

Link to tracking issue

Fixes #10866

@SamerJ SamerJ requested review from a team and jpkrohling August 14, 2024 07:54
@SamerJ SamerJ changed the title Fixed bug component_test.go.tmpl Fixed bug in component_test.go.tmpl Aug 14, 2024
@SamerJ SamerJ changed the title Fixed bug in component_test.go.tmpl [cmd/mdatagen] Fixed bug in component_test.go.tmpl Aug 14, 2024
@codecov
Copy link

codecov bot commented Aug 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.62%. Comparing base (343f449) to head (55c5120).
Report is 6 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #10882   +/-   ##
=======================================
  Coverage   91.62%   91.62%           
=======================================
  Files         406      406           
  Lines       19046    19046           
=======================================
  Hits        17450    17450           
  Misses       1237     1237           
  Partials      359      359           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SamerJ SamerJ force-pushed the bugfix branch 2 times, most recently from 0960e8e to 8188e08 Compare August 16, 2024 12:06
@SamerJ
Copy link
Contributor Author

SamerJ commented Aug 19, 2024

Hello @jpkrohling,

All checks for the PR are green. Should I rebase my PR on the latest commit every now and then or should I leave the PR?
For the time being, I leave this PR to be reviewed manually.

Thanks in Advanace

SamerJ added 2 commits August 21, 2024 16:01
When SkipLifecycle & SkipShutdown are set to true, the generated file will have an unused import confmaptest
Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

thanks for the fix!

@codeboten codeboten merged commit 1c20608 into open-telemetry:main Aug 22, 2024
@github-actions github-actions bot added this to the next release milestone Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[cmd/mdatagen] Mdatagen generates faulty test code

2 participants