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

Yang: Add Prometheus Remote Write/Cortex Exporter #9

Closed
wants to merge 24 commits into from
Closed

Yang: Add Prometheus Remote Write/Cortex Exporter #9

wants to merge 24 commits into from

Conversation

huyan0
Copy link

@huyan0 huyan0 commented Aug 7, 2020

This shows the branch I want to merge to upstream repository.

@huyan0 huyan0 force-pushed the prw_exporter branch 2 times, most recently from df5bcc3 to f0ab533 Compare August 12, 2020 00:29
@huyan0 huyan0 force-pushed the prw_exporter branch 2 times, most recently from 3ce4d79 to 2fb133f Compare August 27, 2020 01:31
@huyan0
add conversion from Int64 and Double OTLP metrics for Promehteus Remo…
03faf67 
…te Write Exporter (open-telemetry#1577)

add prometheus remote write exporter

address lint issues

improve test coverage

fix lint error

add prometheus remote write exporter to default components

change metric name label

add conversion from ns to ms

rename tests

add attribute to label functionality in exporter.go

format code

resolve conflicts with master

switch to fmt.Errorf

add conversion from Int64 and Double OTLP metrics

add prometheus remote write exporter

address lint issues

improve test coverage

fix lint error

add prometheus remote write exporter to default components

change metric name label

add conversion from ns to ms

rename tests

add attribute to label functionality in exporter.go

format code

resolve conflicts with master

add check for nil

change data format to dataold
@huyan0 huyan0 force-pushed the prw_exporter branch 2 times, most recently from 7b952a7 to 73f05c5 Compare August 28, 2020 17:34
bogdandrutu and others added 23 commits August 28, 2020 12:43
@bogdandrutu
Rename some internal funcs to allow the oc to new structs to reuse th…
88634ae 
…e names (open-telemetry#1664)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Add testdata for the new metrics structs (open-telemetry#1665)
02bf8dc 
Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Tune testdata, and update testdataold to use same names (open-telemet…
1f7fddc 
…ry#1667)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Add OC to new metrics structure conversion. (open-telemetry#1666)
fadd7d5 
Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Remove dependency on the old errors (open-telemetry#1668)
4649b3b 
Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Fix flaky test TestOCToMetrics_ResourceInMetric (open-telemetry#1670)
5257ff0 
Use only one resource label to avoid ordering issues in the resource labels list.

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Add pdatautil support for the new internal metrics structs (open-tele…
c157903 
…metry#1669)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Rename getters for oneof and metricdata types (open-telemetry#1675)
f9166ee 
Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Fix typo Mame -> Name (open-telemetry#1676)
7c315f9 
Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Add initial draft of the oneof generator, fixes issue in contrib (ope…
52c9e7f 
…n-telemetry#1677)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Change resource processor to use the new metrics internal structs (op…
c9135e0 
…en-telemetry#1685)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Change queued processor/exporterhelper tests to use the new metrics i…
f36fd70 
…nternal structs (open-telemetry#1687)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Change opencensus recv/exp to use the new metrics internal structs (o…
6e05a9a 
…pen-telemetry#1686)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Change processor tests to use the new metrics internal structs (open-…
6a0d262 
…telemetry#1680)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Fix bug in copy data, the data were not deeply copied (open-telemetry…
a1ec7a8 
…#1683)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Add SetDataType and remove Set for every type (open-telemetry#1684)
550df0e 
Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Change sink exporter to use the new metrics internal structs (open-te…
474f475 
…lemetry#1678)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Change logging exporter to use the new metrics internal structs (open…
c3f7dd5 
…-telemetry#1679)

Signed-off-by: Bogdan Drutu <[email protected]>
@dependabot
Bump github.com/google/uuid from 1.1.1 to 1.1.2 (open-telemetry#1690)
0e1fe61 
Bumps [github.com/google/uuid](https://github.com/google/uuid) from 1.1.1 to 1.1.2.
- [Release notes](https://github.com/google/uuid/releases)
- [Commits](google/uuid@v1.1.1...v1.1.2)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump github.com/google/go-cmp from 0.5.1 to 0.5.2 (open-telemetry#1688)
e65a63f 
Bumps [github.com/google/go-cmp](https://github.com/google/go-cmp) from 0.5.1 to 0.5.2.
- [Release notes](https://github.com/google/go-cmp/releases)
- [Commits](google/go-cmp@v0.5.1...v0.5.2)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@bogdandrutu
Change host metrics receiver to use the new metrics internal structs (o…
fc77725 
…pen-telemetry#1682)

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu
Change tests to use the new internal metrics structs (open-telemetry#…
09d5ee7 
…1695)

Signed-off-by: Bogdan Drutu <[email protected]>
@huyan0
implement histogram conversion
ebb3f11
@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Sep 16, 2020
@github-actions
Copy link

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot closed this Sep 23, 2020
JamesJHPark pushed a commit that referenced this pull request Nov 3, 2021
@jpkrohling
Bump to OpenTelemetry Collector 0.20.0 (#10)
1da8f73 
Closes #9

Signed-off-by: Juraci Paixão Kröhling <[email protected]>
JamesJHPark pushed a commit that referenced this pull request Nov 3, 2021
@jpkrohling @bogdandrutu
@joe-elliott @jiwen624 @Ashmita152 @fangyi-zhou @crearys @pmalek-sumo @secat @sodabrew @pmalek @KKelvinLo @himanshu007-creator @u5surf @zchee @Aneurysm9 @loomis-relativity
Move opentelemetry-collector-builder to core (open-telemetry#4286)
4063a46 
* Initial commit

* Add CODEOWNERS file (#2)

* Add CODEOWNERS file

* Update CODEOWNERS

* Moved from github.com/observatorium/opentelemetry-collector-builder (#3)

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* fixed panics (#6)

Signed-off-by: Joe Elliott <[email protected]>

* Replace master with main in CI and mergify files (#8)

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Bump to OpenTelemetry Collector 0.20.0 (#10)

Closes #9

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Explicitly enable Go modules in quickstart instructions (#13)

* Update to collector v0.21.0 (#17)

Fixes #16

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update to collector v0.22.0 (#19)

* Download go modules before building (#20)

Fixes #14

* Add version command (#25)

Signed-off-by: Ashmita Bohara <[email protected]>

* Pass errors from cobra Execute back to main for correct exit code (#28)

* pass errors from cobra execute back to main

* print the error

* Update to collector v0.23.0 (#27)

* Generate a warning if the builder and collector base version mismatch (#30)

* Generate a warning if the builder and collector base version mismatch

* Show current default version in the warning message

* Update to OpenTelemetry Collector 0.24.0

* Don't use %w formatting with log.Fatal (#35)

* Update to OpenTelemetry Collector 0.25.0 (#36)

Signed-off-by: Serge Catudal <[email protected]>

* Update to 0.26.0 and update BuildInfo (#39)

* Sync build and CI Go versions at latest 1.16 (#34)

* Sync build and CI Go versions at latest 1.16

* Run go mod tidy

* Set go binary to use in the compilation phase in tests

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

Co-authored-by: Juraci Paixão Kröhling <[email protected]>

* Add option to generate go code only (no compile) (#40)

* Issue#24 Add option to generate go code only (no compile)

* Update cmd/root.go logging

Suggested by @jpkkrohling

Co-authored-by: Juraci Paixão Kröhling <[email protected]>

* remove verbose help .. created by corba

* suggestion by jpkrohling to keep generateandcompile

* lint error: remove unused var

* reword cmd option and add back help message for default

Co-authored-by: Juraci Paixão Kröhling <[email protected]>

* Don't reuse exec.Cmd (#42)

* Update to OpenTelemetry Collector 0.27.0 (#43)

* Add CI Badge (#47)

* Update to Collector v0.28.0 (#49)

* Update to Collector v0.28.0

Closes #48

Addresses the breaking API change in
open-telemetry#3163,
besides the usual version number changes.

Signed-off-by: Fangyi Zhou <[email protected]>

* Use `go mod tidy` instead of `go mod download`

It appears that this magically resolves the go.mod file issue.
https://stackoverflow.com/questions/67203641/missing-go-sum-entry-for-module-providing-package-package-name

Signed-off-by: Fangyi Zhou <[email protected]>

* Account for go mod download in go1.17 not updating go.sum (#50)

* Update to collector v0.29.0 (#54)

* Update replaces.builder.yaml

* Update nocore.builder.yaml

* Update config.go

* Update README.md

* Update main.go

* Update to collector v0.30.0 (#57)

* cmd: fix module flag default value to github.com/open-telemetry (#58)

Signed-off-by: Koichi Shiraishi <[email protected]>

* Update to collector v0.31.0 (#60)

* Update to v0.33.0 (#62)

Signed-off-by: Anthony J Mirabella <[email protected]>

* Add excludes support to generated go.mod (#63)

Signed-off-by: Anthony J Mirabella <[email protected]>

Co-authored-by: Juraci Paixão Kröhling <[email protected]>

* Small cleanup for the builder files (#64)

Signed-off-by: Bogdan Drutu <[email protected]>

* Support building with Go 1.17 (#66)

* Support building with Go 1.17
Fixes #65

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update workflows to use Go 1.17

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Add gosec exceptions for exec.Command

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Update to OpenTelemetry core 0.34.0 (#68)

Fixes #67

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

* Upgrade to OpenTelemetry Collector 0.35.0 (#70)

Signed-off-by: Fangyi Zhou <[email protected]>

* Upgrade to OpenTelemetry Collector 0.36.0 (#76)

* Generate custom service code for Windows (#75)

* update main to include windows service code

* use main version from tag 0.35.0

* update main function

* align with upstream v0.36.0 tag

* dummy change to trigger build

* Revert "dummy change to trigger build"

This reverts commit 629d499461da2d2c240bf1e495b5fe0558e3547f.

* Remove Core from Module type (#77)

Fixes #15

Signed-off-by: yugo-horie <[email protected]>

* release 0.37.0 (#78)

* release 0.37.0

* update use of NewCommand

* Move builder to subdirectory

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

Co-authored-by: Bogdan Drutu <[email protected]>
Co-authored-by: Bogdan Drutu <[email protected]>
Co-authored-by: Joe Elliott <[email protected]>
Co-authored-by: Eric Yang <[email protected]>
Co-authored-by: Brian Gibbins <[email protected]>
Co-authored-by: Ashmita <[email protected]>
Co-authored-by: Fangyi Zhou <[email protected]>
Co-authored-by: Shaun Creary <[email protected]>
Co-authored-by: Patryk Małek <[email protected]>
Co-authored-by: Serge Catudal <[email protected]>
Co-authored-by: Aaron Stone <[email protected]>
Co-authored-by: Patryk Małek <[email protected]>
Co-authored-by: Aaron Stone <[email protected]>
Co-authored-by: Kelvin Lo <[email protected]>
Co-authored-by: Himanshu <[email protected]>
Co-authored-by: Y.Horie <[email protected]>
Co-authored-by: Koichi Shiraishi <[email protected]>
Co-authored-by: Anthony Mirabella <[email protected]>
Co-authored-by: Cal Loomis <[email protected]>
Co-authored-by: alrex <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@huyan0 @bogdandrutu