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

Fixes the remaining tests. #684

Closed
wants to merge 40 commits into from

Commits on Feb 7, 2022

  1. Configuration menu
    Copy the full SHA
    2e75522 View commit details
    Browse the repository at this point in the history
  2. Update changelog

    MadVikingGod committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    09f0446 View commit details
    Browse the repository at this point in the history
  3. fix pull_test

    MadVikingGod committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    20d9758 View commit details
    Browse the repository at this point in the history
  4. Replace recordingSpan attributes implementation with slice of attribu…

    …tes (open-telemetry#2576)
    
    * Replace recordingSpan attributes implementation
    
    Instead of an LRU strategy for cap-ing span attributes, comply with the
    specification and drop last added. To do this, the attributesmap is
    replaced with a slice of attributes.
    
    * Remove attributesmap files
    
    * Refine addition algorithm
    
    Unify duplicated code.
    
    Fix deduplication algorithm.
    
    Fix droppedAttributes to always be returned, even if the span has no
    attributes.
    
    * Unify span SetAttributes tests
    
    * Doc fix to attr drop order in changelog
    
    * Test span and snapshot attrs
    
    * fix lint
    
    * Add tests for recordingSpan method defaults
    
    * Comment why pre-allocation is not done
    
    * Correct grammar in recordingSpan allocation comment
    
    * Update sdk/trace/tracer.go
    
    Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>
    
    Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>
    MrAlias and Aneurysm9 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    98bb105 View commit details
    Browse the repository at this point in the history

Commits on Feb 8, 2022

  1. Configuration menu
    Copy the full SHA
    10b58d6 View commit details
    Browse the repository at this point in the history

Commits on Feb 9, 2022

  1. Export resource attributes from zipkin exporter (open-telemetry#2589)

    * Export resource attributes from zipkin exporter
    
    * Update CHANGELOG.md
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Refactoring
    
    * Refactoring
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    tttanikawa and MrAlias committed Feb 9, 2022
    Configuration menu
    Copy the full SHA
    9f42a81 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    010ca4f View commit details
    Browse the repository at this point in the history

Commits on Feb 11, 2022

  1. refactor: use Key's Defined method (open-telemetry#2593)

    * optimize(attribute): use Key's Defined method
    
    * Update kv.go
    1046102779 committed Feb 11, 2022
    Configuration menu
    Copy the full SHA
    b01dc21 View commit details
    Browse the repository at this point in the history
  2. Release v1.4.0/v0.27.0 (open-telemetry#2600)

    * Update versions.yaml
    
    * Update changelog
    
    * Upgrade stable-v1 to version v1.4.0
    
    * Upgrade experimental-metrics to version v0.27.0
    
    * Upgrade bridge to version v0.27.0
    
    * Update CHANGELOG.md
    MrAlias committed Feb 11, 2022
    Configuration menu
    Copy the full SHA
    1bda062 View commit details
    Browse the repository at this point in the history

Commits on Feb 14, 2022

  1. Lock down golangci-lint config (open-telemetry#2602)

    Disable all default linters prior to enabling the ones we want to ensure
    that no upgrade that include new default linters introduce changes to
    the CI system.
    MrAlias committed Feb 14, 2022
    Configuration menu
    Copy the full SHA
    25827f0 View commit details
    Browse the repository at this point in the history
  2. Bump actions/setup-go from 2.1.5 to 2.2.0 (open-telemetry#2603)

    Bumps [actions/setup-go](https://github.com/actions/setup-go) from 2.1.5 to 2.2.0.
    - [Release notes](https://github.com/actions/setup-go/releases)
    - [Commits](actions/setup-go@v2.1.5...v2.2.0)
    
    ---
    updated-dependencies:
    - dependency-name: actions/setup-go
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    dependabot[bot] committed Feb 14, 2022
    Configuration menu
    Copy the full SHA
    f5c4874 View commit details
    Browse the repository at this point in the history

Commits on Feb 15, 2022

  1. Refactor development/CI tooling (open-telemetry#2609)

    * Refactor Makefile
    
    * Update dependabot targets
    
    * Sync github actions config with Makefile ci target
    
    * Update test targets
    
    * Use sed instead of parameter indexing
    
    * Remove dependabot-generate
    
    Address in open-telemetry#2613 instead.
    MrAlias committed Feb 15, 2022
    Configuration menu
    Copy the full SHA
    cd21df4 View commit details
    Browse the repository at this point in the history

Commits on Feb 16, 2022

  1. Fix race condition in reading the dropped spans number (open-telemetr…

    …y#2615)
    
    * Fix race condition in reading the dropped spans number
    
    As any race condition this should be consider an undefined behavior, and a patch release should be done.
    
    Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
    
    * Update CHANGELOG.md
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    Bogdan Drutu and MrAlias committed Feb 16, 2022
    Configuration menu
    Copy the full SHA
    2a9cedf View commit details
    Browse the repository at this point in the history
  2. Release v1.4.1/v0.27.1 (open-telemetry#2619)

    * Update versions.yaml
    
    * Prepare stable-v1 for version v1.4.1
    
    * Prepare experimental-metrics for version v0.27.1
    
    * Prepare bridge for version v0.27.1
    
    * Update CHANGELOG
    
    * Revert "Prepare experimental-metrics for version v0.27.1"
    
    This reverts commit 838743a.
    
    * Do not release experimental-metrics with v0.27.1
    MrAlias committed Feb 16, 2022
    Configuration menu
    Copy the full SHA
    065ba75 View commit details
    Browse the repository at this point in the history
  3. feature/exporter: add Drop Counts for oltptracer's event and link (op…

    …en-telemetry#2601)
    
    * feature/exporter: add Drop Counts for oltptracer's event
    
    Signed-off-by: 1046102779 <seachen@tencent.com>
    
    * feature/exporter: add Drop Counts for oltptracer's event
    
    Signed-off-by: 1046102779 <seachen@tencent.com>
    
    * feature/exporter: add Drop Counts for oltptracer's event and link
    
    Signed-off-by: 1046102779 <seachen@tencent.com>
    
    * feature/exporter: add Drop Counts for oltptracer's event and link
    
    Signed-off-by: 1046102779 <seachen@tencent.com>
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    1046102779 and MrAlias committed Feb 16, 2022
    Configuration menu
    Copy the full SHA
    67f508b View commit details
    Browse the repository at this point in the history

Commits on Feb 17, 2022

  1. Add dependabot-generate make target (open-telemetry#2613)

    * Refactor common repo code for crosslink
    
    * Add dbotconf utility
    
    * Add dependabot-generate target to Makefile
    
    * Generate dependabot.yml
    
    * Update Makefile targets related to dependabot-generate
    MrAlias committed Feb 17, 2022
    Configuration menu
    Copy the full SHA
    d46c0d2 View commit details
    Browse the repository at this point in the history
  2. Fix typo in using instrumentation libraries markdown (open-telemetry#…

    …2621)
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    bvwells and MrAlias committed Feb 17, 2022
    Configuration menu
    Copy the full SHA
    b675dda View commit details
    Browse the repository at this point in the history
  3. Add env support for span limits configuration (open-telemetry#2606)

    * add env support for otel_span configuration
    
    Signed-off-by: Cuichen Li <cuichli@cisco.com>
    
    * update changelog
    
    * update changelog and some logic based on comment
    
    * Update CHANGELOG.md
    
    Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>
    
    * add document about retrieve value from environment variable
    
    Signed-off-by: Cuichen Li <cuichli@cisco.com>
    
    * remove trailing whitespace
    
    Signed-off-by: Cuichen Li <cuichli@cisco.com>
    
    * parse environment variable before apply the options
    
    * Update CHANGELOG.md
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update sdk/trace/provider_test.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update CHANGELOG.md
    
    Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    3 people committed Feb 17, 2022
    Configuration menu
    Copy the full SHA
    98c2c9d View commit details
    Browse the repository at this point in the history
  4. Remove the otlp trace exporter limit of SpanEvents when exporting (op…

    …en-telemetry#2616)
    
    * remove the limit of SpanEvents when exporting
    
    * fix changelog
    
    * Update exporters/otlp/otlptrace/internal/tracetransform/span.go
    
    Co-authored-by: Sam Xie <xsambundy@gmail.com>
    
    * Update exporters/otlp/otlptrace/internal/tracetransform/span.go
    
    Co-authored-by: Sam Xie <xsambundy@gmail.com>
    
    * Update exporters/otlp/otlptrace/internal/tracetransform/span.go
    
    Co-authored-by: Sam Xie <xsambundy@gmail.com>
    
    * Update CHANGELOG.md
    
    Co-authored-by: Sam Xie <xsambundy@gmail.com>
    
    * fix unused param
    
    * fix changelog
    
    * Update CHANGELOG.md
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * fix unittest
    
    * fix code format
    
    Co-authored-by: Sam Xie <xsambundy@gmail.com>
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    3 people committed Feb 17, 2022
    Configuration menu
    Copy the full SHA
    8297dbf View commit details
    Browse the repository at this point in the history
  5. Bump github.com/golangci/golangci-lint from 1.44.0 to 1.44.1 in /inte…

    …rnal/tools (open-telemetry#2624)
    
    * Bump github.com/golangci/golangci-lint in /internal/tools
    
    Bumps [github.com/golangci/golangci-lint](https://github.com/golangci/golangci-lint) from 1.44.0 to 1.44.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.44.0...v1.44.1)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/golangci/golangci-lint
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    
    * Auto-fix go.sum changes in dependent modules
    
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: MrAlias <MrAlias@users.noreply.github.com>
    dependabot[bot] and MrAlias committed Feb 17, 2022
    Configuration menu
    Copy the full SHA
    8ebef75 View commit details
    Browse the repository at this point in the history

Commits on Feb 18, 2022

  1. Setup benchmarks github action (open-telemetry#2559)

    * setup benchmarks github action
    
    * run on go 1.16
    
    * only run benchmarks on a push
    
    * pin all actions
    dmathieu committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    5ba3866 View commit details
    Browse the repository at this point in the history

Commits on Feb 22, 2022

  1. Bump github.com/golangci/golangci-lint from 1.44.1 to 1.44.2 in /inte…

    …rnal/tools (open-telemetry#2626)
    
    * Bump github.com/golangci/golangci-lint in /internal/tools
    
    Bumps [github.com/golangci/golangci-lint](https://github.com/golangci/golangci-lint) from 1.44.1 to 1.44.2.
    - [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.44.1...v1.44.2)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/golangci/golangci-lint
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    
    * Auto-fix go.sum changes in dependent modules
    
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: MrAlias <MrAlias@users.noreply.github.com>
    dependabot[bot] and MrAlias committed Feb 22, 2022
    Configuration menu
    Copy the full SHA
    8ba8f68 View commit details
    Browse the repository at this point in the history
  2. Use randomly-available port in otlp exporters tests (open-telemetry#2627

    )
    
    * use randomly-available port in otlp exporters tests
    
    * Update exporters/otlp/otlpmetric/otlpmetricgrpc/client_test.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update exporters/otlp/otlpmetric/otlpmetricgrpc/client_test.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update exporters/otlp/otlpmetric/otlpmetricgrpc/client_test.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update exporters/otlp/otlpmetric/otlpmetricgrpc/client_test.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update exporters/otlp/otlptrace/otlptracegrpc/client_test.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>
    3 people committed Feb 22, 2022
    Configuration menu
    Copy the full SHA
    d51d1b3 View commit details
    Browse the repository at this point in the history

Commits on Feb 23, 2022

  1. fix(tracestate): drop right-most member in tracestate (open-telemetry…

    …#2592)
    
    * todo(tracestate): drop right-most member in tracestate
    
    Signed-off-by: 1046102779 <seachen@tencent.com>
    
    * fix(tracestate): drop right-most member in tracestate
    
    Signed-off-by: 1046102779 <seachen@tencent.com>
    
    * fix(tracestate): drop right-most member in tracestate
    
    Signed-off-by: 1046102779 <seachen@tencent.com>
    
    * fix(tracestate): drop right-most member in tracestate
    
    Signed-off-by: 1046102779 <seachen@tencent.com>
    
    * fix(tracestate): drop right-most member in tracestate
    
    Signed-off-by: 1046102779 <seachen@tencent.com>
    
    * Update trace/tracestate.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    1046102779 and MrAlias committed Feb 23, 2022
    Configuration menu
    Copy the full SHA
    94b1848 View commit details
    Browse the repository at this point in the history
  2. Use port 4318 for otlp*http client default (open-telemetry#2625)

    * Use port 4318 for otlptracehttp client default
    
    * Use port 4318 for otlpmetrichttp client default
    
    * Add changes to changelog
    
    * Fix arg pass error
    
    * Simplify defaultPath path parsing
    MrAlias committed Feb 23, 2022
    Configuration menu
    Copy the full SHA
    0a6e4d8 View commit details
    Browse the repository at this point in the history
  3. change gc runtime name to go (open-telemetry#2560)

    Per the specification: open-telemetry/opentelemetry-specification#2262
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    dmathieu and MrAlias committed Feb 23, 2022
    Configuration menu
    Copy the full SHA
    a15269a View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    52e921c View commit details
    Browse the repository at this point in the history
  5. Fixes for PR comments.

    MadVikingGod committed Feb 23, 2022
    Configuration menu
    Copy the full SHA
    b6c7e7c View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    18b9fc4 View commit details
    Browse the repository at this point in the history
  7. Fix linting

    MadVikingGod committed Feb 23, 2022
    Configuration menu
    Copy the full SHA
    32b6eba View commit details
    Browse the repository at this point in the history

Commits on Feb 24, 2022

  1. Fix mod tidy.

    MadVikingGod committed Feb 24, 2022
    Configuration menu
    Copy the full SHA
    d8ebfab View commit details
    Browse the repository at this point in the history

Commits on Feb 25, 2022

  1. Update BatchSpanProcessor debug message (open-telemetry#2640)

    * update debug message
    
    * update changelog
    
    Co-authored-by: Aaron Clawson <MadVikingGod@users.noreply.github.com>
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    3 people committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    8eb3766 View commit details
    Browse the repository at this point in the history
  2. add hanyuancheung as an approver (open-telemetry#2638)

    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    hanyuancheung and MrAlias committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    76bff3f View commit details
    Browse the repository at this point in the history
  3. Add Marshaling implementations for exporters (open-telemetry#2578)

    * Add Marshaling implementations for exporters
    
    * Changelog
    
    * Fix changelog
    
    Co-authored-by: Aaron Clawson <MadVikingGod@users.noreply.github.com>
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    3 people committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    a1fff3c View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    0dd1d52 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    775fb28 View commit details
    Browse the repository at this point in the history

Commits on Mar 1, 2022

  1. Replace master with specific jaeger tag version in otel-collector (op…

    …en-telemetry#2622)
    
    * replace master with specific jaeger tag version in otel-collector
    
    * change version
    
    * update jeager-operator version in makefile
    hanyuancheung committed Mar 1, 2022
    Configuration menu
    Copy the full SHA
    33a5a49 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    79f6bc1 View commit details
    Browse the repository at this point in the history

Commits on Mar 2, 2022

  1. Unify OTLP path parsing/default Logic (open-telemetry#2639)

    * unify otlp path parsing/default logic
    
    * add changelog
    
    * add license and unit test
    
    * remove else branch
    
    * increase unitt test coverage
    
    * add vanity import
    
    * Update exporters/otlp/internal/config.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update exporters/otlp/internal/config.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update exporters/otlp/internal/config.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update CHANGELOG.md
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update exporters/otlp/internal/config_test.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * Update exporters/otlp/internal/config_test.go
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    
    * format the config_test.go
    
    * Update exporters/otlp/internal/config_test.go
    
    Co-authored-by: Sam Xie <sam@samxie.me>
    
    * Update exporters/otlp/internal/config_test.go
    
    Co-authored-by: Sam Xie <sam@samxie.me>
    
    * Update exporters/otlp/internal/config_test.go
    
    Co-authored-by: Sam Xie <sam@samxie.me>
    
    * Update exporters/otlp/internal/config_test.go
    
    Co-authored-by: Sam Xie <sam@samxie.me>
    
    * Update exporters/otlp/internal/config.go
    
    Co-authored-by: Sam Xie <sam@samxie.me>
    
    * Update exporters/otlp/internal/config.go
    
    Co-authored-by: Sam Xie <sam@samxie.me>
    
    * change URLPath to urlPath
    
    Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
    Co-authored-by: Sam Xie <sam@samxie.me>
    3 people committed Mar 2, 2022
    Configuration menu
    Copy the full SHA
    b66c902 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    4792a7c View commit details
    Browse the repository at this point in the history