Skip to content

fix(ci): use better-sqlite3 v13 prebuilds under Yarn 4 - #5403

Closed
jonkoops wants to merge 1 commit into
redhat-developer:mainfrom
jonkoops:fix/better-sqlite3-yarn-prebuild
Closed

jonkoops wants to merge 1 commit into
redhat-developer:mainfrom
jonkoops:fix/better-sqlite3-yarn-prebuild

Conversation

@jonkoops

Copy link
Copy Markdown
Contributor

After #5359, cluster-free E2E fails on yarn install because Yarn 4 runs node-gyp whenever a dependency ships binding.gyp, even though better-sqlite3 13 bundles platform prebuilds and sets gypfile: false for npm (WiseLibs/better-sqlite3#1503).

This adds dependenciesMeta.better-sqlite3.built: false (same pattern as keytar) so Yarn skips the native build and loads the shipped prebuild at runtime.

RHDHBUGS-3725

Yarn ignores gypfile:false and rebuilds any package that ships binding.gyp.
Set dependenciesMeta.better-sqlite3.built=false so install uses the bundled
prebuilds instead of node-gyp, which unblocks the cluster-free harness on
images without a compiler toolchain.

Ref: RHDHBUGS-3725
Signed-off-by: Jon Koops <jonkoops@gmail.com>
@openshift-ci
openshift-ci Bot requested review from kim-tsao and polasudo September 15, 2026 09:01
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

The container image build workflow finished with status: failure.

@jonkoops

Copy link
Copy Markdown
Contributor Author

Closing per author request.

@jonkoops jonkoops closed this Sep 15, 2026
@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.68%. Comparing base (8fb16b7) to head (bae0681).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5403   +/-   ##
=======================================
  Coverage   63.68%   63.68%           
=======================================
  Files         121      121           
  Lines        2288     2288           
  Branches      527      527           
=======================================
  Hits         1457     1457           
  Misses        829      829           
  Partials        2        2           
Flag Coverage Δ *Carryforward flag
rhdh 63.68% <ø> (ø) Carriedforward from 8fb16b7

*This pull request uses carry forward flags. Click here to find out more.

Components Coverage Δ
Backend plugins 100.00% <ø> (ø)
Backend app 69.69% <ø> (ø)
Frontend app 58.82% <ø> (ø)
Plugin utils 100.00% <ø> (ø)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8fb16b7...bae0681. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant