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.

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

Powpeg-node Configuration Refactor #301

Merged
merged 30 commits into from
Aug 23, 2024

Conversation

apancorb
Copy link
Contributor

Summary

This change presents a major refactor to how configuration is handled in the powpeg-node. What follows is a list of a few major changes:

  • Dynamically load the difficultyTarget config value from PowHSM blockchainParameters RPC
  • Revisit outdated default config values for PowHSMBookkeepingConfig
  • Refactor and create PowHSMConfig which contains all the configuration handlers related to PowHSM
  • Refactor and create PowpegNodeSystemProperties
  • In-memory configuration files for tests related to powpeg-node
  • Refactor KEY_ID suffix from elements in PowPegNodeKeyId

@apancorb apancorb self-assigned this Jul 31, 2024
@apancorb apancorb marked this pull request as ready for review July 31, 2024 14:28
@apancorb apancorb requested a review from a team as a code owner July 31, 2024 14:28
@marcos-iov marcos-iov force-pushed the powhsm-bookkeping-config-integration branch 2 times, most recently from 63a2edd to 9c99021 Compare August 9, 2024 14:53
@marcos-iov marcos-iov force-pushed the powhsm-bookkeping-config-integration branch from 9c99021 to 461dd1d Compare August 14, 2024 15:16
apancorb and others added 23 commits August 23, 2024 10:05
refactor(config): rename local variable in PowHSM config test
refactor(config): fix typos in powHSM config tests
refactor(config): add sonar comments to refactor of powpeg-node config

refactor(config): resolve comments

refactor(btcreleaseclient): fix sonar warning
refactor(config): resolve comments
feat(config): nit rebase changes

feat(config): sonar comments
@marcos-iov marcos-iov force-pushed the powhsm-bookkeping-config-integration branch from b5fc935 to 1b6f810 Compare August 23, 2024 13:07
Copy link

sonarcloud bot commented Aug 23, 2024

@marcos-iov
Copy link
Collaborator

pipeline:run

@marcos-iov marcos-iov merged commit fe97bd7 into master Aug 23, 2024
7 checks passed
@marcos-iov marcos-iov deleted the powhsm-bookkeping-config-integration branch August 23, 2024 15:10
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