Fix warning message for OTLP gRPC exporter mixin by ocelotl · Pull Request #2781 · open-telemetry/opentelemetry-python
Navigation Menu
{{ message }}
open-telemetry / opentelemetry-python Public
- Notifications You must be signed in to change notification settings
- Fork 834
Merged
Fix warning message for OTLP gRPC exporter mixin#2781
ocelotl merged 7 commits intoopen-telemetry:mainfrom
Fix warning message for OTLP gRPC exporter mixin#2781
ocelotl merged 7 commits intoopen-telemetry:mainfrom
Conversation
ocelotl
requested a review
from a team
ocelotl
self-assigned this
ocelotl
force-pushed
the
issue_2780
branch
3 times, most recently
from
a6c0d6c to
b09fe25
Compare
codeboten reviewed Jun 24, 2022
Copy link Copy Markdown
Contributor
codeboten
left a comment
codeboten
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd recommend adding a changelog with the fix as it's user facing error messages.
lzchen approved these changes Jun 24, 2022
srikanthccv suggested changes Jun 25, 2022
...pentelemetry-exporter-otlp-proto-grpc/src/opentelemetry/exporter/otlp/proto/grpc/exporter.py
Outdated
Show resolved
Hide resolved
...pentelemetry-exporter-otlp-proto-grpc/src/opentelemetry/exporter/otlp/proto/grpc/exporter.py Outdated Show resolved Hide resolved
ocelotl
requested review from
codeboten and
srikanthccv
ocelotl
added
the
Approve Public API check
label
Jun 27, 2022
ocelotl
force-pushed
the
issue_2780
branch
from
26bb469 to
9da6dca
Compare
srikanthccv reviewed Jun 30, 2022
...pentelemetry-exporter-otlp-proto-grpc/src/opentelemetry/exporter/otlp/proto/grpc/exporter.py
Outdated
Show resolved
Hide resolved
...pentelemetry-exporter-otlp-proto-grpc/src/opentelemetry/exporter/otlp/proto/grpc/exporter.py Outdated Show resolved Hide resolved
ocelotl
mentioned this pull request
ocelotl
force-pushed
the
issue_2780
branch
from
c28621e to
8fb1919
Compare
ocelotl
requested a review
from srikanthccv
srikanthccv approved these changes Jul 6, 2022
ocelotl added 6 commits
July 6, 2022 18:59
ocelotl
force-pushed
the
issue_2780
branch
from
7bb7c7f to
d2f806b
Compare
ocelotl
merged commit
fe26ada
into
open-telemetry:main
ocelotl
deleted the
issue_2780
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 participants