Skip to content

Conversation

@johnsimons
Copy link
Member

@johnsimons johnsimons commented Sep 10, 2024

Symptoms

In ServicePulse, we display "no plugin installed" in the heartbeats screens when an instance has no heartbeat data.

Who's affected

Customers using ServicePulse

Root cause

We are initializing the heartbeat data with a minimum date, which ends up failing to be rendered in ServicePulse.

@johnsimons johnsimons self-assigned this Sep 10, 2024
HeartbeatInformation = lastSeen.HasValue ? new HeartbeatInformation
{
ReportedStatus = status == HeartbeatStatus.Alive ? HeartbeatMonitoringStatus.Beating : HeartbeatMonitoringStatus.Dead,
LastReportAt = lastSeen ?? DateTime.MinValue
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
LastReportAt = lastSeen ?? DateTime.MinValue
LastReportAt = lastSeen.Value

@johnsimons johnsimons merged commit 34fca68 into master Sep 10, 2024
@johnsimons johnsimons deleted the john/no_plugin branch September 10, 2024 04:54
@johnsimons johnsimons added this to the 5.9.0 milestone Sep 10, 2024
@adamralph adamralph mentioned this pull request Sep 11, 2024
@johnsimons johnsimons changed the title The frontend needs to be aware when no plugin is sending heartbeats Inform ServicePulse that no plugin is sending heartbeats Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants