-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[datasetexporter]: Extract serverHost from the logs and traces #24069
Closed
martin-majlis-s1
wants to merge
63
commits into
open-telemetry:main
from
scalyr:DSET-4002-datasetexporter-user-server-host
Closed
[datasetexporter]: Extract serverHost from the logs and traces #24069
martin-majlis-s1
wants to merge
63
commits into
open-telemetry:main
from
scalyr:DSET-4002-datasetexporter-user-server-host
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
…n_event configuration
observed timetamp in case LogRecord doesn't contain timestamp. Even though ObservedTimestamp should always be present, we fall back to current time in case it's not. In addition to that, remove duplicate and redundant "timestamp" attribute from the AddEvents event.
DSET-3998 feat: export Logs resource info based on export_resource_info_on_event configuration
…try-collector-contrib into event_no_timestamp_fix_use_occurence_time
…e_time [DSET-4071] [DSET-4007] Fix timestamp handling for LogRecord objects without a timestamp
This prefix is nor desired nor wanted.
severity to DataSet severity. Also remove "severity.text" and "severity.number" field from the event since it's redundant - we already have event severity (sev) field value.
information which is not already available (events which are sampled will already be ingested and visible in the ui so flags.is_sampled is redundant).
and update affected code and tests.
functions. Also move functionality for making a test record into a utility function.
[DSET-4037] [DSET-4014] [DSET-4034] [DSET-4034] [DSET-4098] Dataset log exporter improvements
…y-collector-contrib into dataset_log_exporter_improvements_fixes
github-actions
bot
added
exporter/sentry
exporter/signalfx
exporter/skywalking
exporter/splunkhec
exporter/sumologic
exporter/syslog
exporter/tanzuobservability
exporter/tencentcloudlogservice
exporter/zipkin
extension/asapauth
extension/awsproxy
extension/basicauth
extension/bearertokenauth
extension/encoding
extension/headerssetter
extension/healthcheck
Health Check Extension
extension/httpforwarder
extension/jaegerremotesampling
extension/oauth2clientauth
extension/observer
extension/oidcauth
extension/pprof
extension/sigv4auth
extension/storage
internal/aws
labels
Jul 12, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
cmd/configschema
configschema command
cmd/mdatagen
mdatagen command
cmd/opampsupervisor
cmd/otelcontribcol
otelcontribcol command
cmd/oteltestbedcol
cmd/telemetrygen
telemetrygen command
confmap/provider/s3provider
connector/count
connector/routing
connector/servicegraph
connector/spanmetrics
examples/demo
exporter/alibabacloudlogservice
Alibaba components
exporter/awscloudwatchlogs
awscloudwatchlogs exporter
exporter/awsemf
awsemf exporter
exporter/awskinesis
exporter/awss3
exporter/awsxray
exporter/azuredataexplorer
exporter/azuremonitor
exporter/carbon
exporter/cassandra
exporter/clickhouse
exporter/coralogix
exporter/datadog
Datadog components
exporter/dataset
exporter/dynatrace
exporter/elasticsearch
exporter/f5cloud
F5 exporter
exporter/file
exporter/googlecloud
exporter/googlecloudpubsub
exporter/googlemanagedprometheus
Google Managed Prometheus exporter
exporter/influxdb
exporter/instana
exporter/jaeger
exporter/jaegerthrifthttp
exporter/kafka
exporter/loadbalancing
exporter/logicmonitor
exporter/logzio
exporter/loki
Loki Exporter
exporter/mezmo
exporter/opencensus
exporter/parquet
exporter/prometheus
exporter/prometheusremotewrite
exporter/pulsar
exporter/sapm
exporter/sentry
exporter/signalfx
exporter/skywalking
exporter/splunkhec
exporter/sumologic
exporter/syslog
exporter/tanzuobservability
exporter/tencentcloudlogservice
exporter/zipkin
extension/asapauth
extension/awsproxy
extension/basicauth
extension/bearertokenauth
extension/encoding
extension/headerssetter
extension/healthcheck
Health Check Extension
extension/httpforwarder
extension/jaegerremotesampling
extension/oauth2clientauth
extension/observer
extension/oidcauth
extension/pprof
extension/sigv4auth
extension/storage
internal/aws
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:
Link to tracking Issue:
Testing:
Documentation:
This is currently blocked by #23881 to avoid merge conflicts.