Skip to content

ci: add Discord notification to sync workflow#71

Merged
pablomendezroyo merged 2 commits intomainfrom
copilot/add-notify-job-to-sync
Feb 6, 2026
Merged

ci: add Discord notification to sync workflow#71
pablomendezroyo merged 2 commits intomainfrom
copilot/add-notify-job-to-sync

Conversation

Copy link
Contributor

Copilot AI commented Feb 6, 2026

Adds Discord notification job to the sync workflow, matching the implementation already deployed in the Geth generic repository.

Changes

  • Added notify job to .github/workflows/sync.yml that triggers on sync completion (success, failure, or cancellation)
  • Uses reusable workflow dappnode/staker-test-util/.github/workflows/notify-discord.yml@main
  • Configured with test_type: "sync" and inherits repository secrets
  notify:
    needs: sync
    if: always() && needs.sync.result != 'skipped'
    uses: dappnode/staker-test-util/.github/workflows/notify-discord.yml@main
    with:
      result: ${{ needs.sync.result }}
      test_type: "sync"
    secrets: inherit
Original prompt

Context

A change was already implemented in the Geth generic repo that adds a Discord notification job to the sync workflow:
dappnode/DAppNodePackage-geth-generic@7552d5b

This same change needs to be applied to this repository.

Task

Add a new notify job at the END of the .github/workflows/sync.yml file.

The job to add is:

  notify:
    needs: sync
    if: always() && needs.sync.result != 'skipped'
    uses: dappnode/staker-test-util/.github/workflows/notify-discord.yml@main
    with:
      result: ${{ needs.sync.result }}
      test_type: "sync"
    secrets: inherit

Requirements

  1. Open the file .github/workflows/sync.yml
  2. Add the notify job at the END of the workflow file (after all existing jobs)
  3. Do NOT modify any other jobs
  4. Do NOT refactor workflow formatting
  5. Do NOT reorder existing jobs
  6. The notify job must depend on sync
  7. Ensure indentation remains valid YAML

Branch and Commit

  • Create branch: add-sync-notify-job
  • Commit message: ci: add discord notify job to sync workflow
  • Open PR to main
  • Do NOT merge the PR

This pull request was created from Copilot chat.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Marketen <36164126+Marketen@users.noreply.github.com>
Copilot AI changed the title [WIP] Add notify job to sync workflow ci: add Discord notification to sync workflow Feb 6, 2026
Copilot AI requested a review from Marketen February 6, 2026 15:16
@pablomendezroyo pablomendezroyo marked this pull request as ready for review February 6, 2026 15:24
@pablomendezroyo pablomendezroyo merged commit 67087db into main Feb 6, 2026
1 check passed
@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

✅ 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. The self-hosted runner is pre-loaded with synced (or near-synced) execution client volumes, enabling fast sync and attestation tests with any client combination. Consensus clients use checkpoint sync, so no pre-synced volume is required.

Sync Test: Verifies that both execution and consensus clients reach a synced state. This test ensures the staker configuration is valid and clients can synchronize with the network.

📦 Clients Used

Component DNP Name DNP Version
Execution hoodi-erigon.dnp.dappnode.eth 0.1.1
Consensus prysm-hoodi.dnp.dappnode.eth 0.1.5
Web3Signer web3signer-hoodi.dnp.dappnode.eth 0.1.3
MEV Boost mev-boost-hoodi.dnp.dappnode.eth 0.1.1
Network hoodi

🔖 Version Tracking

Execution Client Versions

Stage Version
Before Install erigon/3.2.0/linux-amd64/go1.24.7
After Install erigon/3.3.7/linux-amd64/go1.25.6

Consensus Client Versions

Stage Version
Before Install Prysm/v7.0.1 (linux amd64)
After Install Prysm/v7.0.1 (linux amd64)

⏱️ Timing Measurements

Environment Setup

Operation Duration Status
SetStakerConfig 50.205s
PackageInstall 40.478s

Test Execution

Operation Duration Status
WaitForBeaconchainSync 36.221s
WaitForExecutionSync 0s

Total Duration: 2m20s

🔴 Container Error Logs

⚠️ Showing up to 3 error lines per container. See CI logs for complete details.

DAppNodePackage-beacon-chain.prysm-hoodi.dnp.dappnode.eth:

2026-02-06T15:49:24.442918666Z time="2026-02-06 15:49:24.44" level=warning msg="Could not update head" error="head at slot 2348224 with weight 123816 is not eligible, finalizedEpoch, justified Epoch 73380, 73381 != 73382, 73382" prefix=blockchain
2026-02-06T15:49:24.523325796Z time="2026-02-06 15:49:24.52" level=warning msg="Could not update head" error="head at slot 2348224 with weight 243837 is not eligible, finalizedEpoch, justified Epoch 73380, 73381 != 73382, 73382" prefix=blockchain
2026-02-06T15:49:24.610922905Z time="2026-02-06 15:49:24.61" level=warning msg="Could not update head" error="head at slot 2348224 with weight 365998 is not eligible, finalizedEpoch, justified Epoch 73380, 73381 != 73382, 73382" prefix=blockchain

📋 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.

3 participants