Issue #2516
closedworker heartbeat log statement mishandles timezone
Description
From bugzilla:
Dec 22 14:03:55 pmoravec-sat62-rhel6 pulp: pulp.server.async.worker_watcher:DEBUG: 'worker-heartbeat' sent at time 2016-12-22 14:03:55.439300 from reserved_resource_worker-3@pmoravec-sat62-rhel6.gsslab.brq.redhat.com, received at time: 2016-12-22 13:03:55.450517
Saying that:
- worker sent heartbeat at 14:03
- the heartbeat was received at 13:03
The machine was set to GTM+1.
The code is here:
https://github.com/pulp/pulp/blob/pulp-2.11.0-1/server/pulp/server/async/worker_watcher.py#L27
Updated by bmbouter over 7 years ago
We generate our own timestamps and only use the worker name from the heartbeat. Instead of logging the heartbeat we receive, we should create our own heartbeat log line with the worker name and timestamp that we are using (the one we generate).
Updated by bizhang over 7 years ago
- Sprint/Milestone set to 31
- Severity changed from 2. Medium to 1. Low
- Triaged changed from No to Yes
Updated by dalley over 7 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to dalley
Updated by dalley over 7 years ago
Unassigning myself for grooming - jumped the gun a bit
Updated by dalley about 7 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to dalley
Assigning it to myself - will be resolved by the fix for Issue 2519 - https://github.com/pulp/pulp/pull/2922
No further effort needed
Updated by dalley about 7 years ago
- Status changed from ASSIGNED to POST
Added by dalley about 7 years ago
Added by dalley about 7 years ago
Revision fd19f890 | View on GitHub
Workers write their own hearbeat records to database.
All workers will write their own records to the database instead of relying on pulp_celerybeat to do so for them using celery heartbeats.
This patch makes use of the Consumer blueprint that celery runs at the start time of a worker. An extra boot step has been added which sets a timer to periodically update the worker record in the database.
http://docs.celeryproject.org/en/master/userguide/extending.html https://groups.google.com/d/msg/celery-users/3fs0ocREYqw/C7U1lCAp56sJ
closes #2519 https://pulp.plan.io/issues/2519 closes #2516 https://pulp.plan.io/issues/2516
Updated by dalley about 7 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp|fd19f890433d0ccdb75fddc1bce5ff9c600ce94b.
Updated by pcreech almost 7 years ago
- Status changed from 5 to CLOSED - CURRENTRELEASE
Updated by dalley over 6 years ago
- Status changed from CLOSED - CURRENTRELEASE to CLOSED - WONTFIX
See discussion in the associated bugzilla. Revisited due to FailedQA, but we recognized that it is perfectly fine to remain the way it is currently, and isn't an issue.
Updated by dalley over 6 years ago
- Status changed from CLOSED - WONTFIX to CLOSED - NOTABUG
I suppose this should actually be "NOTABUG" instead of "WONTFIX"
Updated by bmbouter about 6 years ago
- Sprint changed from Sprint 16 to Sprint 14
Workers write their own hearbeat records to database.
All workers will write their own records to the database instead of relying on pulp_celerybeat to do so for them using celery heartbeats.
This patch makes use of the Consumer blueprint that celery runs at the start time of a worker. An extra boot step has been added which sets a timer to periodically update the worker record in the database.
http://docs.celeryproject.org/en/master/userguide/extending.html https://groups.google.com/d/msg/celery-users/3fs0ocREYqw/C7U1lCAp56sJ
closes #2519 https://pulp.plan.io/issues/2519 closes #2516 https://pulp.plan.io/issues/2516