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

Suppress ConfigUtils warning on Kubernetes startup #10043

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

steven-sheehy
Copy link
Member

Description:

Suppress ConfigUtils warning on startup in Kubernetes

Related issue(s):

Notes for reviewer:

Example logs:

2025-01-03T21:15:31.381Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc was requested, but not found in namespace : dev 
2025-01-03T21:15:31.384Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc-kubernetes was requested, but not found in namespace : dev 
2025-01-03T21:15:31.436Z  INFO main c.h.m.g.GrpcApplication The following 1 profile is active: "kubernetes" 
2025-01-03T21:15:39.151Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc was requested, but not found in namespace : dev 
2025-01-03T21:15:39.151Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc-kubernetes was requested, but not found in namespace : dev 
2025-01-03T21:15:39.166Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc was requested, but not found in namespace : dev 
2025-01-03T21:15:39.166Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc-kubernetes was requested, but not found in namespace : dev 
2025-01-03T21:15:39.182Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc was requested, but not found in namespace : dev 
2025-01-03T21:15:39.182Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc-kubernetes was requested, but not found in namespace : dev 
2025-01-03T21:15:39.196Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc was requested, but not found in namespace : dev 
2025-01-03T21:15:39.196Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc-kubernetes was requested, but not found in namespace : dev 
2025-01-03T21:15:39.208Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc was requested, but not found in namespace : dev 
2025-01-03T21:15:39.208Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc-kubernetes was requested, but not found in namespace : dev 
2025-01-03T21:15:39.223Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc was requested, but not found in namespace : dev 
2025-01-03T21:15:39.223Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc-kubernetes was requested, but not found in namespace : dev 
2025-01-03T21:15:39.236Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc was requested, but not found in namespace : dev 
2025-01-03T21:15:39.236Z  WARN main o.s.c.k.c.c.ConfigUtils sourceName : hedera-mirror-grpc-kubernetes was requested, but not found in namespace : dev 
2025-01-03T21:15:39.620Z  INFO main c.h.m.g.c.GrpcHealthIndicator Started gRPC server on *:5600 
2025-01-03T21:15:39.638Z  INFO main c.h.m.g.GrpcApplication Started GrpcApplication in 10.534 seconds (process running for 10.966) 

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@steven-sheehy steven-sheehy added the bug Type: Something isn't working label Jan 3, 2025
@steven-sheehy steven-sheehy added this to the 0.121.0 milestone Jan 3, 2025
@steven-sheehy steven-sheehy self-assigned this Jan 3, 2025
@steven-sheehy steven-sheehy requested a review from a team as a code owner January 3, 2025 22:01
Copy link

sonarqubecloud bot commented Jan 3, 2025

@steven-sheehy steven-sheehy changed the title Suppress ConfigUtils warning on startup in Kubernetes Suppress ConfigUtils warning on Kubernetes startup Jan 3, 2025
Copy link

codecov bot commented Jan 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.23%. Comparing base (41e24c8) to head (d5033dc).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main   #10043   +/-   ##
=========================================
  Coverage     92.23%   92.23%           
  Complexity     7856     7856           
=========================================
  Files           956      956           
  Lines         32918    32918           
  Branches       4153     4153           
=========================================
  Hits          30363    30363           
  Misses         1579     1579           
  Partials        976      976           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@jnels124 jnels124 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@xin-hedera xin-hedera left a comment

Choose a reason for hiding this comment

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

LGTM

@steven-sheehy steven-sheehy merged commit 12c429e into main Jan 3, 2025
35 of 36 checks passed
@steven-sheehy steven-sheehy deleted the supress-configutils-warning branch January 3, 2025 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Type: Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants