Skip to content

fix(docker): reject incomplete OpenROAD builds - #4567

Merged
vvbandeira merged 1 commit into
The-OpenROAD-Project:masterfrom
vvbandeira:ci-image-integrity
Sep 25, 2026
Merged

vvbandeira merged 1 commit into
The-OpenROAD-Project:masterfrom
vvbandeira:ci-image-integrity

Conversation

@vvbandeira

Copy link
Copy Markdown
Member

No description provided.

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request adds a check to verify that the OpenROAD binary is built and executable during the Docker build process. The feedback suggests replacing the simple 'test -x' command with an explicit conditional block that outputs a clear error message upon failure, which will improve debuggability in CI/CD pipelines.

Comment thread docker/Dockerfile.builder Outdated
Verific integration can succeed while the OpenROAD build silently
fails to produce the binary. Check for it explicitly and exit
before the image is layered further.

Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
@vvbandeira
vvbandeira merged commit 2bf0c95 into The-OpenROAD-Project:master Sep 25, 2026
10 of 12 checks passed
@openroad-ci

openroad-ci commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

🔍 QoR check

Metrics reflect the PR merge build — i.e. what will land on the target branch.

Commit 273e666 · Jenkins build #2 · Baseline: build · View build on dashboard

62 design(s) checked — 0 with regression(s), 0 without a comparable baseline.

@vvbandeira
vvbandeira deleted the ci-image-integrity branch September 26, 2026 00:22
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.

2 participants