Skip to content

kona: build cannon from local monorepo source instead of cloning a pinned tag#19183

Merged
Inphi merged 1 commit intodevelopfrom
inphi/kona-local-cannon-prestate-build
Feb 13, 2026
Merged

kona: build cannon from local monorepo source instead of cloning a pinned tag#19183
Inphi merged 1 commit intodevelopfrom
inphi/kona-local-cannon-prestate-build

Conversation

@Inphi
Copy link
Contributor

@Inphi Inphi commented Feb 12, 2026

Now that kona lives in the monorepo, the prestate build no longer needs to clone the optimism repo at a pinned commit to build cannon. Instead, cannon is built from the local monorepo source via a Docker named build context, removing the cannon_tag config file and simplifying the build.

fixes #18965

…nned tag

Now that kona lives in the monorepo, the prestate build no longer needs
to clone the optimism repo at a pinned commit to build cannon. Instead,
cannon is built from the local monorepo source via a Docker named build
context, removing the cannon_tag config file and simplifying the build.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@wiz-inc-a178a98b5d
Copy link

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities -
Data Finding Sensitive Data -
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations 4 Medium 1 Info
SAST Finding SAST Findings -
Software Management Finding Software Management Findings -
Total 4 Medium 1 Info

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@codecov
Copy link

codecov bot commented Feb 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.7%. Comparing base (3235773) to head (b3aa8c4).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #19183      +/-   ##
===========================================
- Coverage     76.3%    75.7%    -0.7%     
===========================================
  Files          188      692     +504     
  Lines        10946    71528   +60582     
===========================================
+ Hits          8360    54165   +45805     
- Misses        2440    17219   +14779     
+ Partials       146      144       -2     
Flag Coverage Δ
cannon-go-tests-64 66.4% <ø> (-0.9%) ⬇️
contracts-bedrock-tests 81.6% <ø> (-0.1%) ⬇️
unit 75.6% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 510 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Inphi Inphi marked this pull request as ready for review February 12, 2026 22:54
Copy link
Contributor

@ajsutton ajsutton left a comment

Choose a reason for hiding this comment

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

LGTM. The top level make reproducible-prestate target is actually broken but it is on develop as well so I don't think that should be a blocker for this. I'll pick up fixing it.

@Inphi Inphi added this pull request to the merge queue Feb 13, 2026
Merged via the queue into develop with commit 7bf99eb Feb 13, 2026
118 of 120 checks passed
@Inphi Inphi deleted the inphi/kona-local-cannon-prestate-build branch February 13, 2026 01:12
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.

ci: build kona prestates with the current version of cannon instead of using kona/.config/cannon_tag

2 participants