Skip to content

Fix BGP GitHub CI by building in the FBOSS image - #16

Closed
KevinYakar wants to merge 1 commit into
meta:mainfrom
KevinYakar:export-D128415158
Closed

KevinYakar wants to merge 1 commit into
meta:mainfrom
KevinYakar:export-D128415158

Conversation

@KevinYakar

Copy link
Copy Markdown
Contributor

Summary:
The BGP GitHub Actions workflow has failed on every run since it was added in July. It runs getdeps directly on an Ubuntu runner, where this dependency closure has never been built: it first fails to resolve the EL9-only gcc12, and past that, third-party code fails to compile under GCC 13.

Build in the FBOSS CentOS Stream 9 image through run-getdeps.py instead, the environment the oss_bgp conveyor and on-diff BGP OSS jobs already build in, so GitHub CI checks the same thing internal CI does.

Differential Revision: D128415158

Summary:
The BGP GitHub Actions workflow has failed on every run since it was added in July. It runs getdeps directly on an Ubuntu runner, where this dependency closure has never been built: it first fails to resolve the EL9-only `gcc12`, and past that, third-party code fails to compile under GCC 13.

Build in the FBOSS CentOS Stream 9 image through `run-getdeps.py` instead, the environment the `oss_bgp` conveyor and on-diff BGP OSS jobs already build in, so GitHub CI checks the same thing internal CI does.

Differential Revision: D128415158
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 29, 2026
@meta-codesync

meta-codesync Bot commented Oct 29, 2026

Copy link
Copy Markdown

@KevinYakar has exported this pull request. If you are a Meta employee, you can view the originating Diff in D128415158.

@meta-codesync

meta-codesync Bot commented Oct 29, 2026

Copy link
Copy Markdown

This pull request has been merged in 2f54d69.

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

Labels

CLA Signed This label is managed by the Meta Open Source bot. Merged meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant