Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[chore] add Sumo Logic distribution #22073

Merged
merged 2 commits into from
May 18, 2023
Merged

[chore] add Sumo Logic distribution #22073

merged 2 commits into from
May 18, 2023

Conversation

andrzej-stencel
Copy link
Member

Description:

Adds Sumo Logic distribution to component docs.

Link to tracking Issue:

mx-psi
mx-psi approved these changes May 18, 2023
View reviewed changes
Copy link
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, would be good to have someone from Sumo give their approval here :)

swiatekm
swiatekm approved these changes May 18, 2023
View reviewed changes
Copy link
Contributor

@swiatekm swiatekm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found one receiver that shouldn't be there, other than that LGTM.

receiver/azuremonitorreceiver/README.md Outdated Show resolved Hide resolved
perk-sumo
perk-sumo approved these changes May 18, 2023
View reviewed changes
Copy link

@perk-sumo perk-sumo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending one comment from @swiatekm-sumo.
Thank you @astencel-sumo! 🚀

@dmitryax dmitryax merged commit cd82096 into open-telemetry:main May 18, 2023
76 of 77 checks passed
@github-actions github-actions bot added this to the next release milestone May 18, 2023
atoulme
atoulme reviewed May 18, 2023
View reviewed changes
receiver/azuremonitorreceiver/metadata.yaml
type: string
azuremonitor.subscription_id:
description: Azure subscription ID
description: Azure subscription ID
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these are whitespace changes only, maybe a separate PR for those?

@andrzej-stencel andrzej-stencel deleted the add-sumo-distro branch May 18, 2023 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atoulme atoulme atoulme left review comments

@perk-sumo perk-sumo perk-sumo approved these changes

@codeboten codeboten codeboten approved these changes

@mx-psi mx-psi mx-psi approved these changes

@dmitryax dmitryax dmitryax approved these changes

@swiatekm swiatekm swiatekm approved these changes

@djaglowski djaglowski Awaiting requested review from djaglowski djaglowski is a code owner

@dashpole dashpole Awaiting requested review from dashpole dashpole is a code owner

@jmacd jmacd Awaiting requested review from jmacd jmacd is a code owner

@fatsheep9146 fatsheep9146 Awaiting requested review from fatsheep9146 fatsheep9146 is a code owner

@jpkrohling jpkrohling Awaiting requested review from jpkrohling jpkrohling is a code owner

@Aneurysm9 Aneurysm9 Awaiting requested review from Aneurysm9

@kovrus kovrus Awaiting requested review from kovrus

@MovieStoreGuy MovieStoreGuy Awaiting requested review from MovieStoreGuy MovieStoreGuy is a code owner

@TylerHelmuth TylerHelmuth Awaiting requested review from TylerHelmuth TylerHelmuth is a code owner

@bogdandrutu bogdandrutu Awaiting requested review from bogdandrutu bogdandrutu is a code owner

@evan-bradley evan-bradley Awaiting requested review from evan-bradley evan-bradley is a code owner

Assignees

@fatsheep9146 fatsheep9146

Labels
cmd/mdatagen mdatagen command connector/count connector/servicegraph connector/spanmetrics exporter/carbon exporter/f5cloud F5 exporter exporter/file exporter/kafka exporter/loadbalancing exporter/prometheus extension/asapauth extension/awsproxy extension/basicauth extension/bearertokenauth extension/headerssetter extension/healthcheck Health Check Extension extension/httpforwarder extension/jaegerremotesampling extension/oauth2clientauth extension/observer extension/oidcauth extension/pprof extension/sigv4auth extension/storage processor/attributes Attributes processor processor/cumulativetodelta Cumulative To Delta processor processor/deltatorate Delta To Rate processor processor/filter Filter processor processor/groupbyattrs Group By Attributes processor processor/groupbytrace Group By Trace processor processor/k8sattributes k8s Attributes processor processor/logstransform Logs Transform processor processor/metricsgeneration Metrics Generation processor processor/metricstransform Metrics Transform processor processor/probabilisticsampler Probabilistic Sampler processor processor/redaction Redaction processor processor/resource Resource processor processor/resourcedetection Resource detection processor processor/routing Routing processor processor/schema Schema processor processor/servicegraph Service graph processor processor/span processor/spanmetrics Span Metrics processor processor/tailsampling Tail sampling processor processor/transform Transform processor receiver/activedirectoryds receiver/aerospike receiver/apache receiver/awscloudwatch receiver/awscontainerinsight receiver/awsecscontainermetrics receiver/awsfirehose receiver/awsxray receiver/azureeventhub receiver/azuremonitor receiver/bigip receiver/carbon receiver/cloudflare
Projects
None yet
Milestone
v0.78.0
Development

Successfully merging this pull request may close these issues.

None yet
8 participants
@andrzej-stencel @atoulme @codeboten @mx-psi @dmitryax @perk-sumo @swiatekm @fatsheep9146