Skip to content

Conversation

@suchapalaver
Copy link
Collaborator

Add support for configuring multiple operator mnemonics to fix 'no attestation' errors for allocations created with different operator keys (e.g., after key rotation or migration).

Changes:

  • Add config field alongside existing
  • Update attestation signer to try all configured mnemonics when deriving wallet for an allocation
  • Maintain full backward compatibility - existing configs work unchanged

Signed off by Joseph Livesey joseph@semiotic.ai

   Add support for configuring multiple operator mnemonics to fix 'no attestation' errors for allocations created with different operator
   keys (e.g., after key rotation or migration).

   Changes:
   - Add  config field alongside existing
   - Update attestation signer to try all configured mnemonics when
     deriving wallet for an allocation
   - Maintain full backward compatibility - existing configs work unchanged

   Configuration example:
@suchapalaver suchapalaver marked this pull request as ready for review December 4, 2025 19:39
@coveralls
Copy link

Pull Request Test Coverage Report for Build 19941587898

Details

  • 191 of 221 (86.43%) changed or added relevant lines in 5 files are covered.
  • 2 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.3%) to 63.647%

Changes Missing Coverage Covered Lines Changed/Added Lines %
crates/monitor/src/attestation.rs 15 17 88.24%
crates/service/src/service/router.rs 6 11 54.55%
crates/config/src/config.rs 124 135 91.85%
crates/attestation/src/lib.rs 45 57 78.95%
Files with Coverage Reduction New Missed Lines %
crates/attestation/src/lib.rs 1 85.71%
crates/monitor/src/attestation.rs 1 87.5%
Totals Coverage Status
Change from base Build 19875961079: 0.3%
Covered Lines: 9687
Relevant Lines: 15220

💛 - Coveralls

@suchapalaver suchapalaver merged commit f42fcbc into main Dec 4, 2025
12 checks passed
@suchapalaver suchapalaver deleted the suchapalaver/feat/support-multiple-operator-mnemonics-for-attestation-signing branch December 4, 2025 20:32
@github-actions github-actions bot mentioned this pull request Dec 4, 2025
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.

4 participants