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.

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

[sumologicexporter] Initialise Sumo Logic exporter #1565

Merged

Conversation

sumo-drosiek
Copy link
Member

Description:
Initialise Sumo Logic exporter

Link to tracking Issue: #1498

Testing:
Unit tests

Documentation:
Readme

@codecov
Copy link

codecov bot commented Nov 13, 2020

Codecov Report

Merging #1565 (3d85609) into master (5bfe9cd) will decrease coverage by 0.07%.
The diff coverage is 65.62%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1565      +/-   ##
==========================================
- Coverage   89.35%   89.28%   -0.08%     
==========================================
  Files         355      358       +3     
  Lines       17616    17680      +64     
==========================================
+ Hits        15741    15785      +44     
- Misses       1396     1416      +20     
  Partials      479      479              
Flag Coverage Δ
integration 70.93% <ø> (ø)
unit 87.94% <65.62%> (-0.08%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
exporter/sumologicexporter/exporter.go 50.00% <50.00%> (ø)
exporter/sumologicexporter/factory.go 82.14% <82.14%> (ø)
exporter/sumologicexporter/config.go 100.00% <100.00%> (ø)
receiver/carbonreceiver/transport/tcp_server.go 66.00% <0.00%> (-1.00%) ⬇️
receiver/prometheusexecreceiver/receiver.go 88.33% <0.00%> (+2.50%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5bfe9cd...3d85609. Read the comment docs.

Copy link
Member

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

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

I think you could already add yourself to the CODEOWNERS already. Same for the dependabot config.

exporter/sumologicexporter/README.md Outdated Show resolved Hide resolved
exporter/sumologicexporter/README.md Outdated Show resolved Hide resolved
exporter/sumologicexporter/README.md Outdated Show resolved Hide resolved
exporter/sumologicexporter/config.go Outdated Show resolved Hide resolved
exporter/sumologicexporter/exporter.go Outdated Show resolved Hide resolved
exporter/sumologicexporter/exporter.go Outdated Show resolved Hide resolved
exporter/sumologicexporter/factory.go Outdated Show resolved Hide resolved
exporter/sumologicexporter/factory.go Outdated Show resolved Hide resolved
exporter/sumologicexporter/go.mod Outdated Show resolved Hide resolved
exporter/sumologicexporter/go.mod Outdated Show resolved Hide resolved
@sumo-drosiek sumo-drosiek changed the title Initialise Sumo Logic exporter [sumologicexporter] Initialise Sumo Logic exporter Nov 17, 2020
@bogdandrutu bogdandrutu merged commit 39592d1 into open-telemetry:master Nov 20, 2020
@sumo-drosiek sumo-drosiek deleted the drosiek-sumologic-output-init branch November 23, 2020 10:35
ljmsc referenced this pull request in ljmsc/opentelemetry-collector-contrib Feb 21, 2022
Bumps [github.com/golangci/golangci-lint](https://github.com/golangci/golangci-lint) from 1.36.0 to 1.37.1.
- [Release notes](https://github.com/golangci/golangci-lint/releases)
- [Changelog](https://github.com/golangci/golangci-lint/blob/master/CHANGELOG.md)
- [Commits](golangci/golangci-lint@v1.36.0...v1.37.1)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants