From a70605a9e6801f15f9346163b90eb3131c476a06 Mon Sep 17 00:00:00 2001 From: Ying Li Date: Thu, 8 Oct 2015 15:42:14 -0700 Subject: [PATCH] Shorten health check name and remove redundant logging Signed-off-by: Ying Li --- cmd/notary-signer/main.go | 2 +- signer/api/rpc_api.go | 6 ------ 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/cmd/notary-signer/main.go b/cmd/notary-signer/main.go index 052a540820..b4d1c227fb 100644 --- a/cmd/notary-signer/main.go +++ b/cmd/notary-signer/main.go @@ -150,7 +150,7 @@ func main() { } health.RegisterPeriodicFunc( - "DB connectable and valid", keyStore.HealthCheck, time.Second*60) + "DB operational", keyStore.HealthCheck, time.Second*60) cryptoService := cryptoservice.NewCryptoService("", keyStore) diff --git a/signer/api/rpc_api.go b/signer/api/rpc_api.go index d810fbe271..c3f16e6e54 100644 --- a/signer/api/rpc_api.go +++ b/signer/api/rpc_api.go @@ -110,9 +110,6 @@ func (s *KeyManagementServer) GetKeyInfo(ctx context.Context, keyID *pb.KeyID) ( //CheckHealth returns the HealthStatus with the service func (s *KeyManagementServer) CheckHealth(ctx context.Context, v *pb.Void) (*pb.HealthStatus, error) { - logger := ctxu.GetLogger(ctx) - logger.Debug("CheckHealth: Returning HealthStatus for KeyManagementServer") - return &pb.HealthStatus{ Status: s.HealthChecker(), }, nil @@ -151,9 +148,6 @@ func (s *SignerServer) Sign(ctx context.Context, sr *pb.SignatureRequest) (*pb.S //CheckHealth returns the HealthStatus with the service func (s *SignerServer) CheckHealth(ctx context.Context, v *pb.Void) (*pb.HealthStatus, error) { - logger := ctxu.GetLogger(ctx) - logger.Debug("CheckHealth: Returning HealthStatus for SignerServer") - return &pb.HealthStatus{ Status: s.HealthChecker(), }, nil