Skip to content

First draft of observability requirements - #72

Merged
Vincent Biret (baywet) merged 11 commits into
masterfrom
dm/logging
Aug 1, 2024
Merged

First draft of observability requirements#72
Vincent Biret (baywet) merged 11 commits into
masterfrom
dm/logging

Conversation

@darrelmiller

@darrelmillerDarrel (darrelmiller) commented Jan 29, 2022•
edited by baywet
Loading

Copy link
Copy Markdown
Contributor

fixes#9
closes#20

Open
sebastienlevert
Sébastien Levert (sebastienlevert) reviewed Feb 21, 2022
View reviewed changes
Comment threadObservability.md Outdated
Comment threadObservability.md
Comment threadObservability.md Outdated
Comment threadObservability.md Outdated
Comment threadObservability.md
@maisarissiMaísa Rissi (maisarissi) mentioned this pull request May 6, 2022
Closed
@baywetVincent Biret (baywet) mentioned this pull request May 13, 2022
Closed
@baywet

Vincent Biret (baywet) commented May 13, 2022

Copy link
Copy Markdown
Member

calling out a piece of feedback provided on the java SDK giving more control over which status codes are logged/traced would also help make the log source usable, it should probably be part of the tags.

darrelmiller
Darrel (darrelmiller) commented Sep 9, 2022
View reviewed changes
Comment threadObservability.md Outdated
@baywetVincent Biret (baywet) mentioned this pull request Sep 9, 2022
Closed
@baywet@darrelmiller
- typo fix
95c56e7
Co-authored-by: Darrel <darrmi@microsoft.com>
@baywet
- adds much needed details about the spans and their attributes
9974f7e
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywetVincent Biret (baywet) linked an issue Apr 19, 2023 that may be closed by this pull request
Define requirements for Logging Handler #9
Closed
baywet
Vincent Biret (baywet) suggested changes Aug 1, 2024
View reviewed changes
Comment threadObservability.md Outdated
Comment threadObservability.md Outdated
Comment threadObservability.md Outdated
Comment threadObservability.md
Comment threadObservability.md Outdated
Comment threadObservability.md Outdated
Comment threadObservability.md Outdated
Comment threadObservability.md Outdated
Comment threadObservability.md
Comment threadObservability.md Outdated
@baywet
Vincent Biret (baywet) marked this pull request as ready for review August 1, 2024 15:07
@baywet
Vincent Biret (baywet) requested a review from a team as a code ownerAugust 1, 2024 15:07
@baywetVincent Biret (baywet) mentioned this pull request Aug 1, 2024
Closed
@baywet
feat: adds missing decompression handler
248414c
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
Merge branch 'master' into dm/logging
11eeb01
@baywet
doc: adds link to observability requirements
4bed259
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
docs: adds the logging handler only applies to PS
169bbd6
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
Vincent Biret (baywet) merged commit 5028b14 into masterAug 1, 2024
@baywet
Vincent Biret (baywet) deleted the dm/logging branch August 1, 2024 15:22
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@sebastienlevertSébastien Levert (sebastienlevert)sebastienlevert left review comments
@baywetVincent Biret (baywet)baywet approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Define requirements for Logging Handler

3 participants

@darrelmiller@baywet@sebastienlevert