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

Centralize Record Configuration Logging #471

Closed
Tracked by #195
mattp-swirldslabs opened this issue Jan 9, 2025 · 0 comments · Fixed by #501
Closed
Tracked by #195

Centralize Record Configuration Logging #471

mattp-swirldslabs opened this issue Jan 9, 2025 · 0 comments · Fixed by #501
Assignees
Labels
Block Node Issues/PR related to the Block Node. Feature Enhancement Enhancing an existing feature driven by business requirements. Typically backwards compatible.
Milestone

Comments

@mattp-swirldslabs
Copy link
Contributor

mattp-swirldslabs commented Jan 9, 2025

As an Administrator
I want to see all properties of the BN runtime configuration logged at boot time
So that I can confirm the configuration on a node or by inspecting the logs on a remote dashboard for troubleshooting

Tech Notes

  • After the Configuration object is built in Server.java, we should be able to delegate it to a central configuration logging component - property configuration precedence should be resolved at that time
  • We should provide an entry in logging.properties to toggle config logging output for the config logging component.
  • The existing configuration management tests for mismatches in ServerMappedConfigurationSourceInitializerTest and fails if detected.
  • BlockNodeConfigExtension.java holds a list of all the Config record classes in the system
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Block Node Issues/PR related to the Block Node. Feature Enhancement Enhancing an existing feature driven by business requirements. Typically backwards compatible.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant