Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

feat: add default exporters #43

Merged
merged 7 commits into from
Sep 27, 2023

Conversation

shanduur
Copy link
Contributor

What this PR does / why we need it:

Explained in #34

Which issue(s) this PR resolves:

Resolves #34

Special notes for your reviewer:

N/A

Additional documentation e.g., enhancement proposals, usage docs, etc.:

Provided via README.

@shanduur shanduur added area/monitoring kind/feature Categorizes issue or PR as related to a new feature. area/api Indicates an issue on COSI api area. labels Sep 21, 2023
@shanduur shanduur self-assigned this Sep 21, 2023
Signed-off-by: Mateusz Urbanek <mateusz.urbanek.98@gmail.com>
Signed-off-by: Mateusz Urbanek <mateusz.urbanek.98@gmail.com>
Signed-off-by: Mateusz Urbanek <mateusz.urbanek.98@gmail.com>
@shanduur shanduur force-pushed the add-default-exporters branch from 839b348 to c26ceb8 Compare September 21, 2023 10:37
@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Merging #43 (bfac00f) into main (4cc2a19) will increase coverage by 8.26%.
The diff coverage is 67.61%.

@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   40.69%   48.95%   +8.26%     
==========================================
  Files           3        3              
  Lines         231      335     +104     
==========================================
+ Hits           94      164      +70     
- Misses        127      151      +24     
- Partials       10       20      +10     
Files Coverage Δ
gocosi.go 73.68% <100.00%> (+0.95%) ⬆️
options.go 45.06% <66.66%> (+36.72%) ⬆️

@shanduur shanduur removed the area/api Indicates an issue on COSI api area. label Sep 21, 2023
Signed-off-by: Mateusz Urbanek <mateusz.urbanek.98@gmail.com>
@shanduur shanduur force-pushed the add-default-exporters branch from 5f36031 to 68c1c4e Compare September 21, 2023 11:19
Signed-off-by: Mateusz Urbanek <mateusz.urbanek.98@gmail.com>
Signed-off-by: Mateusz Urbanek <mateusz.urbanek.98@gmail.com>
@shanduur shanduur force-pushed the add-default-exporters branch from 9b8ad04 to b00b81f Compare September 27, 2023 16:39
Signed-off-by: Mateusz Urbanek <mateusz.urbanek.98@gmail.com>
@shanduur shanduur force-pushed the add-default-exporters branch from b00b81f to bfac00f Compare September 27, 2023 16:47
@shanduur shanduur merged commit 5506387 into doomshrine:main Sep 27, 2023
@shanduur shanduur deleted the add-default-exporters branch September 27, 2023 16:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/monitoring kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE]: Add default OTLP exporter configuration for Metrics and Tracing
1 participant