Skip to content

feat: add isAggregator flag to validator configuration#108

Draft
ch4r10t33r wants to merge 2 commits intomainfrom
feat/add-aggregator-flag
Draft

feat: add isAggregator flag to validator configuration#108
ch4r10t33r wants to merge 2 commits intomainfrom
feat/add-aggregator-flag

Conversation

@ch4r10t33r
Copy link
Contributor

Summary

Add support for configuring nodes as aggregators through validator-config.yaml. This allows selective designation of nodes to perform aggregation duties by setting isAggregator: true in the validator configuration.

Changes

  • Add isAggregator field (default: false) to all validators in both local and ansible configs
  • Update parse-vc.sh to extract and export isAggregator flag
  • Modify all client command scripts (zeam, ream, qlean, lighthouse, lantern, grandine, ethlambda) to pass --is-aggregator flag when enabled
  • Add isAggregator status to node information output in parse-vc.sh

Usage

To designate a node as an aggregator, simply set isAggregator: true in the validator-config.yaml:

validators:
  - name: "zeam_0"
    privkey: "..."
    enrFields:
      ip: "..."
      quic: 9001
    metricsPort: 9095
    isAggregator: true  # This node will be an aggregator
    count: 1

The spin-node.sh script will automatically detect this configuration and pass the --is-aggregator flag to the client.

Test plan

  • Verify that nodes with isAggregator: false start without the --is-aggregator flag
  • Verify that nodes with isAggregator: true receive the --is-aggregator flag
  • Test with both local and ansible deployment modes
  • Confirm backward compatibility (nodes without the field default to false)

Add support for configuring nodes as aggregators through validator-config.yaml.
This allows selective designation of nodes to perform aggregation duties by
setting isAggregator: true in the validator configuration.

Changes:
- Add isAggregator field (default: false) to all validators in both local and ansible configs
- Update parse-vc.sh to extract and export isAggregator flag
- Modify all client command scripts to pass --is-aggregator flag when enabled
- Add isAggregator status to node information output
--log-level debug \
--hash-sig-key-dir $configDir/hash-sig-keys"
--hash-sig-key-dir $configDir/hash-sig-keys \
$aggregator_flag"
Copy link
Member

@g11tech g11tech Feb 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

node is not an agrregator based on the flag but based on the validator id which is an aggregator which is parsed via the validator-config to annotated_validaors.yaml

but the main question is how do we assign aggregator to the validator of which of the nodes.

the solution for that is: all the nodes who want to run the aggregator as reflected via validator-config, one of their validator ids need to be picked and flagged true as ggregator in anootated validators

also validator-config needs to now carrynum subnets, which will also need to be inserted in config.yaml

for now this number is 1

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but this flag is determined/initialised from the validator-config.yaml. Having said that, this PR is still in draft and I am waiting for the zeam PR to be merged before I can work and complete this.

@ch4r10t33r ch4r10t33r marked this pull request as draft February 8, 2026 12:14
@ch4r10t33r ch4r10t33r force-pushed the feat/add-aggregator-flag branch from 951526d to c571bc7 Compare February 8, 2026 20:26
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.

2 participants