-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
expandStringValues function support to map[string]interface{} #4748
expandStringValues function support to map[string]interface{} #4748
Conversation
|
@kilingzhang consider to add a test to prove that this is needed. See #4649 where I explicitly added tests to prove that is not needed, maybe I missed something, but would be good to have a unit tests for that. |
Pressed by mistake closed instead of comment sorry. |
Codecov Report
@@ Coverage Diff @@
## main #4748 +/- ##
=======================================
Coverage 90.72% 90.73%
=======================================
Files 179 179
Lines 10690 10696 +6
=======================================
+ Hits 9699 9705 +6
Misses 770 770
Partials 221 221
Continue to review full report at Codecov.
|
ok . I added some tests to prove it was necessary . |
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.
Good fix. Thanks
Can you please rebase (resolve merge conflicts)? |
eg. processors: attributes/example: actions: - key: "${DB_KEY}" action: "${OPERATION}"
799a41e
to
3f87dc5
Compare
* formatting fixes (#4751) Cleaning up the changelog with formatting fixes. * expandStringValues function support to map[string]interface{} (#4748) * expandStringValues function support to map[string]interface{} eg. processors: attributes/example: actions: - key: "${DB_KEY}" action: "${OPERATION}" * add some test to prove that this is needed . * update changelog Co-authored-by: Alex Boten <[email protected]> * prepare v0.43.1 release (#4753) This release includes a regression in the 0.43.0 release. Co-authored-by: kilingzhang <[email protected]>
Description:
The use and expansion of environment variables is supported in the Collector configuration. For example: