Skip to content

Migrate to new shared build infrastructure. - #5424

Merged
lornajane merged 11 commits into
OAI:mainfrom
handrews:shared-infra
Aug 13, 2026
Merged

Migrate to new shared build infrastructure.#5424
lornajane merged 11 commits into
OAI:mainfrom
handrews:shared-infra

Conversation

@handrews

@handrews handrews commented Jul 4, 2026

Copy link
Copy Markdown
Member

This Codex-authored draft pull request is the companion to OAI/build-infra#1 and MUST NOT be merged until after that PR has been merged, and until after we have proven that the new system works with OAI/sig-lifecycle#1. I am posting it here purely to illustrate the impact so the complete approach can be considered. See OAI/build-infra#1 for full context and a discussion of the use of AI and how to review these changes.

This might try to run CI which may or may not work given the state of OAI/build-infra. I'll look into any failures and either explain them or fix them prior to marking this ready to review.

This new system pushes all dependency management into the build-infra repository, eliminating the continual flood of dependabot updates and branch syncs we are currently experiencing. It also makes it trivial to set up the SIG repositories for specificaiton publishing.

  • no schema changes are needed for this pull request

@handrews
handrews force-pushed the shared-infra branch 2 times, most recently from cb6896e to f159e14 Compare July 4, 2026 21:05
@handrews

handrews commented Jul 4, 2026

Copy link
Copy Markdown
Member Author

Some failures were due to a platform-specific dependency installation problem. This has been fixed, and the issue has been documented in the build-infra repository with the instructions on setting up repos to use the shared infrastructure.

The last failure was due to linkspector checking a link to a build-infra markdown file that is not yet on main, so I added it to the exclusion list temporarily. I'll remove that commit from this branch before taking this out of draft, assuming we decide this PR is worth reviewing in its current form.

Comment thread package.json
@handrews

Copy link
Copy Markdown
Member Author

Force-pushed (twice because I messed it up the first time) for rebase (no changes as package conflicts were just over having deleted pretty much everything in favor of the git repo).

Also one new commit to add --allow-git=root to all npm ci workflow installs.

@handrews
handrews marked this pull request as ready for review July 25, 2026 23:37
@handrews
handrews requested review from a team as code owners July 25, 2026 23:37
@handrews handrews changed the title [DRAFT] Migrate to new shared build infrastructure. Migrate to new shared build infrastructure. Jul 26, 2026
Comment thread .github/workflows/respec.yaml Outdated
@handrews

Copy link
Copy Markdown
Member Author

regarding the CI failure: I think this does not properly handle main (without src/oas.md) vs other branches. I will look into it but it might be a few days. setting back to draft for now.

@handrews

handrews commented Aug 6, 2026

Copy link
Copy Markdown
Member Author

I'm going to close and re-open this to try to get CI unstuck, it seems to be hung up on something even with a few re-runs.

@handrews

handrews commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

The only remaining CI failure is due to the broken link fixed in PR #5471

handrews and others added 10 commits August 8, 2026 17:42
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
@handrews
handrews marked this pull request as ready for review August 9, 2026 17:59
@handrews

handrews commented Aug 9, 2026

Copy link
Copy Markdown
Member Author

CI is passing! This is ready for review — the most recent problems were purely linkspector issues, nothing actually in this PR has changed other than the --allow-git=root update in response to @baywet , plus the most recent update to which commit in build-infra we are pointing.

@lornajane lornajane left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, this is an experimental merge as discussed

@lornajane
lornajane merged commit ac3f1be into OAI:main Aug 13, 2026
2 checks passed
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.

3 participants