Project

Profile

Help

Task #5786

Remove "missing workers" from Status API

Added by bmbouter 4 months ago. Updated 4 months ago.

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

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
Yes
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 62

Description

Problem

Environments that have random worker names chosen each time a worker starts, e.g. k8s/openshift, will accumulate a "missing worker" each time a worker starts. This part of the API made sense when the workers always used the same names.

Solution

Remove the "missing workers" from the Task API

Associated revisions

Revision bf8794ac View on GitHub
Added by bmbouter 4 months ago

Remove "missing_workers" from Status API

Having "missing_workers" did not work well when worker names are unique each time. It accumulated an endless number of worker names.

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

Revision c0fcdb7c View on GitHub
Added by bmbouter 4 months ago

Remove "missing_workers" from Status API

Having "missing_workers" did not work well when worker names are unique each time. It accumulated an endless number of worker names.

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

(cherry picked from commit bf8794ac47f34d0cb487b66ffc41bde496fad761)

History

#1 Updated by daviddavis 4 months ago

  • Tracker changed from Issue to Task
  • % Done set to 0
  • Groomed changed from No to Yes
  • Sprint set to Sprint 62

#2 Updated by bmbouter 4 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter

#3 Updated by bmbouter 4 months ago

Also should we?

online_worker -> workers
online_content_apps -> content_apps

#4 Updated by daviddavis 4 months ago

That sounds fine to me.

#5 Updated by bmbouter 4 months ago

daviddavis wrote:

That sounds fine to me.

I made the change and decided to revert it. Having online_ does make it really clear these are the "online" workers.

#6 Updated by bmbouter 4 months ago

  • Status changed from ASSIGNED to POST

#7 Updated by bmbouter 4 months ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#8 Updated by bmbouter 4 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF