Skip to content
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

Change binary to use io.Reader #2329

Merged
merged 3 commits into from
Dec 22, 2023
Merged

Change binary to use io.Reader #2329

merged 3 commits into from
Dec 22, 2023

Conversation

therve
Copy link
Contributor

@therve therve commented Dec 22, 2023

os.File is very inconvenient when you work with in memory data. This is a small backward incompatible change but I think it's alright.

os.File is very inconvenient when you work with in memory data. This is
a small backward incompatible change but I think it's alright.
@therve therve added the changelog/Changed Changed features results into a major version bump label Dec 22, 2023
@therve therve requested a review from a team as a code owner December 22, 2023 13:21
@therve therve merged commit 2a5860b into master Dec 22, 2023
13 checks passed
@therve therve deleted the therve/fix-file-usage branch December 22, 2023 14:24
github-actions bot pushed a commit that referenced this pull request Dec 22, 2023
* Change binary to use io.Reader

os.File is very inconvenient when you work with in memory data. This is
a small backward incompatible change but I think it's alright.

* Cassettes
:# 2a5860b
mx-psi pushed a commit to open-telemetry/opentelemetry-collector-contrib that referenced this pull request Jan 16, 2024
…30577)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[github.com/DataDog/datadog-api-client-go/v2](https://togithub.com/DataDog/datadog-api-client-go)
| `v2.20.0` -> `v2.21.0` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.20.0/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.20.0/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>DataDog/datadog-api-client-go
(github.com/DataDog/datadog-api-client-go/v2)</summary>

###
[`v2.21.0`](https://togithub.com/DataDog/datadog-api-client-go/releases/tag/v2.21.0)

[Compare
Source](https://togithub.com/DataDog/datadog-api-client-go/compare/v2.20.0...v2.21.0)

<!-- Release notes generated using configuration in .github/release.yml
at v2.21.0 -->

#### What's Changed

##### Added

- Add priority field to SDS rule and standard-pattern by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2317
- Document new `resource_collection` and
`is_security_command_center_enabled` fields in GCP APIs by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2318
- Add SAML attributes by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2311
- Security Monitoring - Support custom third party rules by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2337
- Add public API support for time-slice SLOs by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2328
- Add included_keyword_configuration field to SDS rule by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2340
- Update Documentation for APM DevSecOps by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2341

##### Changed

- Change binary to use io.Reader by
[@&#8203;therve](https://togithub.com/therve) in
[DataDog/datadog-api-client-go#2329
- Mark v1 downtime endpoints as deprecated by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2334
- Adding Cloud Cost Management API Documentation by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2326
- Add support for Cloudflare API `zone` and `resource` fields by
[@&#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2339

**Full Changelog**:
DataDog/datadog-api-client-go@v2.20.0...v2.21.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 has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMjcuMCIsInVwZGF0ZWRJblZlciI6IjM3LjEyNy4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <[email protected]>
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Feb 1, 2024
…pen-telemetry#30577)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[github.com/DataDog/datadog-api-client-go/v2](https://togithub.com/DataDog/datadog-api-client-go)
| `v2.20.0` -> `v2.21.0` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.20.0/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.20.0/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>DataDog/datadog-api-client-go
(github.com/DataDog/datadog-api-client-go/v2)</summary>

###
[`v2.21.0`](https://togithub.com/DataDog/datadog-api-client-go/releases/tag/v2.21.0)

[Compare
Source](https://togithub.com/DataDog/datadog-api-client-go/compare/v2.20.0...v2.21.0)

<!-- Release notes generated using configuration in .github/release.yml
at v2.21.0 -->

#### What's Changed

##### Added

- Add priority field to SDS rule and standard-pattern by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2317
- Document new `resource_collection` and
`is_security_command_center_enabled` fields in GCP APIs by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2318
- Add SAML attributes by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2311
- Security Monitoring - Support custom third party rules by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2337
- Add public API support for time-slice SLOs by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2328
- Add included_keyword_configuration field to SDS rule by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2340
- Update Documentation for APM DevSecOps by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2341

##### Changed

- Change binary to use io.Reader by
[@&open-telemetry#8203;therve](https://togithub.com/therve) in
[DataDog/datadog-api-client-go#2329
- Mark v1 downtime endpoints as deprecated by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2334
- Adding Cloud Cost Management API Documentation by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2326
- Add support for Cloudflare API `zone` and `resource` fields by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2339

**Full Changelog**:
DataDog/datadog-api-client-go@v2.20.0...v2.21.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 has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMjcuMCIsInVwZGF0ZWRJblZlciI6IjM3LjEyNy4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <[email protected]>
anthoai97 pushed a commit to anthoai97/opentelemetry-collector-contrib that referenced this pull request Feb 12, 2024
…pen-telemetry#30577)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[github.com/DataDog/datadog-api-client-go/v2](https://togithub.com/DataDog/datadog-api-client-go)
| `v2.20.0` -> `v2.21.0` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.20.0/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fDataDog%2fdatadog-api-client-go%2fv2/v2.20.0/v2.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>DataDog/datadog-api-client-go
(github.com/DataDog/datadog-api-client-go/v2)</summary>

###
[`v2.21.0`](https://togithub.com/DataDog/datadog-api-client-go/releases/tag/v2.21.0)

[Compare
Source](https://togithub.com/DataDog/datadog-api-client-go/compare/v2.20.0...v2.21.0)

<!-- Release notes generated using configuration in .github/release.yml
at v2.21.0 -->

#### What's Changed

##### Added

- Add priority field to SDS rule and standard-pattern by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2317
- Document new `resource_collection` and
`is_security_command_center_enabled` fields in GCP APIs by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2318
- Add SAML attributes by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2311
- Security Monitoring - Support custom third party rules by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2337
- Add public API support for time-slice SLOs by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2328
- Add included_keyword_configuration field to SDS rule by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2340
- Update Documentation for APM DevSecOps by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2341

##### Changed

- Change binary to use io.Reader by
[@&open-telemetry#8203;therve](https://togithub.com/therve) in
[DataDog/datadog-api-client-go#2329
- Mark v1 downtime endpoints as deprecated by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2334
- Adding Cloud Cost Management API Documentation by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2326
- Add support for Cloudflare API `zone` and `resource` fields by
[@&open-telemetry#8203;api-clients-generation-pipeline](https://togithub.com/api-clients-generation-pipeline)
in
[DataDog/datadog-api-client-go#2339

**Full Changelog**:
DataDog/datadog-api-client-go@v2.20.0...v2.21.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 has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMjcuMCIsInVwZGF0ZWRJblZlciI6IjM3LjEyNy4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/Changed Changed features results into a major version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants