Skip to content

decouple config for different collectors#45

Merged
junaidk merged 3 commits intomainfrom
netx-522-decouple-configs-validations
Jul 24, 2025
Merged

decouple config for different collectors#45
junaidk merged 3 commits intomainfrom
netx-522-decouple-configs-validations

Conversation

@junaidk
Copy link
Contributor

@junaidk junaidk commented Jul 18, 2025

No description provided.

prometheus.GaugeValue,
balance,
[]string{account.Address, (*wb.RPCs)[account.ChainName].ChainID, account.Denom, status}...,
[]string{account.Address, (*wb.RPCs)[account.ChainName].ChainID, account.Denom, status, strings.Join(account.Tags, ",")}...,
Copy link
Contributor

Choose a reason for hiding this comment

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

tags must defineable in the configs e.g.

accounts:
  - address: archway1avwvqzu9gv86g5fxx5p2xqe0w33wklt27jusdrhszwccnfnxx0rsmzz8nu
    chainName: archway
    denom: aarch
    tags: tag1,tag2

Copy link
Contributor

Choose a reason for hiding this comment

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

my bad, it expect a list of strings

accounts:

  • address: archway1avwvqzu9gv86g5fxx5p2xqe0w33wklt27jusdrhszwccnfnxx0rsmzz8nu
    chainName: archway
    denom: aarch
    tags: ["tag1","tag2"]

@junaidk junaidk merged commit 6cbecca into main Jul 24, 2025
3 checks passed
@junaidk junaidk deleted the netx-522-decouple-configs-validations branch July 24, 2025 13:23
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