1
0
mirror of https://github.com/open-telemetry/opentelemetry-go.git synced 2024-12-12 10:04:29 +02:00
Commit Graph

119 Commits

Author SHA1 Message Date
Tyler Yahn
2051927b9b
Correct CI workflow syntax (#1403)
Remove duplicate strategy that is causing an error.
2020-12-15 18:25:04 -05:00
Azfaar Qureshi
bdf87a7896
Migrate CircleCI ci.yml workflow to GitHub Actions (#1382)
* migrating CircleCI jobs to GitHub Actions

* using container matrix instead

* prevent entire workflow from stopping if one go version job fails

* updating github ci to use setup-go

* updating changelog
2020-12-14 19:40:15 -08:00
Krzesimir Nowak
0021ab0a3a
Rework proto generation (#1371)
* Rework proto generation

The changes here are:

- Fix the default goal (using "default" target is not doing it).

- Bail out with a useful message if proto submodule is not checked
  out.

- Replace the use of docker image with downloading the protoc binary
  and building the gogofast plugin ourselves. This gives us a greater
  control over the invocation of protoc.

- Use rsync to copy the generated code, instead of pax. Pax did not
  work for me (it was complaining about the unknown -0 flag).

The control over the protoc invocation will be useful later, when we
will want to generate a code with data structures in one place and the
collector code elsewhere. The collector code may or may not depend on
gRPC, but data structures have no need for it. This split will happen
when we move the gRPC code out of the OTLP exporter module into a
submodule.

Getting rid of docker has the upside that the generated files do not
belong to root, so there is no hassle of changing the ownership of the
files, and it is not requires to use sudo for the `clean` target. And
not using docker is faster.

The downside of this work is that it depends on more tools: rsync,
unzip and wget. I can only hope that macOS users have those tools too,
and that those tools are invoked the same.

* Update protogen workflow
2020-12-07 09:36:06 -08:00
Krzesimir Nowak
e0819780f9
Move opencensus example to the example directory (#1359)
And update .gitignore, so the built binary does not show up in git
status.
2020-11-20 13:34:40 -08:00
David Ashpole
4619c068e2
Add opencensus bridge example and readme (#1348)
* add opencensus bridge example

* Apply suggestions from code review

Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>

Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
2020-11-19 12:41:09 -08:00
David Ashpole
f6df5df938
add opencensus tracing bridge (#1305) 2020-11-12 10:51:52 -08:00
ET
c9726efd60
Allow up to 0.5% coverage change before failing build (#1162)
Don't bother to add a report unless there has been a coverage change.

Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
2020-09-10 12:49:08 -07:00
ET
34c02d1d24
Add Skip Changelog label to Dependabot-sourced PRs (#1161)
* Add "Skip Changelog" label to all dependabot PR requests

* Update CHANGELOG

* Include PR# in CHANGELOG

Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>
2020-09-10 12:24:04 -07:00
Anthony Mirabella
2e221fdf3e
Move tools package under internal (#1141)
It is not user-facing, so there is no reason to leave it in an
importable location. Fixes #1126.
2020-09-09 09:14:03 -07:00
Anthony Mirabella
1d2189088d
Add workflow to check for presence of CHANGELOG changes on PRs (#1067) 2020-09-08 20:50:05 -04:00
Tyler Yahn
3b3c263d9a
Update dependabot config (#1100)
Add check for github-actions.
Add missing examples and SDK go.mod
Remove redundant comments.
Change check to be weekly on Sunday to reduce load and churn.
Sort alphanumerically.
Add check to Makefile to ensure if there is a `go.mod` file there is a dependabot entry for that directory.
2020-08-26 16:34:41 -07:00
Bogdan Drutu
9907328c5b
Export coverage report to codecov (#995)
* Export coverage report to codecov

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

* Use orb and individual coverage files per module

* Run go cover inside module directory to account for local rewrites

Still try to upload using a single file, since orb docs seem to
indicate if you have more than one file, you're going to need to run
the orb multiple times.

* Use correct filename

* Don't do anything silly like appending a file to itself

* Handle subtle differences between FreeBSD and Linux

* Ignore opentelemetry-proto-gen files in exporters/otlp/internal

* Does codecov orb leave current directory in a bad state?

* Copy instead of move

* Ignore generated protobuf code in jaeger exporter module

Co-authored-by: Evan Torrie <evantorrie@users.noreply.github.com>
Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>
Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
2020-08-24 09:43:29 -07:00
Tyler Yahn
8f9f2d84cf
Move OTLP to the OTLP exporter (#1050)
* Move proto to OTLP exporter

* Update OTLP exporter import of proto

* Use gogo protobuf

To stop using the deprecated github.com/golang/protobuf and match what
the collector is doing, switch to generating OTLP with the
github.com/gogo/protobuf/proto instead of
github.com/golang/protobuf/proto.

* Clean dependencies

Remove all protobuf dependencies from otel package and all of its
dependencies.

* Update CHANGELOG

* Clean OTLP exporter go mod

Remove submodule beforehand to avoid unneeded direct dependencies.

* Use default ref for GitHub workflow

* Update path that triggers proto gen action

* Correct license-check exclusion for OTLP

* Update commented location of the OTLP and code
2020-08-10 19:55:52 -07:00
Tyler Yahn
799c178925
Make opentracing bridge into own Go module (#1038)
* Make opentracing bridge into own Go module

* Update dependabot config

* Clean dependencies of project

Now the bridge is a module, clean all upstream modules that no longer
implicitly depend on it.

* Update Changelog

* go mod tidy
2020-08-06 07:59:41 -07:00
Tyler Yahn
3b01a854d1
Make the stdout exporter a package (#963)
* Make the stdout exporter its own package

Follow the pattern of the other exporters.

* Update dependabot with stdout exporter

* Add replace directives for stdout exporter

* Remove outdated example test from metric SDK

* go mod tidy

* Update othttp example test

Remove unused stdout exporter.

* Remove tests in API that depend on stdout exporter

The global package does not need to be validated with the SDK. A more
properly constructed end-to-end integration test should be built if this
is actually needed.

* Add replace clause for otel in stdout go.mod
2020-07-24 20:44:51 -07:00
ET
dd79483e20
Create protobuf generation GitHub action (#938)
* Add protogen workflow

* Move out protobuf generation to Makefile.proto

* Include in Changelog

* Update Makefile.proto

Move file-local mode variable to line by itself.

Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>

Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
2020-07-16 10:07:59 -07:00
ET
2f59a27a28
Allow changes to go.sum *and* go.mod when running dependabot tidy-up (#871)
Transitive dependency changes sometimes do require a change to go.mod
files as well as go.sum. This is already live in the
opentelemetry-go-contrib repo, and this PR brings it here.
2020-06-24 13:35:14 -07:00
ET
f29cc497bf
Apply transitive dependabot go.mod dependency updates as part of automatic Github workflow (#844) 2020-06-22 12:34:11 -04:00
Liz Fong-Jones
a7a839bc55
Create dependabot.yml (#814)
Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>
2020-06-15 18:41:33 -04:00