-
Notifications
You must be signed in to change notification settings - Fork 199
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
Make preagg standard metrics work with ikey and role name overrides #2472
Merged
Conversation
This file contains 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
trask
force-pushed
the
metrics-ikey-overrides
branch
3 times, most recently
from
August 28, 2022 23:09
3b5a80c
to
7d44e8b
Compare
trask
changed the title
Make preagg standard metrics work with ikey overrides
Make preagg standard metrics work with ikey and role name overrides
Aug 29, 2022
trask
force-pushed
the
connection-string-overrides
branch
from
August 29, 2022 00:34
7dc43a8
to
0f7c497
Compare
trask
force-pushed
the
metrics-ikey-overrides
branch
2 times, most recently
from
August 29, 2022 00:54
a11c123
to
b5e515c
Compare
trask
force-pushed
the
metrics-ikey-overrides
branch
from
August 29, 2022 18:04
3d461d3
to
abdbba4
Compare
trask
force-pushed
the
connection-string-overrides
branch
from
August 29, 2022 20:30
25afac6
to
a1767c3
Compare
heyams
reviewed
Aug 29, 2022
...c/main/java/io/opentelemetry/instrumentation/api/instrumenter/http/TemporaryMetricsView.java
Show resolved
Hide resolved
.../src/main/java/com/azure/monitor/opentelemetry/exporter/implementation/MetricDataMapper.java
Show resolved
Hide resolved
.../src/main/java/com/azure/monitor/opentelemetry/exporter/implementation/MetricDataMapper.java
Outdated
Show resolved
Hide resolved
trask
force-pushed
the
metrics-ikey-overrides
branch
from
August 29, 2022 20:44
abdbba4
to
95d73ea
Compare
heyams
reviewed
Aug 30, 2022
...c/main/java/io/opentelemetry/instrumentation/api/instrumenter/http/TemporaryMetricsView.java
Show resolved
Hide resolved
...c/main/java/io/opentelemetry/instrumentation/api/instrumenter/http/TemporaryMetricsView.java
Show resolved
Hide resolved
...ng/src/main/java/com/microsoft/applicationinsights/agent/internal/init/SecondEntryPoint.java
Show resolved
Hide resolved
.../src/main/java/com/azure/monitor/opentelemetry/exporter/implementation/MetricDataMapper.java
Show resolved
Hide resolved
trask
force-pushed
the
metrics-ikey-overrides
branch
from
August 30, 2022 04:42
1abe831
to
8e33b47
Compare
trask
force-pushed
the
metrics-ikey-overrides
branch
from
August 30, 2022 16:16
72e807f
to
cc16190
Compare
heyams
approved these changes
Aug 30, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Built on top of #2471