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

Handle partial-success responses for OTLP trace #3106

Merged
merged 22 commits into from Sep 6, 2022
Merged

Handle partial-success responses for OTLP trace #3106

merged 22 commits into from Sep 6, 2022

Commits on Aug 23, 2022

  1. Handle partial-success responses for OTLP trace and metrics

    @jmacd
    jmacd committed Aug 23, 2022
    Copy the full SHA
    92c8191 View commit details
    Browse the repository at this point in the history

  2. lint

    @jmacd
    jmacd committed Aug 23, 2022
    Copy the full SHA
    a67f694 View commit details
    Browse the repository at this point in the history

  3. changelog entry

    @jmacd
    jmacd committed Aug 23, 2022
    Copy the full SHA
    bc374c3 View commit details
    Browse the repository at this point in the history

Commits on Aug 24, 2022

  1. lint

    @jmacd
    jmacd committed Aug 24, 2022
    Copy the full SHA
    0bda269 View commit details
    Browse the repository at this point in the history

  2. Apply suggestions from code review 

    Co-authored-by: David Ashpole <dashpole@google.com>
    @jmacd @dashpole
    jmacd and dashpole committed Aug 24, 2022
    Copy the full SHA
    4ddf3b5 View commit details
    Browse the repository at this point in the history

  3. Update exporters/otlp/otlpmetric/otlpmetricgrpc/client.go 

    Co-authored-by: David Ashpole <dashpole@google.com>
    @jmacd @dashpole
    jmacd and dashpole committed Aug 24, 2022
    Copy the full SHA
    0e8a80c View commit details
    Browse the repository at this point in the history

  4. Merge branch 'main' of github.com:open-telemetry/opentelemetry-go int… 

    …o jmacd/partial_success
    @jmacd
    jmacd committed Aug 24, 2022
    Copy the full SHA
    8284366 View commit details
    Browse the repository at this point in the history

Commits on Aug 29, 2022

  1. Merge branch 'main' into jmacd/partial_success

    @hanyuancheung
    hanyuancheung committed Aug 29, 2022
    Copy the full SHA
    8723d2e View commit details
    Browse the repository at this point in the history

  2. Update exporters/otlp/otlpmetric/otlpmetricgrpc/client_test.go 

    Co-authored-by: Chester Cheung <cheung.zhy.csu@gmail.com>
    @jmacd @hanyuancheung
    jmacd and hanyuancheung committed Aug 29, 2022
    Copy the full SHA
    b14e132 View commit details
    Browse the repository at this point in the history

  3. Add testing; export error type for callers

    @jmacd
    jmacd committed Aug 29, 2022
    Copy the full SHA
    0c71272 View commit details
    Browse the repository at this point in the history

Commits on Aug 30, 2022

  1. less code more coverage

    @jmacd
    jmacd committed Aug 30, 2022
    Copy the full SHA
    14db574 View commit details
    Browse the repository at this point in the history

  2. improve coverage

    @jmacd
    jmacd committed Aug 30, 2022
    Copy the full SHA
    54d9dcf View commit details
    Browse the repository at this point in the history

  3. cleanup

    @jmacd
    jmacd committed Aug 30, 2022
    Copy the full SHA
    7d538e5 View commit details
    Browse the repository at this point in the history

Commits on Aug 31, 2022

  1. Revert "improve coverage" 

    This reverts commit 54d9dcf.
Mixed results from the style question. We prefer less test, more readability it seems.
    @jmacd
    jmacd committed Aug 31, 2022
    Copy the full SHA
    34b7fa5 View commit details
    Browse the repository at this point in the history

  2. revert otlpmetric portion

    @jmacd
    jmacd committed Aug 31, 2022
    Copy the full SHA
    d3ff369 View commit details
    Browse the repository at this point in the history

Commits on Sep 1, 2022

  1. Update exporters/otlp/otlptrace/otlptracehttp/client_test.go 

    Co-authored-by: Aaron Clawson <3766680+MadVikingGod@users.noreply.github.com>
    @jmacd @MadVikingGod
    jmacd and MadVikingGod committed Sep 1, 2022
    Copy the full SHA
    e207c7a View commit details
    Browse the repository at this point in the history

  2. defer the body.Close(); only parse partial success in OK case

    @jmacd
    jmacd committed Sep 1, 2022
    Copy the full SHA
    3c35b52 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'jmacd/partial_success' of github.com:jmacd/opentelemetr… 

    …y-go into jmacd/partial_success
    @jmacd
    jmacd committed Sep 1, 2022
    Copy the full SHA
    7331bd6 View commit details
    Browse the repository at this point in the history

  4. Update exporters/otlp/otlptrace/otlptracegrpc/client_test.go 

    Co-authored-by: Aaron Clawson <3766680+MadVikingGod@users.noreply.github.com>
    @jmacd @MadVikingGod
    jmacd and MadVikingGod committed Sep 1, 2022
    Copy the full SHA
    4998910 View commit details
    Browse the repository at this point in the history

  5. revert not sure how

    @jmacd
    jmacd committed Sep 1, 2022
    Copy the full SHA
    20f5c53 View commit details
    Browse the repository at this point in the history

  6. update chlog

    @jmacd
    jmacd committed Sep 1, 2022
    Copy the full SHA
    a04ac8a View commit details
    Browse the repository at this point in the history

Commits on Sep 6, 2022

  1. Merge branch 'main' into jmacd/partial_success

    @MadVikingGod
    MadVikingGod committed Sep 6, 2022
    Copy the full SHA
    96eadb0 View commit details
    Browse the repository at this point in the history