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

Add /schemas in YAML, /schemas/list in HTML, and a 404 page for the rest #3537

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Nov 14, 2023

Previews

Redirect tests

Screenshots / output

YAML list (/schemas):

# OpenTelemetry Schemas v1.23.0
#
# For details, see https://opentelemetry.io/schemas/list

- 1.23.0
- 1.22.0
- 1.21.0
- 1.20.0
- 1.19.0
- 1.18.0
- 1.17.0
- 1.16.0
- 1.15.0
- 1.14.0
- 1.13.0
- 1.12.0
- 1.11.0
- 1.10.0
- 1.9.0
- 1.8.0
- 1.7.0
- 1.6.1
- 1.5.0
- 1.4.0

List page (/schemas/list):

image

404 page (/schemas/x.y.z):

image

HTTP status tests

$ curl -sI https://deploy-preview-3537--opentelemetry.netlify.app/schemas/x.y.z | grep -E 'HTTP|-type|location' 
HTTP/2 404 
content-type: text/html; charset=UTF-8
$ curl -sI https://deploy-preview-3537--opentelemetry.netlify.app/schemas/1.23.0 | grep -E 'HTTP|-type|location' 
HTTP/2 200 
content-type: application/yaml

@chalin chalin requested a review from a team November 14, 2023 10:48
@chalin chalin force-pushed the chalin-im-schema-list-as-data-2023-11-14 branch 2 times, most recently from e3b7229 to 707c1de Compare November 14, 2023 10:53
@chalin chalin marked this pull request as draft November 14, 2023 10:58
@chalin chalin marked this pull request as ready for review November 14, 2023 11:08
@chalin chalin force-pushed the chalin-im-schema-list-as-data-2023-11-14 branch 2 times, most recently from 664951b to d9ecad7 Compare November 14, 2023 11:23
@chalin chalin requested review from a team November 14, 2023 11:26
@chalin chalin force-pushed the chalin-im-schema-list-as-data-2023-11-14 branch from d9ecad7 to 0c05527 Compare November 14, 2023 11:35
@chalin chalin force-pushed the chalin-im-schema-list-as-data-2023-11-14 branch from 0c05527 to 8c0f14d Compare November 14, 2023 11:38
@chalin chalin changed the title Schemas at /schemas in YAML, and /schemas/list in HTML Add /schemas in YAML, /schemas/list in HTML, and a 404 page for the rest Nov 14, 2023
@tigrannajaryan
Copy link
Member

  • Adds /schemas, the list of schemas published on the site, in YAML format

I think this needs to be defined in the spec repo.

@chalin chalin marked this pull request as draft November 14, 2023 16:22
@chalin
Copy link
Contributor Author

chalin commented Nov 14, 2023

Putting this in draft mode until the machine-readable list appears in the spec repo. open-telemetry/opentelemetry-specification#3762 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Blocked
Development

Successfully merging this pull request may close these issues.

/schemas is the top 404
3 participants