Skip to content

fix: keep package identities in variants (SPEC-T01) - #95

Merged
3alpha merged 1 commit into
mainfrom
fix/spec-t01-variant-identities
Nov 9, 2026
Merged

3alpha merged 1 commit into
mainfrom
fix/spec-t01-variant-identities

Conversation

@3alpha

@3alpha 3alpha commented Nov 9, 2026

Copy link
Copy Markdown
Member

Context

Correct SPEC-T01 from the package specification checks.

Based on the default branch main.

Approach

Keep package identities in the variant manifests and remove the phantom root identity. Preserve every effective variant name and version.

Test instructions

Run the SDK package-spec validation on this branch for all package variants.

Verified locally:

  • SDK loading and manifest/optional-asset schemas pass for all 3 variants.
  • Release-file generation, avatar verification and docker compose config --quiet pass for every variant.
  • Compared effective service settings, image/version pins and package identities with the parent: preserved.
  • git diff --check and replay of the complete PR stack pass.
  • SPEC-T01 findings in this layer: 1 → 0.

Full container images were not built locally.

@3alpha
3alpha merged commit f854e51 into main Nov 9, 2026
4 checks passed
@tropibot

tropibot Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

✅ TropiBot report: SYNC TEST REPORT - PASSED

This CI runs on a real DAppNode using the same RPC calls a user would make to configure a staker setup: execution client, consensus client, web3signer, MEV-boost, and relays.

📦 Package Under Test

📦 Clients Used

Component DNP Name
Execution hoodi-nethermind.dnp.dappnode.eth
Consensus lodestar-hoodi.dnp.dappnode.eth
Web3Signer web3signer-hoodi.dnp.dappnode.eth
MEV Boost mev-boost-hoodi.dnp.dappnode.eth
Network hoodi

🔖 Version Tracking

Client Before After
Execution Nethermind/v2.2.2+be201120-f/linux-x64/dotnet10.0.14 Nethermind/v2.2.2+be201120-f/linux-x64/dotnet10.0.14
Consensus Lodestar/v1.50.1/56cf1ab Lodestar/v1.50.1/56cf1ab

⏱️ Timing

Operation Duration Status
SetStakerConfig 55.6s ✅
PackageInstall 16.7s ✅
WaitForSync 30.5s ✅
Cleanup 7.3s ✅

Total Duration: 1m 59s


📋 View full CI logs

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