Skip to content

ci: drop mount-path for devnet verify build#75

Merged
dev-jodee merged 1 commit intomainfrom
ci/devnet-verify-drop-mount-path
Apr 30, 2026
Merged

ci: drop mount-path for devnet verify build#75
dev-jodee merged 1 commit intomainfrom
ci/devnet-verify-drop-mount-path

Conversation

@dev-jodee
Copy link
Copy Markdown
Collaborator

Summary

  • Remove mount-path: program from the devnet verify-build step.

Root Cause

solana-verify expects a Cargo.lock at the directory it mounts/builds. With mount-path: program it looked for program/Cargo.lock, which does not exist — the workspace Cargo.lock lives at the repo root. Pre-flight failed with:

Mount directory must contain a Cargo.lock file
Error: Error verifying program: Missing Cargo.lock file at
'/tmp/solana-verify/<id>/subscriptions/program/Cargo.lock'

Dropping mount-path lets solana-verify use the repo root and resolve the program via --library-name subscriptions, matching how the earlier Build verified program step invokes solana-verify.

Test Plan

  • Trigger Release workflow against devnet from this branch.
  • Confirm Verify build (devnet) clears pre-flight, builds, and writes the verification PDA.

Failing run for reference: https://github.com/solana-program/subscriptions/actions/runs/25183322572

solana-verify expects Cargo.lock at the mount-path it builds from. We
were passing `mount-path: program`, but the workspace Cargo.lock lives at
the repo root, so the verifier failed pre-flight with:

  Mount directory must contain a Cargo.lock file
  Error: Error verifying program: Missing Cargo.lock file at
  '/tmp/solana-verify/<id>/subscriptions/program/Cargo.lock'

Drop the mount-path input so solana-verify uses the repo root (where the
workspace Cargo.lock lives) and resolves the program via
`--library-name subscriptions`. This matches how the earlier
`Build verified program` step invokes solana-verify.
@github-actions
Copy link
Copy Markdown

Compute Unit Report

Instruction Samples Min CUs Max CUs Avg CUs Est Cost (Low) [SOL] Est Cost (Med) [SOL] Est Cost (High) [SOL]
cancel_subscription 11 1805 2118 2003 0.000005000 0.000005080 0.000006001
close_subscription_authority 7 1881 1916 1886 0.000005000 0.000005075 0.000005943
create_fixed_delegation 38 3621 12647 5229 0.000005001 0.000005209 0.000007614
create_plan 84 3540 15553 5471 0.000005001 0.000005218 0.000007735
create_recurring_delegation 27 3640 11173 5535 0.000005001 0.000005221 0.000007767
delete_plan 8 416 416 416 0.000005000 0.000005016 0.000005208
init_subscription_authority 142 8931 18351 11878 0.000005003 0.000005475 0.000010939
revoke_delegation 19 318 585 420 0.000005000 0.000005016 0.000005210
subscribe 21 6666 18690 8890 0.000005002 0.000005355 0.000009445
transfer_fixed 6 10105 11605 11104 0.000005003 0.000005444 0.000010552
transfer_recurring 17 10212 16212 11743 0.000005003 0.000005469 0.000010871
transfer_subscription 10 10495 17946 12000 0.000005003 0.000005480 0.000011000
update_plan 21 424 500 474 0.000005000 0.000005018 0.000005237

Generated: 2026-04-30

@dev-jodee dev-jodee merged commit c8777ec into main Apr 30, 2026
8 checks passed
@dev-jodee dev-jodee deleted the ci/devnet-verify-drop-mount-path branch April 30, 2026 19:31
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