tyk icon indicating copy to clipboard operation
tyk copied to clipboard

[WIP] OTel tracing PoC

Open tbuchaillot opened this issue 2 years ago • 4 comments

Description

Related Issue

Motivation and Context

How This Has Been Tested

Screenshots (if appropriate)

Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] Refactoring or add test (improvements in base code or adds test coverage to functionality)

Checklist

  • [ ] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). If pulling from your own fork, don't request your master!
  • [ ] Make sure you are making a pull request against the master branch (left side). Also, you should start your branch off our latest master.
  • [ ] My change requires a change to the documentation.
    • [ ] If you've changed APIs, describe what needs to be updated in the documentation.
    • [ ] If new config option added, ensure that it can be set via ENV variable
  • [ ] I have updated the documentation accordingly.
  • [ ] Modules and vendor dependencies have been updated; run go mod tidy && go mod vendor
  • [ ] When updating library version must provide reason/explanation for this update.
  • [ ] I have added tests to cover my changes.
  • [ ] All new and existing tests passed.
  • [ ] Check your code additions will not fail linting checks:
    • [ ] go fmt -s
    • [ ] go vet

tbuchaillot avatar Sep 12 '22 15:09 tbuchaillot

:boom: CI tests failed :see_no_evil:

CI test log

all ok

gofmt

trace/handler.go trace/manager.go trace/otlp/otel.go trace/trace.go

goimports

trace/handler.go trace/manager.go trace/otlp/otel.go trace/trace.go

gogenerate

all ok

If the above are ok, please look at the run or in the Checks tab.

github-actions[bot] avatar Sep 12 '22 15:09 github-actions[bot]

API tests result: success :white_check_mark: Branch used: refs/pull/4295/merge Commit:
Triggered by: pull_request (@tbuchaillot) Execution page

buger avatar Sep 12 '22 15:09 buger

:boom: CI tests failed :see_no_evil:

CI test log

all ok

gofmt

gateway/middleware.go gateway/mw_version_check.go gateway/reverse_proxy.go trace/handler.go trace/manager.go trace/otlp/otel.go trace/trace.go

goimports

gateway/middleware.go gateway/mw_version_check.go gateway/reverse_proxy.go trace/handler.go trace/manager.go trace/otlp/otel.go trace/trace.go

gogenerate

all ok

If the above are ok, please look at the run or in the Checks tab.

github-actions[bot] avatar Sep 12 '22 17:09 github-actions[bot]

API tests result: success :white_check_mark: Branch used: refs/pull/4295/merge Commit: 19cb832d30b652fd5b05d8bc82a290dc6e18c1f6 Triggered by: pull_request (@tbuchaillot) Execution page

buger avatar Sep 12 '22 18:09 buger

:boom: CI tests failed :see_no_evil:

CI test log

all ok

gofmt

gateway/middleware.go gateway/mw_version_check.go gateway/reverse_proxy.go trace/handler.go trace/manager.go trace/otlp/otel.go trace/trace.go

goimports

gateway/analytics_test.go gateway/middleware.go gateway/mw_version_check.go gateway/reverse_proxy.go trace/handler.go trace/manager.go trace/otlp/otel.go trace/trace.go

gogenerate

all ok

If the above are ok, please look at the run or in the Checks tab.

github-actions[bot] avatar Oct 07 '22 10:10 github-actions[bot]

API tests result: success :white_check_mark: Branch used: refs/pull/4295/merge Commit: f8560189bda813e309fdd261d94f15fc534ccad6 Triggered by: pull_request (@tbuchaillot) Execution page

Tyk-ITS avatar Oct 07 '22 10:10 Tyk-ITS