forked from open-telemetry/opentelemetry-collector-contrib
-
Notifications
You must be signed in to change notification settings - Fork 6
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
[pull] main from open-telemetry:main #47
Open
pull
wants to merge
12
commits into
AmadeusITGroup:main
Choose a base branch
from
open-telemetry:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
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
…try::max_retries (#35571) **Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> The new retry::max_retries will be exactly retry::max_requests - 1, but it will be much more intuitive to the end user. Deprecate retry::max_requests. **Link to tracking Issue:** <Issue number if applicable> Fixes #32344 **Testing:** <Describe what testing was performed and which tests were added.> **Documentation:** <Describe the documentation added.>
#### Description Fixed a typo in the pkg/stanza/docs/types/field.md documentation. The link was incorrectly pointing to ../types/field.md and is now corrected to ../types/entry.md. #### Testing #### Documentation Signed-off-by: YifanYang6 <[email protected]>
…35233) **Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> Configuration validation is done during collector's startup, making it redundant when being called inside component's logic. This PR removes the Validate call done during exporter's constructor. **Link to tracking Issue:** <Issue number if applicable> #33498 (Last component, will close the issue) **Testing:** <Describe what testing was performed and which tests were added.> Added default config use case (validate error) **Documentation:** <Describe the documentation added.>
…only (#35839) <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> #### Description Make OTel mapping mode use RequireDataStream in docappender, which means it will only send to data streams. This prevents auto creating regular indices in OTel mapping mode due to a race condition in Elasticsearch where otel-data index templates are not ready. <!-- Issue number (e.g. #1234) or full URL to issue, if applicable. --> #### Link to tracking issue <!--Describe what testing was performed and which tests were added.--> #### Testing <!--Describe the documentation added.--> #### Documentation <!--Please delete paragraphs that you did not use before submitting.-->
**Description:** Follow receiver contract for `loki`. This also includes an internal errorutil package which will be used by other network receivers as well. **Link to tracking Issue:** #5909 **Testing:** Added
…e` (#35864) Setting `heaader` to `false` results in error when starting the collector. The value of `header` must either be a map or `null`.
**Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> Rewrites most tests to use a pkg/golden like approach: - tests are directories under `testdata/` - tests consist of multiple stages, ran in series - each stage is a [txtar](https://pkg.go.dev/golang.org/x/tools/txtar), containing: - `in`: pmetric yaml of the `ConsumeMetrics` input - `out`: expected output on the sink after calling `ConsumeMetrics` The multi-stage setup allows to exercise multiple Requests in one test. Using txtar allows to co-locate common yaml easily. I plan to add metric assertions in there later too. **Link to tracking Issue:** none **Testing:** Tests were rewritten **Documentation:** not needed
This follows the recommendations from the new lint version Signed-off-by: Alex Boten <[email protected]>
#### Description Fixes some typos in the Azure Receiver readme.
#### Description Fixes incomplete mechanical update in #30890. The new version of golang-lru uses type params. Recreates #35824, which Renovate kindly closed without merging 🥲 #### Link to tracking issue Supersedes #30890 #### Testing Ran the unit tests. #### Documentation N/A Co-authored-by: Alex Boten <[email protected]>
…35797) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/Antonboom/testifylint](https://redirect.github.com/Antonboom/testifylint) | `v1.4.3` -> `v1.5.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fAntonboom%2ftestifylint/v1.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fAntonboom%2ftestifylint/v1.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fAntonboom%2ftestifylint/v1.4.3/v1.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fAntonboom%2ftestifylint/v1.4.3/v1.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>Antonboom/testifylint (github.com/Antonboom/testifylint)</summary> ### [`v1.5.0`](https://redirect.github.com/Antonboom/testifylint/releases/tag/v1.5.0): New `contains`, `regexp`, `encoded-compare'`! [Compare Source](https://redirect.github.com/Antonboom/testifylint/compare/v1.4.3...v1.5.0) #### What's Changed - ci: checkout before setup-go by [@​mmorel-35](https://redirect.github.com/mmorel-35) in [https://github.com/Antonboom/testifylint/pull/137](https://redirect.github.com/Antonboom/testifylint/pull/137) - new checker 'regexp' by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/136](https://redirect.github.com/Antonboom/testifylint/pull/136) - new checker `contains` by [@​mmorel-35](https://redirect.github.com/mmorel-35) in [https://github.com/Antonboom/testifylint/pull/152](https://redirect.github.com/Antonboom/testifylint/pull/152) - build(deps): bump golang.org/x/tools from 0.21.0 to 0.22.0 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/Antonboom/testifylint/pull/163](https://redirect.github.com/Antonboom/testifylint/pull/163) - build(deps): bump golang.org/x/tools from 0.22.0 to 0.23.0 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/Antonboom/testifylint/pull/168](https://redirect.github.com/Antonboom/testifylint/pull/168) - enable several rules from revive by [@​mmorel-35](https://redirect.github.com/mmorel-35) in [https://github.com/Antonboom/testifylint/pull/164](https://redirect.github.com/Antonboom/testifylint/pull/164) - build(deps): bump golang.org/x/tools from 0.23.0 to 0.24.0 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/Antonboom/testifylint/pull/173](https://redirect.github.com/Antonboom/testifylint/pull/173) - useless-assert: more cases by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/182](https://redirect.github.com/Antonboom/testifylint/pull/182) - formatter: support require only and suite only packages by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/188](https://redirect.github.com/Antonboom/testifylint/pull/188) - new checker 'encoded-compare' by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/179](https://redirect.github.com/Antonboom/testifylint/pull/179) - empty: adjust report for NotEmpty+len cases by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/189](https://redirect.github.com/Antonboom/testifylint/pull/189) - error-nil: support Empty and Zero error by [@​mmorel-35](https://redirect.github.com/mmorel-35) in [https://github.com/Antonboom/testifylint/pull/153](https://redirect.github.com/Antonboom/testifylint/pull/153) - Go 1.23 by [@​Antonboom](https://redirect.github.com/Antonboom) in [https://github.com/Antonboom/testifylint/pull/191](https://redirect.github.com/Antonboom/testifylint/pull/191) **Full Changelog**: Antonboom/testifylint@v1.4.3...v1.5.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4xMjAuMSIsInVwZGF0ZWRJblZlciI6IjM4LjEyMC4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJkZXBlbmRlbmNpZXMiLCJyZW5vdmF0ZWJvdCJdfQ==--> --------- Signed-off-by: Alex Boten <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> Co-authored-by: Alex Boten <[email protected]>
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.
See Commits and Changes for more details.
Created by pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )