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

Incomplete sentence in OK status code description #1040

Closed
MrAlias opened this issue Sep 30, 2020 · 0 comments · Fixed by #1067
Closed

Incomplete sentence in OK status code description #1040

MrAlias opened this issue Sep 30, 2020 · 0 comments · Fixed by #1067
Labels
area:error-reporting Related to error reporting bug Something isn't working release:allowed-for-ga Editorial changes that can still be added before GA since they don't require action by SIGs spec:trace Related to the specification/trace directory

Comments

@MrAlias
Copy link
Contributor

MrAlias commented Sep 30, 2020

The Span status code Ok in the StatusCodeCanonical section of the trace API
has a discription with a partial sentence

The operation has been validated by an Application developers or Operator to have completed successfully, or contain

Possibly "... or contains no error"?

@carlosalberto or @tedsuo do you have an idea of what that was supposed to be?

@MrAlias MrAlias added bug Something isn't working spec:trace Related to the specification/trace directory labels Sep 30, 2020
@arminru arminru added area:error-reporting Related to error reporting release:allowed-for-ga Editorial changes that can still be added before GA since they don't require action by SIGs labels Sep 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:error-reporting Related to error reporting bug Something isn't working release:allowed-for-ga Editorial changes that can still be added before GA since they don't require action by SIGs spec:trace Related to the specification/trace directory
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants