Skip to content

application logs: add ENVOY_TAGGED_CONN_LOG and ENVOY_TAGGED_STREAM_LOG#28117

Merged
yanavlasov merged 5 commits intoenvoyproxy:mainfrom
ohadvano:conn_tagged_log
Jun 26, 2023
Merged

application logs: add ENVOY_TAGGED_CONN_LOG and ENVOY_TAGGED_STREAM_LOG#28117
yanavlasov merged 5 commits intoenvoyproxy:mainfrom
ohadvano:conn_tagged_log

Conversation

@ohadvano
Copy link
Contributor

@ohadvano ohadvano commented Jun 23, 2023

Additional Description: These macros extend ENVOY_TAGGED_LOG (added in #27882) with a connection/stream and add the connection/stream ID to the tags, similar to ENVOY_CONN_LOG and ENVOY_STREAM_LOG
Risk Level: Low
Testing: Unit tests
Docs Changes: None
Release Notes: None
Platform Specific Features: None

ohadvano added 5 commits June 22, 2023 20:19
Signed-off-by: ohadvano <ohadvano@gmail.com>
Signed-off-by: ohadvano <ohadvano@gmail.com>
Signed-off-by: ohadvano <ohadvano@gmail.com>
Signed-off-by: ohadvano <ohadvano@gmail.com>
Signed-off-by: ohadvano <ohadvano@gmail.com>
@ohadvano
Copy link
Contributor Author

/assign-from @envoyproxy/senior-maintainers

@repokitteh-read-only
Copy link

@envoyproxy/senior-maintainers assignee is @yanavlasov

🐱

Caused by: a #28117 (comment) was created by @ohadvano.

see: more, trace.

@ohadvano
Copy link
Contributor Author

ping @yanavlasov

@yanavlasov yanavlasov merged commit 8a3286e into envoyproxy:main Jun 26, 2023
@ohadvano ohadvano deleted the conn_tagged_log branch June 26, 2023 18:21
asheryerm pushed a commit to asheryerm/envoy that referenced this pull request Jul 5, 2023
…OG (envoyproxy#28117)

Signed-off-by: ohadvano <ohadvano@gmail.com>
Signed-off-by: asheryer <asheryer@amazon.com>
reskin89 pushed a commit to reskin89/envoy that referenced this pull request Jul 11, 2023
…OG (envoyproxy#28117)

Signed-off-by: ohadvano <ohadvano@gmail.com>
Signed-off-by: Ryan Eskin <ryan.eskin89@protonmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants