Story #2659
closedAs a User I would like Pulp3 to not show me offline workers in the status api
100%
Description
Assumption: Pulp workers that are missing should not be shown in the status API.
In Pulp3, the Worker model has an 'online' boolean which is meant to allow Pulp to store records about workers even after they have gone offline. Additionally workers should be considered offline if their heartbeat time is older than 30 seconds. We need to filter for both cases because of the following failure scenario:
1) a working Pulp system with exactly 1 worker that has pid 1234
2) kill -9 1234
In this case, the DB record will show 'online' without the 30 second part of the filter.
To resolve this we should:
1) add a method called online_workers
to the TaskManager which returns a queryset with the two filters applied as described above
2) refactor the get_unreserved_worker method to use online_workers
instead. This will cause us to stay DRY w.r.t. this filtering.
3) Have the ViewSet of the /status/ API only return workers from the online_workers
queryset.
4) Update the docstrings so that those updates will flow to the docs
Updated by bizhang over 7 years ago
- Subject changed from Stale worker documents present in the db for Pulp3 to As a User I would like Pulp3 to not show me stale worker in the status api
Updated by bizhang over 7 years ago
- Tracker changed from Issue to Story
- Subject changed from As a User I would like Pulp3 to not show me stale worker in the status api to As a User I would like Pulp3 to not show me stale workers in the status api
- % Done set to 0
Updated by bmbouter about 7 years ago
- Subject changed from As a User I would like Pulp3 to not show me stale workers in the status api to As a User I would like Pulp3 to not show me offline workers in the status api
- Description updated (diff)
- Sprint Candidate changed from No to Yes
Updated by mhrivnak about 7 years ago
- Groomed changed from No to Yes
It may be helpful in the future to somehow represent workers that have gone "offline", but we need to answer more questions about that state. For example, we'd probably want to differentiate between workers that went offline gracefully vs. unexpectedly.
So for now, only showing online workers is I think the best move.
Updated by dalley about 7 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to dalley
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 f96d5b7f | View on GitHub
Show only the online workers in the status API
Updated by dalley about 7 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp|f96d5b7f3db18ae23b2687f41d88dd34eddfcac2.
Updated by kersom over 6 years ago
- Tags Functional Test added
- Tags deleted (
Pulp 3)
Updated by bmbouter about 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Show only the online workers in the status API
closes #2659 https://pulp.plan.io/issues/2659