Project

Profile

Help

Story #2659

closed

As a User I would like Pulp3 to not show me offline workers in the status api

Added by bizhang over 7 years ago. Updated about 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Functional Test
Sprint:
Sprint 28
Quarter:

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

Actions #1

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
Actions #2

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
Actions #3

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
Actions #4

Updated by bmbouter about 7 years ago

  • Description updated (diff)
Actions #5

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.

Actions #6

Updated by mhrivnak about 7 years ago

  • Sprint/Milestone set to 47
Actions #7

Updated by dalley about 7 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dalley
Actions #8

Updated by dalley about 7 years ago

  • Status changed from ASSIGNED to POST

Added by dalley about 7 years ago

Revision f96d5b7f | View on GitHub

Show only the online workers in the status API

closes #2659 https://pulp.plan.io/issues/2659

Added by dalley about 7 years ago

Revision f96d5b7f | View on GitHub

Show only the online workers in the status API

closes #2659 https://pulp.plan.io/issues/2659

Actions #9

Updated by dalley about 7 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #11

Updated by bmbouter almost 7 years ago

  • Sprint set to Sprint 28
Actions #12

Updated by bmbouter almost 7 years ago

  • Sprint/Milestone deleted (47)
Actions #13

Updated by kersom over 6 years ago

  • Tags Functional Test added
  • Tags deleted (Pulp 3)
Actions #14

Updated by dkliban@redhat.com over 6 years ago

  • Tags Pulp 3 added
Actions #15

Updated by daviddavis over 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #16

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions #17

Updated by bmbouter about 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF