Skip to content

Poll Scan for featured app rights in the bridge app-rewards gate; pin LocalNet to Splice 0.8.3 - #211

Merged
0xNeshi merged 4 commits into
mainfrom
fix-traffic-in-scripts
Oct 8, 2026
Merged

0xNeshi merged 4 commits into
mainfrom
fix-traffic-in-scripts

Conversation

@0xNeshi

@0xNeshi 0xNeshi commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

The scheduled live-ledger-gates run failed in bridge-app-rewards:
https://github.com/OpenZeppelin/canton-specs/actions/runs/37567539249/job/112618584215

[bridge-rewards] FAIL: Scan reports 0 featured app rights for br, expected 1

The harness read /v0/featured-apps on Scan right after the featuring transaction committed, before Scan had ingested it. Scan is eventually consistent, so the one-shot read was a race condition.

Changes

  • The bridge harness confirms the FeaturedAppRight on the participant's ACS, then polls Scan until it lists the contract.
  • The traffic-rewards harness gets the same treatment: it keeps the contract id the wallet returns from the self-grant and polls Scan until it lists that right.
  • LocalNet defaults to Splice 0.8.3, the release dars/manifest.yaml already pins.
  • The Splice template ids in both harnesses carry comments that explain their format and link to the definitions at the 0.8.3 commit.

@0xNeshi 0xNeshi self-assigned this Oct 7, 2026

@ericnordelo ericnordelo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@0xNeshi
0xNeshi merged commit 2753040 into main Oct 8, 2026
6 checks passed
@0xNeshi
0xNeshi deleted the fix-traffic-in-scripts branch October 8, 2026 10:02
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