diff --git a/source/server/server.cc b/source/server/server.cc index b948a32c13842..70c234a1362d9 100644 --- a/source/server/server.cc +++ b/source/server/server.cc @@ -195,7 +195,7 @@ void InstanceImpl::flushStatsInternal() { server_stats_->days_until_first_cert_expiring_.set( sslContextManager().daysUntilFirstCertExpires()); server_stats_->state_.set( - enumToInt(Utility::serverState(initManager().state(), !healthCheckFailed()))); + enumToInt(Utility::serverState(initManager().state(), healthCheckFailed()))); InstanceUtil::flushMetricsToSinks(config_.statsSinks(), stats_store_); // TODO(ramaraochavali): consider adding different flush interval for histograms. if (stat_flush_timer_ != nullptr) { diff --git a/test/server/server_test.cc b/test/server/server_test.cc index b5044f1b87c3c..9809dec3093c6 100644 --- a/test/server/server_test.cc +++ b/test/server/server_test.cc @@ -315,6 +315,7 @@ TEST_P(ServerInstanceImplTest, EmptyShutdownLifecycleNotifications) { server_thread->join(); // Validate that initialization_time histogram value has been set. EXPECT_TRUE(stats_store_.histogram("server.initialization_time").used()); + EXPECT_EQ(0L, TestUtility::findGauge(stats_store_, "server.state")->value()); } TEST_P(ServerInstanceImplTest, LifecycleNotifications) {