Skip to content

Declarative Config Spec Docs - Schema transform#9654

Open
jaydeluca wants to merge 11 commits intoopen-telemetry:mainfrom
jaydeluca:pr1-js-core-tests
Open

Declarative Config Spec Docs - Schema transform#9654
jaydeluca wants to merge 11 commits intoopen-telemetry:mainfrom
jaydeluca:pr1-js-core-tests

Conversation

@jaydeluca
Copy link
Copy Markdown
Member

@jaydeluca jaydeluca commented Apr 14, 2026

  • I have read and followed the Contributing docs, especially the "First-time contributing?" section.
  • This PR has content that I did not fully write myself.
  • I have the experience and knowledge necessary to understand, review, and validate all content in this PR.1

Part 1 of #9521

The completed implementation we are working towards is in #9383 - here is the final page this code will be a part of. I am breaking things up to hopefully make reviewing this easier

Context for this change:

We are going to have the opentelemetry-configuration repo as a git submodule, and then mount the schema json file as a static asset for the site. We will then use the assets/js/configSchemaTransform.mjs script in this PR in order to convert the raw schema data into a format that is easier to use with the page we are building.

cc @jack-berg

Footnotes

  1. Yes, I can answer maintainer questions about the content of this PR, without using AI.

@jaydeluca jaydeluca requested a review from a team as a code owner April 14, 2026 23:50
@github-actions github-actions Bot added the dependencies Pull requests that update a dependency file label Apr 14, 2026
@otelbot-docs otelbot-docs Bot added the missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer label Apr 14, 2026
@otelbot-docs otelbot-docs Bot added ready-to-be-merged This PR is ready to be merged by a maintainer and removed missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer labels Apr 15, 2026
Copy link
Copy Markdown
Member

@jack-berg jack-berg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Verified by running against my local copy of opentelemetry-configuration/opentelemetry_configuration.json.

When is opentelemetry-configuration going to be added to .gitmodules?

@jaydeluca
Copy link
Copy Markdown
Member Author

When is opentelemetry-configuration going to be added to .gitmodules?

I had assumed #9448 would land before this so I didn't include it in this one

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

Labels

dependencies Pull requests that update a dependency file ready-to-be-merged This PR is ready to be merged by a maintainer

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants