Skip to content

Rename TraceOptions to TraceFlags to be w3c compatible#234

Merged
bogdandrutu merged 2 commits intoopen-telemetry:masterfrom
bogdandrutu:traceflags
Sep 7, 2019
Merged

Rename TraceOptions to TraceFlags to be w3c compatible#234
bogdandrutu merged 2 commits intoopen-telemetry:masterfrom
bogdandrutu:traceflags

Conversation

@bogdandrutu
Copy link
Copy Markdown
Member

@bogdandrutu
Copy link
Copy Markdown
Member Author

Ping, I need few more approves

@bogdandrutu
Copy link
Copy Markdown
Member Author

Approvers please approve this PR. I think this is a trivial (and good) change.

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
@bogdandrutu bogdandrutu merged commit a0c0cb8 into open-telemetry:master Sep 7, 2019
@bogdandrutu bogdandrutu deleted the traceflags branch September 7, 2019 20:33
bg451 pushed a commit to bg451/opentelemetry-specification that referenced this pull request Sep 9, 2019
…y#234)

* Rename TraceOptions to TraceFlags to be w3c compatible

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

* Fix more TraceOptions and rename recorded with sampled

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Brandon Gonzalez <bg@lightstep.com>
songy23 added a commit to songy23/opentelemetry-java that referenced this pull request Sep 10, 2019
yurishkuro pushed a commit that referenced this pull request Sep 13, 2019
* Removes references of spandata

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* initial stab at tracing spec

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* adds link and event timestamp

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* re-add whitespace to reduce diff size

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* typo fix

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Adds OutOfBand flag to span options

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* specification/api-distributedcontext: fix typo (#236)

contest -> context

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Adds comment+link to Remove Out Of Band Support RFC

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* remove comment in doc

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Remove work_in_progress dir (#225)

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Move start timestamp above the possibly removed fields

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Fix typo in FailedNotRetryable (#243)

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Rename TraceOptions to TraceFlags to be w3c compatible (#234)

* Rename TraceOptions to TraceFlags to be w3c compatible

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

* Fix more TraceOptions and rename recorded with sampled

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Respond to comment
SergeyKanzhelev pushed a commit to SergeyKanzhelev/opentelemetry-specification that referenced this pull request Feb 18, 2020
…y#234)

* Rename TraceOptions to TraceFlags to be w3c compatible

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

* Fix more TraceOptions and rename recorded with sampled

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
SergeyKanzhelev pushed a commit to SergeyKanzhelev/opentelemetry-specification that referenced this pull request Feb 18, 2020
* Removes references of spandata

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* initial stab at tracing spec

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* adds link and event timestamp

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* re-add whitespace to reduce diff size

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* typo fix

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Adds OutOfBand flag to span options

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* specification/api-distributedcontext: fix typo (open-telemetry#236)

contest -> context

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Adds comment+link to Remove Out Of Band Support RFC

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* remove comment in doc

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Remove work_in_progress dir (open-telemetry#225)

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Move start timestamp above the possibly removed fields

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Fix typo in FailedNotRetryable (open-telemetry#243)

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Rename TraceOptions to TraceFlags to be w3c compatible (open-telemetry#234)

* Rename TraceOptions to TraceFlags to be w3c compatible

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

* Fix more TraceOptions and rename recorded with sampled

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Respond to comment
TuckTuckFloof pushed a commit to TuckTuckFloof/opentelemetry-specification that referenced this pull request Oct 15, 2020
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
…y#234)

* Rename TraceOptions to TraceFlags to be w3c compatible

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

* Fix more TraceOptions and rename recorded with sampled

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
* Removes references of spandata

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* initial stab at tracing spec

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* adds link and event timestamp

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* re-add whitespace to reduce diff size

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* typo fix

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Adds OutOfBand flag to span options

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* specification/api-distributedcontext: fix typo (open-telemetry#236)

contest -> context

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Adds comment+link to Remove Out Of Band Support RFC

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* remove comment in doc

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Remove work_in_progress dir (open-telemetry#225)

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Move start timestamp above the possibly removed fields

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Fix typo in FailedNotRetryable (open-telemetry#243)

Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Rename TraceOptions to TraceFlags to be w3c compatible (open-telemetry#234)

* Rename TraceOptions to TraceFlags to be w3c compatible

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

* Fix more TraceOptions and rename recorded with sampled

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Brandon Gonzalez <bg@lightstep.com>

* Respond to comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants