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

setStatus method conforms to the specified behavior regarding status … #6808

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

setStatus method conforms to the specified behavior regarding status … #6808

wants to merge 23 commits into from

Commits on Oct 21, 2024

  1. setStatus method conforms to the specified behavior regarding status … 

    …code priorities and description handling

Relate to open-telemetry#6797
    @Victorsesan
    Victorsesan committed Oct 21, 2024
    Configuration menu
    Copy the full SHA
    7f9ef5f View commit details
    Browse the repository at this point in the history

Commits on Oct 23, 2024

  1. fix 

    Relate to open-telemetry#6797
    @Victorsesan
    Victorsesan committed Oct 23, 2024
    Configuration menu
    Copy the full SHA
    ef46f8f View commit details
    Browse the repository at this point in the history

  2. Merge branch 'open-telemetry:main' into main

    @Victorsesan
    Victorsesan authored Oct 23, 2024
    Configuration menu
    Copy the full SHA
    9590742 View commit details
    Browse the repository at this point in the history

Commits on Oct 24, 2024

  1. Fixed and adjusted the following. 

    *Gradle build in build.gradle.kts had failed during spotless check because the plugin org.graalvm.buildtools.native was not found
-Solution: I have applied a plugin and made it work

*spotlessJavaCheck task failed because certain files did not conform to the formatting rules defined in the Spotless configuration
-Solution: Initiated an auto fix using ./gradlew :sdk:trace:spotlessApply

*Lastly as requested i initiated
./gradlew spotlessApply to ensure a consist code formating and the build was successful with no errors!

Relate to open-telemetry#6797
    @Victorsesan
    Victorsesan committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    a9fb915 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' of github.com:Victorsesan/opentelemetry-java

    @Victorsesan
    Victorsesan committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    500d261 View commit details
    Browse the repository at this point in the history

Commits on Oct 25, 2024

  1. fix comments

    @Victorsesan
    Victorsesan committed Oct 25, 2024
    Configuration menu
    Copy the full SHA
    aa731c7 View commit details
    Browse the repository at this point in the history

  2. fix comment

    @Victorsesan
    Victorsesan committed Oct 25, 2024
    Configuration menu
    Copy the full SHA
    bed6535 View commit details
    Browse the repository at this point in the history

Commits on Oct 26, 2024

  1. Created a test in the sdkspantest to see if codecov will have line 43… 

    …7 and 438 covered(just for trials)
    @Victorsesan
    Victorsesan committed Oct 26, 2024
    Configuration menu
    Copy the full SHA
    c21608c View commit details
    Browse the repository at this point in the history

  2. test 2

    @Victorsesan
    Victorsesan committed Oct 26, 2024
    Configuration menu
    Copy the full SHA
    ec33889 View commit details
    Browse the repository at this point in the history

  3. .

    @Victorsesan
    Victorsesan committed Oct 26, 2024
    Configuration menu
    Copy the full SHA
    ab67793 View commit details
    Browse the repository at this point in the history

  4. .

    @Victorsesan
    Victorsesan committed Oct 26, 2024
    Configuration menu
    Copy the full SHA
    189c416 View commit details
    Browse the repository at this point in the history

  5. .

    @Victorsesan
    Victorsesan committed Oct 26, 2024
    Configuration menu
    Copy the full SHA
    7991abf View commit details
    Browse the repository at this point in the history

Commits on Nov 3, 2024

  1. ensured span's status does not change after it has ended in the nothi… 

    …ngChangedAfterEnd test

Ref Fix Span#setStatus open-telemetry#6797
    @Victorsesan
    Victorsesan committed Nov 3, 2024
    Configuration menu
    Copy the full SHA
    448ece8 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'open-telemetry:main' into main

    @Victorsesan
    Victorsesan authored Nov 3, 2024
    Configuration menu
    Copy the full SHA
    5a62252 View commit details
    Browse the repository at this point in the history

Commits on Nov 4, 2024

  1. #

    @Victorsesan
    Victorsesan committed Nov 4, 2024
    Configuration menu
    Copy the full SHA
    2fdd372 View commit details
    Browse the repository at this point in the history

  2. #

    @Victorsesan
    Victorsesan committed Nov 4, 2024
    Configuration menu
    Copy the full SHA
    d452c3e View commit details
    Browse the repository at this point in the history

Commits on Nov 8, 2024

  1. Update sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkSpan.java 

    Co-authored-by: John Watson <[email protected]>
    @Victorsesan @jkwatson
    Victorsesan and jkwatson authored Nov 8, 2024
    Configuration menu
    Copy the full SHA
    f04bb8d View commit details
    Browse the repository at this point in the history

  2. Update sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkSpan.java 

    Co-authored-by: John Watson <[email protected]>
    @Victorsesan @jkwatson
    Victorsesan and jkwatson authored Nov 8, 2024
    Configuration menu
    Copy the full SHA
    5aa430e View commit details
    Browse the repository at this point in the history

  3. Update sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkSpan.java 

    Co-authored-by: John Watson <[email protected]>
    @Victorsesan @jkwatson
    Victorsesan and jkwatson authored Nov 8, 2024
    Configuration menu
    Copy the full SHA
    db6a054 View commit details
    Browse the repository at this point in the history

  4. Update sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkSpan.java 

    Co-authored-by: John Watson <[email protected]>
    @Victorsesan @jkwatson
    Victorsesan and jkwatson authored Nov 8, 2024
    Configuration menu
    Copy the full SHA
    b1776cf View commit details
    Browse the repository at this point in the history

  5. Update sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkSpan.java 

    Co-authored-by: John Watson <[email protected]>
    @Victorsesan @jkwatson
    Victorsesan and jkwatson authored Nov 8, 2024
    Configuration menu
    Copy the full SHA
    d9e11e8 View commit details
    Browse the repository at this point in the history

  6. Added a new test to help ensure that my setStatus method behaves corr… 

    …ectly under the specified conditions, maintaining the integrity of the span's status.

Ref Fix Span#setStatus open-telemetry#6797
    @Victorsesan
    Victorsesan committed Nov 8, 2024
    Configuration menu
    Copy the full SHA
    8fc9659 View commit details
    Browse the repository at this point in the history

  7. Merge branch 'main' of github.com:Victorsesan/opentelemetry-java

    @Victorsesan
    Victorsesan committed Nov 8, 2024
    Configuration menu
    Copy the full SHA
    cb9283e View commit details
    Browse the repository at this point in the history