forked from open-telemetry/opentelemetry-collector-contrib
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Demo mitch #25
Open
acarbonetto
wants to merge
28
commits into
opensearchexporter-pub
Choose a base branch
from
demo_mitch
base: opensearchexporter-pub
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Demo mitch #25
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
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
# Conflicts: # cmd/otelcontribcol/go.sum
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
i.e. the correct way to update stability. Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Creates an opensearch bulk indexer for each trace record. Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
- Simple end to end test. - Consolidate opensearch-related code in sso_trace_exporter. - Separate marshalling to JSON from sending bulk requests. Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: Max Ksyunz <max.ksyunz@improving.com>
Signed-off-by: acarbonetto <andrewc@bitquilltech.com>
Signed-off-by: acarbonetto <andrewc@bitquilltech.com>
Signed-off-by: acarbonetto <andrewc@bitquilltech.com>
MaxKsyunz
force-pushed
the
opensearchexporter-pub
branch
5 times, most recently
from
September 21, 2023 20:51
7b2bd2c
to
1bd1a94
Compare
MaxKsyunz
force-pushed
the
opensearchexporter-pub
branch
5 times, most recently
from
November 10, 2023 02:39
d8d44cb
to
f82d250
Compare
MaxKsyunz
force-pushed
the
opensearchexporter-pub
branch
4 times, most recently
from
November 24, 2023 07:45
10f91e1
to
4d5cd14
Compare
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.
Description: Update to "resource"."process.command_args" in the OpenSearchExporter to pass a string instead of a string[].
This will fix a bug in the demo that was sending data that violated the ss4o schema.
Link to tracking Issue: N/A
Testing: To reproduce, build the opensearchexporter in the contrib-collector and run the
exporters/opensearchexporter/demo
.Documentation: N/A