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

[pull] main from open-telemetry:main #31

Merged
merged 27 commits into from
Dec 4, 2023

Conversation

pull[bot]
Copy link

@pull pull bot commented Dec 1, 2023

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

renovate bot and others added 4 commits November 30, 2023 12:21
[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/SAP/go-hdb](https://togithub.com/SAP/go-hdb) | require |
patch | `v1.6.2` -> `v1.6.3` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>SAP/go-hdb (github.com/SAP/go-hdb)</summary>

###
[`v1.6.3`](https://togithub.com/SAP/go-hdb/blob/HEAD/RELEASENOTES.md#v163)

[Compare
Source](https://togithub.com/SAP/go-hdb/compare/v1.6.2...v1.6.3)

-   updated dependencies

</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:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/spf13/cast](https://togithub.com/spf13/cast) | require |
minor | `v1.5.1` -> `v1.6.0` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>spf13/cast (github.com/spf13/cast)</summary>

### [`v1.6.0`](https://togithub.com/spf13/cast/releases/tag/v1.6.0)

[Compare
Source](https://togithub.com/spf13/cast/compare/v1.5.1...v1.6.0)

#### What's Changed

- Bump actions/setup-go from 3 to 4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/spf13/cast/pull/180](https://togithub.com/spf13/cast/pull/180)
- Bump github.com/frankban/quicktest from 1.14.4 to 1.14.5 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/spf13/cast/pull/185](https://togithub.com/spf13/cast/pull/185)
- Updates by [@&#8203;sagikazarmark](https://togithub.com/sagikazarmark)
in
[https://github.com/spf13/cast/pull/190](https://togithub.com/spf13/cast/pull/190)
- deps/update20231128 by [@&#8203;bep](https://togithub.com/bep) in
[https://github.com/spf13/cast/pull/205](https://togithub.com/spf13/cast/pull/205)

**Full Changelog**:
spf13/cast@v1.5.1...v1.6.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:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
**Link to tracking Issue:** resolve #27293

**Testing:**
unittest
manual test with opamp-go example:
* collector version: modified otelcontribcol including opampextension
* collector config: 
```yaml
receivers:
  # otlp:
  #   protocols:
  #     grpc:
  filelog:
    include:
      - /tmp/xxxxx.log

exporters:
  debug:
    #verbosity: debug

extensions:
  opamp:
    server:
      ws:
        endpoint: ws://127.0.0.1:4320/v1/opamp
        tls:
          insecure_skip_verify: true

service:
  pipelines:
    logs:
      receivers: [filelog]
      exporters: [debug]
  telemetry:
    logs:
      level: info
  extensions: [opamp]
```
* server ui: ![Screenshot 2023-11-28 at 22-14-59 OpAMP
Server](https://github.com/open-telemetry/opentelemetry-collector-contrib/assets/9400582/38ea49bb-f2f7-4618-859f-2b66257ef89a)

**Documentation:** README.md

---------

Co-authored-by: Jared Tan <jian.tan@daocloud.io>
**Description:** 

Adds a new `ConditionSequence` struct to help handle processing a list
of Conditions. The primary reason to use a `ConditionSequence` is to let
the struct handle errors. Since that is its defining purpose, I opted to
make `ErrorMode` a required argument in the "constructor" instead of an
Option.

I also update `internal/filterottl` to use this struct instead of
`Statements`. This is a non-breaking change.

If we like this pattern, I will do a breaking change to replace
`Statements` with a similar `StatementSequence` struct in a future PR .

See these structs implemented in components here:
#29294

**Link to tracking Issue:** <Issue number if applicable>

Related to
#13545

**Testing:**

Added new tests

**Documentation:** <Describe the documentation added.>
Added godoc comments

---------

Co-authored-by: Evan Bradley <11745660+evan-bradley@users.noreply.github.com>
@pull pull bot added the ⤵️ pull label Dec 1, 2023
sakulali and others added 11 commits December 1, 2023 13:33
…ACE) of TestExtensionObservePods (#29463)

**Description:**
Try to fix flaky test (DATA RACE) of `TestExtensionObservePods`. Since
`config.ObservePods` is set true by default, for
`TestExtensionObserveServices` and `TestExtensionObserveNodes` test
case, we just focus on services observer and nodes observer, so we don't
need to use `k8sObserver.podListerWatcher` which can avoid data race in
the same process.


**Link to tracking Issue:**

#29448

**Testing:** 
go test for k8sobserver

**Preparation:**
``` Shell
DIR = opentelemetry-collector-contrib/extension/observer/k8sobserver
DATA_DIR = /path/to/otel/coverage/unit
CMD = go test -run TestExtensionObserve -v -count=1 -race -timeout 300s -parallel 4 --tags="" -cover ./... -covermode=atomic -args -test.gocoverdir=${DATA_DIR}
MULTI_CMD = for i in {1..50}; do echo "Run $i"; ./{CMD} &; done
```

**Tests:**
1. I found `config.ObservePods` is set true by default. Then
`TestExtensionObserveServices` and `TestExtensionObserveNodes` will
enable `k8sObserver.podListerWatcher` and try to access the same http
proxy since they are in the same process.
https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/39641dfac9482c62ece9505fef89fcff81ee258b/extension/observer/k8sobserver/factory.go#L26-L33


2. I add log(as shown below) for `TestExtensionObserve*` functions and
found test case **in the same process**. Refer to [stack
overflow](https://stackoverflow.com/a/24376644), go test are executed as
goroutines and executed concurrently.
<img width="783" alt="image"
src="https://github.com/open-telemetry/opentelemetry-collector-contrib/assets/3955787/a9baba19-0911-4d49-9395-8fb4a58a5e9b">

> --------- TestExtensionObserveServices process ID: 38910
> --------- TestExtensionObservePods process ID: 38910
> --------- TestExtensionObserveNodes process ID: 38910

3. I add log (as shown below) for http proxy setting addr, and found use
the same address:
<img width="951" alt="image"
src="https://github.com/open-telemetry/opentelemetry-collector-contrib/assets/3955787/a5c8e4e7-b6bd-4d8b-a82f-c9142d31a249">

> -------------- use proxy connectMethodForRequest t.Proxy addr:
0xc0007142f0 -------------------
> -------------- transport t.Proxy (is) nil
> -------------- send request: {Method:GET
URL:https://mock:12345/api/v1/pods?limit=500&resourceVersion=0
Proto:HTTP/1.1 ProtoMajor:1 ProtoMinor:1
Header:map[Accept:[application/json, */*]
User-Agent:[k8sobserver.test/v0.0.0 (darwin/amd64) kubernetes/$Format]]
Body:<nil> GetBody:<nil> ContentLength:0 TransferEncoding:[] Close:false
Host:mock:12345 Form:map[] PostForm:map[] MultipartForm:<nil>
Trailer:map[] RemoteAddr: RequestURI: TLS:<nil> Cancel:<nil>
Response:<nil> ctx:0xc000410120}

4. So we may find one goroutine is writing http proxy and the other
goroutine read with http proxy(HTTP request for **Pods Lister**). As a
result, this has led to data race.

**Inference:**
I found that `config.ObservePods` is set true by default. For
`TestExtensionObserveServices` and `TestExtensionObserveNodes` test
case, we just focus on services observer and nodes observer, so we don't
need to use `k8sObserver.podListerWatcher` which can avoid data race in
the same process.

**Documentation:**

---------

Signed-off-by: sakulali <sakulali@126.com>
…on retry (#29584)

**Description:** <Describe what has changed.>
Do not repeat marshal proto on retry.

**Link to tracking Issue:** <Issue number if applicable>
n/a
**Testing:** <Describe what testing was performed and which tests were
added.>
n/a
**Documentation:** <Describe the documentation added.>
n/a
**Description:** 
This PR brings the `Statements` API up-to-date with the
`ConditionSequence` API.

**Link to tracking Issue:** <Issue number if applicable>
Closes
#13545

**Testing:** 
Unit tests

**Documentation:** <Describe the documentation added.>
Filled in godoc comments.
Fixes #27845 

The failure mechanism was very subtle. The test waits for the contents
of the short file to be emitted before canceling the context which is
used in the file reader. However, the reader may require one additional
scan to determine that it's reached the end of the file, and therefore
in a tiny proportion of cases, the context cancelation could actually
trigger the reader to return before it had determined it should delete
the file. The solution involves validating file deletion before
canceling the context. I was able to reproduce failure locally only
about 1/3000 times, but with this change was able to run 30,000 times
without a failure. The PR also cleans up a few related parts of the
code.
…n Windows (#29532)

**Description:**
Last part to re-enable all Windows tests disabled due to issue #11451.

Notes:

* The default test timeout is not enough for the `cmd` GROUP on CI and
it was increased to 1200s just for this group on Windows, the default
stays at 300s (which was also enough for my local run).
* The `fileexporter` life cycle test indicates an issue that shouldn't
affect most usages of the collector: if the same configuration is used
to export multiple signals it opens one file handle for each signal but
just closes the one of them at shutdown (the helper only executes the
shutdown for the one signal). This is an issue for the test because
during cleanup the testing attempts to delete the test folder and there
are still handles open to the file that lives inside it. This shouldn't
be a problem for collector users. That said the exporter should close
all file handles on shutdown. I will take look at what can be done to
correct this behavior.
* Fixed a minor typo on the receivers test that affected many lines.

**Link to tracking Issue:**
Fix #28679 - last part.

**Testing:**
Local run and GH CI on my fork with `Run Windows` label.

**Documentation:**
N/A
…ner (#29606)

**Documentation:** <Describe the documentation added.>
Yet another change to code owners documentation, I found the `make
update-codeowners` target in the `Makefile` so I'm updating the doc to
point to this.

Also, moved extra context of what's actually happening in this command
to the workaround description, and added the `make generate` command to
the manual workaround.
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
renovate bot and others added 12 commits December 4, 2023 11:53
…ector-contrib/cmd/telemetrygen to v0.90.1 (#29591)

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

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[github.com/open-telemetry/opentelemetry-collector-contrib/cmd/telemetrygen](https://togithub.com/open-telemetry/opentelemetry-collector-contrib)
| require | minor | `v0.89.0` -> `v0.90.1` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector-contrib
(github.com/open-telemetry/opentelemetry-collector-contrib/cmd/telemetrygen)</summary>

###
[`v0.90.1`](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/blob/HEAD/CHANGELOG.md#v0901)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/compare/v0.90.0...v0.90.1)

##### 🧰 Bug fixes 🧰

- `exporters`: Upgrade core dependency to remove noisy "Exporting
finished" log message in exporters.
([#&#8203;29612](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29612))

###
[`v0.90.0`](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/blob/HEAD/CHANGELOG.md#v0900)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/compare/v0.89.0...v0.90.0)

##### 🛑 Breaking changes 🛑

- `dockerstatsreceiver`: Add \[container.cpu.limit],
\[container.cpu.shares] and \[container.restarts] metrics from docker
container api
([#&#8203;21087](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/21087))
    It requires API version 1.25 or greater.

##### 🚀 New components 🚀

- `failoverconnector`: New component that will allow for pipeline
failover triggered by the health of target downstream exporters
([#&#8203;20766](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/20766))
- `gitproviderreceiver`: add repo, branch, and contributor count metrics
([#&#8203;22028](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/22028))

##### 💡 Enhancements 💡

- `opensearchexporter`: Promote opensearchexporter to alpha.
([#&#8203;24668](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/24668))

- `awsemfexporter`: Improve NaN value checking for Summary metric types.
([#&#8203;28894](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/28894))

- `awsemfexporter`: Logs relating to the start and finish of processing
metrics have been reduced to debug level
([#&#8203;29337](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29337))

- `azuremonitorreceiver`: Support Azure gov cloud
([#&#8203;27573](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/27573))

- `clickhouseexporter`: Added support for more control over TTL
configuration. Currently, it supports timelines only in days, now also
in hours, minutes and seconds (propertyName ttl_days --> ttl).
([#&#8203;28675](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/28675))

- `datasetexporter`: Collect usage metrics with Otel and send grouped
attributes in session info.
([#&#8203;27650](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/27650),
[#&#8203;27652](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/27652))

- `resourcedetectionprocessor`: Add k8s cluster name detection when
running in EKS
([#&#8203;26794](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/26794))

- `pkg/ottl`: Add new IsDouble function to facilitate type checking.
([#&#8203;27895](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/27895))

- `configschema`: Generate metadata for connectors.
([#&#8203;26990](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/26990))

- `telemetrygen`: Exposes the span duration as a command line argument
`--span-duration`
([#&#8203;29116](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29116))

- `honeycombmarkerexporter`: Change honeycombmarkerexporter to alpha
([#&#8203;27666](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/27666))

- `mysqlreceiver`: expose tls in mysqlreceiver
([#&#8203;29269](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29269))
    If `tls` is not set, the default is to disable TLS connections.

- `processor/transform`: Convert between sum and gauge in metric context
when alpha feature gate
`processor.transform.ConvertBetweenSumAndGaugeMetricContext` enabled
([#&#8203;20773](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/20773))

- `receiver/mongodbatlasreceiver`: adds project config to mongodbatlas
metrics to filter by project name and clusters.
([#&#8203;28865](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/28865))

- `pkg/stanza`: Add "namedpipe" operator.
([#&#8203;27234](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/27234))

- `pkg/resourcetotelemetry`: Do not clone data in
pkg/resourcetotelemetry by default
([#&#8203;29327](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29327))
    -   The resulting consumer will be marked as `MutatesData` instead

- `pkg/stanza`: Improve performance by not calling decode when nop
encoding is defined
([#&#8203;28899](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/28899))

- `exporter/prometheusremotewrite`: prometheusremotewrite exporter add
option to send metadata
([#&#8203;13849](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/13849))

- `receivercreator`: Added support for discovery of endpoints based on
K8s services
([#&#8203;29022](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29022))
By discovering endpoints based on K8s services, a dynamic probing of K8s
service leveraging for example the httpcheckreceiver get enabled

- `signalfxexporter`: change default timeout to 10 seconds
([#&#8203;29436](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29436))

- `awss3exporter`: add support for `s3_force_path_style` and
`disable_ssl` parameters
([#&#8203;29331](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29331))
In order to support alternative object-storage, these parameters are
useful and help to leverage those systems not
compatible with domain style path, or just hosted without ssl (like just
deployed in a k8s namespace).

- `hostmetricsreceiver`: Add optional Linux-only metric
`system.linux.memory.available`
([#&#8203;7417](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/7417))
This is an alternative to `system.memory.usage` metric with
`state=free`.
Linux starting from 3.14 exports "available" memory. It takes "free"
memory as a baseline, and then factors in kernel-specific values.
    This is supposed to be more accurate than just "free" memory.
For reference, see the calculations
[here](https://superuser.com/a/980821).
See also `MemAvailable` in
[/proc/meminfo](https://man7.org/linux/man-pages/man5/proc.5.html).

- `azuremonitorexporter`: Updated Azure Monitor Exporter service version
from v2.0 to v2.1.
([#&#8203;29234](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29234))

##### 🧰 Bug fixes 🧰

- `cassandraexporter`: Exist check for keyspace and dynamic timeout
([#&#8203;27633](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/27633))
- `datadogreceiver`: Fix set telemetry.sdk.language=dotnet instead of
.NET
([#&#8203;29459](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29459))
- `filelogreceiver`: Fix issue where files were unnecessarily kept open
on Windows
([#&#8203;29149](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29149))
- `receiver/activedirectoryds`: Fix shutdown of
`activedirectorydsreceiver` when shutdown was called right after
creation, without a corresponding start call.
([#&#8203;29505](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29505))
- `honeycombmarkerexporter`: Fix default api_url and dataset_slug
([#&#8203;29309](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29309))
- `influxdbexporter`: When InfluxDB v1 compatibility is enabled AND
username\&password are set, the exporter panics. Not any more!
([#&#8203;27084](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/27084))
- `mongodbreceiver`: add `receiver.mongodb.removeDatabaseAttr` Alpha
feature gate to remove duplicate database name attribute
([#&#8203;24972](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/24972))
- `pkg/stanza`: Fix panic during stop for udp async mode only.
([#&#8203;29120](https://togithub.com/open-telemetry/opentelemetry-collector-contrib/issues/29120))

</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:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuODEuMyIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
It wasn't picked up because it has `.yml`, not `.yaml` extension.

@open-telemetry/collector-contrib-approvers do you think we should add
support for both `.yaml` and `.yml` files in `.chloggen`?
**Description:** 
This PR updates `pkg/otlp/logs` to bring in the changes from the
following PR:
DataDog/opentelemetry-mapping-go#207.

**Link to tracking Issue:** #26382

Fixes #26382
…le updates (#29636)

**Description:** Ignores updates to
`github.com/DataDog/datadog-agent/pkg/trace/exportable` since:
- It is a deprecated module for which we won't make any updates
- It is causing trouble with renovatebot

**Link to tracking Issue:** Relates to #29635, should unblock #29411
Fixing a typo in error log text.
…value (#29552)

**Description:**
Fixing a bug where Splunk rejects malformed HEC events with a nil event
field.

**Link to tracking Issue:**
Fixes #29551
[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/facebook/time](https://togithub.com/facebook/time) |
require | digest | `cb922d5` -> `4139d74` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### 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:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/aws/aws-sdk-go](https://togithub.com/aws/aws-sdk-go) |
require | patch | `v1.48.9` -> `v1.48.11` |
| [github.com/aws/aws-sdk-go-v2](https://togithub.com/aws/aws-sdk-go-v2)
| require | patch | `v1.23.4` -> `v1.23.5` |
|
[github.com/aws/aws-sdk-go-v2/config](https://togithub.com/aws/aws-sdk-go-v2)
| require | patch | `v1.25.10` -> `v1.25.11` |
|
[github.com/aws/aws-sdk-go-v2/credentials](https://togithub.com/aws/aws-sdk-go-v2)
| require | patch | `v1.16.8` -> `v1.16.9` |
|
[github.com/aws/aws-sdk-go-v2/service/kinesis](https://togithub.com/aws/aws-sdk-go-v2)
| require | patch | `v1.24.1` -> `v1.24.2` |
|
[github.com/aws/aws-sdk-go-v2/service/s3](https://togithub.com/aws/aws-sdk-go-v2)
| require | patch | `v1.47.1` -> `v1.47.2` |
|
[github.com/aws/aws-sdk-go-v2/service/sts](https://togithub.com/aws/aws-sdk-go-v2)
| require | patch | `v1.26.1` -> `v1.26.2` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>aws/aws-sdk-go (github.com/aws/aws-sdk-go)</summary>

###
[`v1.48.11`](https://togithub.com/aws/aws-sdk-go/blob/HEAD/CHANGELOG.md#Release-v14811-2023-12-01)

[Compare
Source](https://togithub.com/aws/aws-sdk-go/compare/v1.48.10...v1.48.11)

\===

##### Service Client Updates

-   `service/qconnect`: Updates service API and documentation
-   `service/rbin`: Updates service API and documentation
-   `service/verifiedpermissions`: Updates service API and documentation

###
[`v1.48.10`](https://togithub.com/aws/aws-sdk-go/blob/HEAD/CHANGELOG.md#Release-v14810-2023-11-30)

[Compare
Source](https://togithub.com/aws/aws-sdk-go/compare/v1.48.9...v1.48.10)

\===

##### Service Client Updates

- `service/arc-zonal-shift`: Updates service API, documentation, and
paginators
-   `service/glue`: Updates service API and documentation
- Adds observation and analyzer support to the GetDataQualityResult and
BatchGetDataQualityResult APIs.
-   `service/sagemaker`: Updates service API and documentation
- This release adds support for 1/ Code Editor, based on Code-OSS,
Visual Studio Code Open Source, a new fully managed IDE option in
SageMaker Studio 2/ JupyterLab, a new fully managed JupyterLab IDE
experience in SageMaker Studio

</details>

<details>
<summary>aws/aws-sdk-go-v2 (github.com/aws/aws-sdk-go-v2)</summary>

###
[`v1.23.5`](https://togithub.com/aws/aws-sdk-go-v2/compare/v1.23.4...v1.23.5)

[Compare
Source](https://togithub.com/aws/aws-sdk-go-v2/compare/v1.23.4...v1.23.5)

</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.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://togithub.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- 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:eyJjcmVhdGVkSW5WZXIiOiIzNy44MS4zIiwidXBkYXRlZEluVmVyIjoiMzcuODEuMyIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
This was reverted during the last release due to a bug that was fixed in
multimod

---------

Signed-off-by: Alex Boten <aboten@lightstep.com>
This PR moves localizes some test utility code. These functions are
either used only once, or only in one test file.

This is a step towards #29643, and ultimately towards sharing test
utilities across multiple packages.
Upgrade the redis library dependency to resolve security vulns in v7
@cparkins cparkins merged commit d30d17f into AmadeusITGroup:main Dec 4, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.