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

All schema files are returning 404 #1846

Closed
tigrannajaryan opened this issue Oct 11, 2022 · 19 comments · Fixed by #1851
Closed

All schema files are returning 404 #1846

tigrannajaryan opened this issue Oct 11, 2022 · 19 comments · Fixed by #1851
Labels
bug Something isn't working

Comments

@tigrannajaryan
Copy link
Member

tigrannajaryan commented Oct 11, 2022

What happened?
The schema file URLs that used to work are now returning 404. E.g. https://opentelemetry.io/schemas/1.7.1 or https://opentelemetry.io/schemas/1.14.0

What did you expect would happen?
It should be a 200 and return the content of relevant file from https://github.com/open-telemetry/opentelemetry-specification/tree/main/schemas

What is the name + path of the page where you encountered the issue?
https://opentelemetry.io/schemas/*

What browser, OS, and platform were you using when you encountered the problem?
Chrome Version 105.0.5195.125 (Official Build) (x86_64), Mac OS 12.5

@tigrannajaryan tigrannajaryan added bug Something isn't working p0-critical labels Oct 11, 2022
@tigrannajaryan
Copy link
Member Author

This is a critical bug that needs a quick fix and we need automated tests to prevent this from happening again.

@tigrannajaryan
Copy link
Member Author

@open-telemetry/docs-approvers can someone help with this? I am not sure when exactly did we break this.

@tigrannajaryan
Copy link
Member Author

The publishing of schema files was previously automated in #715

@cartermp
Copy link
Contributor

@chalin would have the most context here.

@svrnm
Copy link
Member

svrnm commented Oct 11, 2022

1.4.0 and 1.5.0 work

@svrnm
Copy link
Member

svrnm commented Oct 11, 2022

ok. the following versions work:

1.10.0 1.11.0 1.12.0 1.4.0 1.5.0 1.6.1 1.7.0 1.8.0 1.9.0

@svrnm
Copy link
Member

svrnm commented Oct 11, 2022

looks like the spec version has not been upgraded for 1.13 and 1.14. git submodule version is at ae87c81640d18763d4df51b898c55c49e2370328, which is 1.12

tigrannajaryan added a commit to tigrannajaryan/opentelemetry-specification that referenced this issue Oct 11, 2022
This downloads the schema files from https://opentelemetry.io/schemas/ and
verifies their content.

Next time we make a release this will force us to make sure
https://github.com/open-telemetry/opentelemetry.io repo is updated
before we make the release.

This helps prevent issues like open-telemetry/opentelemetry.io#1846
@tigrannajaryan
Copy link
Member Author

@svrnm will you fix it or want me to take a look at this?

@svrnm
Copy link
Member

svrnm commented Oct 11, 2022

@svrnm will you fix it or want me to take a look at this?

I tried it but tbh my git skills are not advanced enough: I tried to pull the 1.14 commit in the Submodule and not HEAD but couldn’t figure it out

So if you know how that would be appreciated

@tigrannajaryan
Copy link
Member Author

Let me try.

@chalin
Copy link
Contributor

chalin commented Oct 11, 2022

Hi all. I'll be back at my desk in about 15 min. I can give it a try then if you'd like. Let me know.

@chalin
Copy link
Contributor

chalin commented Oct 11, 2022

Ok, back. Btw, there are some extra issues to be dealt with -- see #1495. I can address all of those in one go.

@tigrannajaryan
Copy link
Member Author

Ok, back. Btw, there are some extra issues to be dealt with -- see #1495. I can address all of those in one go.

Please do and ping me when done so that I can re-run the verification job on the spec repo: open-telemetry/opentelemetry-specification#2868

@chalin
Copy link
Contributor

chalin commented Oct 11, 2022

@tigrannajaryan - I'm on it. The spec has changed, resulting in broken links, so I'm fixing that now.

@chalin
Copy link
Contributor

chalin commented Oct 11, 2022

@tigrannajaryan et al.: the PR is ready for review

@chalin
Copy link
Contributor

chalin commented Oct 11, 2022

The schema file URLs that used to work are now returning 404. E.g. https://opentelemetry.io/schemas/1.7.1 or https://opentelemetry.io/schemas/1.14.0

@tigrannajaryan - I don't see any 1.7.1 schema under https://github.com/open-telemetry/opentelemetry-specification/tree/main/schemas. Did you mean 1.6.1?

@chalin
Copy link
Contributor

chalin commented Oct 11, 2022

The new spec is live. If you would like any further changes, let me know and I'll include them in a followup PR.

@chalin
Copy link
Contributor

chalin commented Oct 11, 2022

The publishing of schema files was previously automated in #715

To clarify: the schema files get published once the spec content-module is updated. The update of the spec isn't automated since it usually requires manual intervention, such as fixing broken links into the spec pages.

@tigrannajaryan
Copy link
Member Author

@tigrannajaryan - I don't see any 1.7.1 schema under https://github.com/open-telemetry/opentelemetry-specification/tree/main/schemas. Did you mean 1.6.1?

My mistake. I was looking for a schema that never existed.

tigrannajaryan added a commit to open-telemetry/opentelemetry-specification that referenced this issue Oct 17, 2022
This downloads the schema files from https://opentelemetry.io/schemas/ and verifies their content.

Next time we make a release this will force us to make sure
https://github.com/open-telemetry/opentelemetry.io repo is updated
before we make the release.

This helps prevent issues like open-telemetry/opentelemetry.io#1846

Also add release procedure to the docs.
ChengJinbao added a commit to ChengJinbao/opentelemetry-specification that referenced this issue Nov 16, 2022
This downloads the schema files from https://opentelemetry.io/schemas/ and verifies their content.

Next time we make a release this will force us to make sure
https://github.com/open-telemetry/opentelemetry.io repo is updated
before we make the release.

This helps prevent issues like open-telemetry/opentelemetry.io#1846

Also add release procedure to the docs.
jsuereth pushed a commit to jsuereth/otel-semconv-test that referenced this issue Apr 19, 2023
This downloads the schema files from https://opentelemetry.io/schemas/ and verifies their content.

Next time we make a release this will force us to make sure
https://github.com/open-telemetry/opentelemetry.io repo is updated
before we make the release.

This helps prevent issues like open-telemetry/opentelemetry.io#1846

Also add release procedure to the docs.
jsuereth pushed a commit to open-telemetry/semantic-conventions that referenced this issue May 11, 2023
This downloads the schema files from https://opentelemetry.io/schemas/ and verifies their content.

Next time we make a release this will force us to make sure
https://github.com/open-telemetry/opentelemetry.io repo is updated
before we make the release.

This helps prevent issues like open-telemetry/opentelemetry.io#1846

Also add release procedure to the docs.
joaopgrassi pushed a commit to dynatrace-oss-contrib/semantic-conventions that referenced this issue Mar 21, 2024
This downloads the schema files from https://opentelemetry.io/schemas/ and verifies their content.

Next time we make a release this will force us to make sure
https://github.com/open-telemetry/opentelemetry.io repo is updated
before we make the release.

This helps prevent issues like open-telemetry/opentelemetry.io#1846

Also add release procedure to the docs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants