Skip to content

Endorsements#44

Merged
redpanda-f merged 5 commits intoFilOzone:mainfrom
wjmelements:endorsements-address
Jan 27, 2026
Merged

Endorsements#44
redpanda-f merged 5 commits intoFilOzone:mainfrom
wjmelements:endorsements-address

Conversation

@wjmelements
Copy link
Contributor

Closes #41
Dependencies FilOzone/synapse-sdk#568 FilOzone/filecoin-services#399
Reviewer @redpanda-f

Context

Filecoin Services PR FilOzone/filecoin-services#386
Synapse PR FilOzone/synapse-sdk#553
FOC Task: #41
synapse now depends on an endorsements contract.
It uses this contract to prefer reputable service providers and improve durability until a repair solution can be developed.
This contract must be deployed into devnet and its address must be provided to synapse.

Changes

  • plumb endorsements from deploy-all into e2e script via contract_addresses.json
  • update synapse-sdk and filecoin-services

@FilOzzy FilOzzy added this to FOC Jan 27, 2026
@github-project-automation github-project-automation bot moved this to 📌 Triage in FOC Jan 27, 2026
@wjmelements
Copy link
Contributor Author

 INFO src/commands/start/foc_deployer/mod.rs:334: Using nonce: 10
 INFO src/commands/start/foc_deployer/mod.rs:334: Error: server returned an error response: error code 1: minimum expected nonce is 11, got 10: message nonce too low
 INFO src/commands/start/foc_deployer/mod.rs:334: Error: Failed to extract ProviderIdSet address
 INFO src/commands/start/foc_deployer/mod.rs:336: Deployment may have failed or output format changed

@wjmelements
Copy link
Contributor Author

Some commands update the nonce before; some update the nonce after. Thereby 6af8698 broke the script. I will need to patch it there.

wjmelements added a commit to FilOzone/filecoin-services that referenced this pull request Jan 27, 2026
Reviewer @rvagg

[Context](FilOzone/foc-devnet#44 (comment))
Updating nonce after a command is most common in this script. There are
some exceptions though, and such an inversion can cause wrong nonce.
#### Changes
* always update nonce after command, never before
#### Test Plan
`SKIP_LOAD_DEPLOYMENTS=true tools/deploy-all-warm-storage.sh`
@wjmelements
Copy link
Contributor Author

Next error is

❌ Error: ENDORSEMENT_ADDRESS is not defined\nReferenceError: ENDORSEMENT_ADDRESS is not defined\n    at main (file:///synapse-sdk/utils/example-storage-e2e.js:126:44)

@github-project-automation github-project-automation bot moved this from 📌 Triage to ✔️ Approved by reviewer in FOC Jan 27, 2026
@redpanda-f redpanda-f merged commit 9136978 into FilOzone:main Jan 27, 2026
2 checks passed
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FOC Jan 27, 2026
@rvagg
Copy link
Contributor

rvagg commented Jan 27, 2026

eggselent!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

Add endorsements contract to devnet

3 participants