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

fix(stanza/converter): handle syslogrecevier attributes correctly #27415

Conversation

sumo-drosiek
Copy link
Member

@sumo-drosiek sumo-drosiek commented Oct 3, 2023

Description:

Syslog receiver produces attribute (structured_data) which is a stringified map. This PR fixes it. See Issue for more details

Link to tracking Issue: #27414

Testing:

UT

Documentation:

In code comments

@sumo-drosiek sumo-drosiek marked this pull request as ready for review October 4, 2023 07:50
@sumo-drosiek sumo-drosiek requested a review from a team as a code owner October 4, 2023 07:50
Signed-off-by: Dominik Rosiek <[email protected]>
@djaglowski djaglowski merged commit 4eca940 into open-telemetry:main Oct 4, 2023
92 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 4, 2023
@sumo-drosiek sumo-drosiek deleted the drosiek-syslog-receiver-attributes branch October 4, 2023 19:24
jorgeancal pushed a commit to jorgeancal/opentelemetry-collector-contrib that referenced this pull request Oct 9, 2023
…en-telemetry#27415)

**Description:**

Syslog receiver produces attribute (`structured_data`) which is a
stringified map. This PR fixes it. See Issue for more details

**Link to tracking Issue:**
open-telemetry#27414

**Testing:**

UT

**Documentation:**

In code comments

---------

Signed-off-by: Dominik Rosiek <[email protected]>
jmsnll pushed a commit to jmsnll/opentelemetry-collector-contrib that referenced this pull request Nov 12, 2023
…en-telemetry#27415)

**Description:**

Syslog receiver produces attribute (`structured_data`) which is a
stringified map. This PR fixes it. See Issue for more details

**Link to tracking Issue:**
open-telemetry#27414

**Testing:**

UT

**Documentation:**

In code comments

---------

Signed-off-by: Dominik Rosiek <[email protected]>
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.

None yet

3 participants