Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add chain-watcher monitoring config support #53

Merged
merged 6 commits into from
Dec 20, 2024
Merged

Add chain-watcher monitoring config support #53

merged 6 commits into from
Dec 20, 2024

Conversation

vovacha
Copy link

@vovacha vovacha commented Dec 19, 2024 •
edited
Loading

Changes

  • Add dependencies: w3f/monitoring-types and w3f/monitoring-config to process Config V2 (monitoring-platform format)
  • Update GitConfigLoader interface with an optional configVersion
  • Update GitLabPrivate implementation to be backward compatible with Config V1
  • Add MonitoringConfigVersion enum
  • Extend InputConfig with optional key monitoringConfigVersion

TODO:

  • Test
  • Update configs with monitoringConfigVersion
    • helmfile.d
    • helm charts
    • config samples

@vovacha vovacha requested a review from ironoa December 19, 2024 10:31
@vovacha vovacha self-assigned this Dec 19, 2024
@ironoa ironoa merged commit 3104b5c into master Dec 20, 2024
5 checks passed
@ironoa ironoa deleted the feature/chain-watcher-config branch December 20, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ironoa ironoa ironoa approved these changes

Assignees

@vovacha vovacha

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vovacha @ironoa