hyperledger / fabric-x-committer

92%

main: 91%

LAST BUILD ON BRANCH default-viper

branch: default-viper

CHANGE BRANCH

x

Reset

• default-viper
• apt-ci-fix
• backoff-v5
• bccsp-race
• bench-report-metrics
• bft-delivery
• bob-rules
• buffer-race
• build-unify
• channel-race
• ci-gotestsum
• cli-refactor
• client-test-race
• client-tls-bug
• commit-config-immediately
• config-docs-yaml
• connection-test-dep
• coveralls
• cryptogen-recreate-support
• default-test-node-startup-setup
• deliver-check-block-order
• deliver-seek-tls-cert
• deliver-seek-tls-cert-fix
• dependabot/go_modules/go_modules-9c5197dcb8
• dependencygraph-race
• deployment-strategy
• doc-loadgen-artifacts
• dynamic-root-ca
• dynamic-tls
• enable-pprof
• expose-ledger-api
• extract-prometheus-startup-and-grpc-server-start
• fabric-lib-go-v1.1.3
• fabric-logging
• fix-connection-reset
• fix-container-image
• fix-container-test-macos
• fix-db-test
• fix-falky-svmupdate-test
• fix-flaky
• fix-flaky-conn-metrics
• fix-flaky-loadgen-coord
• fix-master-tserver-rf
• fix-port-container-test
• fix-release-path
• fix-sign
• fix-test-node-with-insecure
• fix-waiting-tx-count
• flaky-test-orderer
• getpeersmspdirs-no-hardcode
• go-fix
• grpc-logger-fix
• grpcservice-refactor
• healthz-check
• ledger-service-wait-behaviour
• lifecycle-policy
• limit-block-apis
• limit-inputs-block-apis
• load-config-block-certs
• loadgen-manual-msp
• loadgen-remove-ctx
• main
• make-ci-local
• make-fixes
• makefile-test-flags
• metrics-tls
• metrics-url-race
• migrate-to-postgres-official-image
• minimize-logs
• mkdocs
• mock-orderer-features
• mock-orderer-stream-holder
• mock-verifier-race
• move-retry-package
• namespace-policy-docs
• notifier-metrics
• optimize-map-block
• orderer-config-race
• pgx5
• postgres-renaming
• qs-pen-test-fix
• race-detection
• rate-limit-sidecar
• refactor-channel
• refactor-cmd
• rename-materials
• rename-util-files
• review-ai-prompt
• rm-gpl-tools
• rm-reployment
• rm-unused
• root-ca-workaround-fix
• sidecar-block-store-code-fix
• sidecar-limit-active-txids-notifier
• sidecar-race
• sidecar-remove-chan-id
• split-tablets
• sustain-config
• sync-block
• template-tls-per-endpoint
• test-container-port-binding
• test-coverage-connection
• test-coverage-improve
• test-coverage-latency
• test-coverage-tests
• test-limiter-max-wait
• test-minor-fixes
• test-query-status
• test-tls-flatten
• testcrypto-race
• testcrypto-refactor
• tls-docs-update
• update-common-dep
• update-to-latest-common
• update-to-yugabytedb-latest-lts
• upgrade-pg
• vc-race
• verifier-test-remove-gomega
• worktree-notification-docs

Build # 24121558350

Build Type

Pull #528

github

Committed by cendhu

Commit Message

[viper] Centralize configuration defaults in service packages

  Move hardcoded default values from cmd/config/viper.go to their respective
  service packages (coordinator, verifier, vc, query). Export these
  values as constants so they can be reused by the viper initialization logic.

  This consolidation reduces duplication and ensures that the defaults used
  by the CLI stay in sync with the service implementations.

  Key changes:
   - Exported Default... constants in service/coordinator, service/verifier, service/vc, and service/query.
   - Updated cmd/config/viper.go to use these constants.
   - Updated cmd/config/app_config_test.go to align with the centralized defaults.

Signed-off-by: Senthilnathan <cendhu@gmail.com>

Pull Request Pull Request #528: [viper] Centralize configuration defaults in service packages

Coverage Stats

54 of 57 new or added lines in 5 files covered. (94.74%)

4 existing lines in 4 files now uncovered.

9981 of 10905 relevant lines covered (91.53%)

42677.35 hits per line