Project

Profile

Help

Task #5787

Have workers auto-set their own names to not require users to set them

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

Managing worker names is not really important to users, but they have to think about it

Solution

1) Remove the -n reserved-resource-worker-%i@%%h from https://github.com/pulp/ansible-pulp/blob/master/roles/pulp-workers/templates/pulpcore-worker%40.service.j2#L18 and the documentation.

2) Have the worker set it's own name here and if it's not named as "resource-manager" then make the worker name "pulpcore-worker-UUID" where UUID is a uuid4().

Associated revisions

Revision 76ad5759 View on GitHub
Added by bmbouter 4 months ago

Workers now auto-name {pid}@{hostname}

Requiring users to specify a name for task workers was difficult and not productive. Having workers auto-name as {pid}@{hostname} is much easier to operate.

This also updates the docs to match.

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

Revision 10be1107 View on GitHub
Added by bmbouter 4 months ago

Workers no longer specify a name

A feature was added to have workers auto-name themselves with {pid}@{hostname}. By not specifying the -n option, each process will receive its name automatically.

Required PR: https://github.com/pulp/pulpcore/pull/414

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

Revision 10be1107 View on GitHub
Added by bmbouter 4 months ago

Workers no longer specify a name

A feature was added to have workers auto-name themselves with {pid}@{hostname}. By not specifying the -n option, each process will receive its name automatically.

Required PR: https://github.com/pulp/pulpcore/pull/414

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

Revision 14cc9b4d View on GitHub
Added by bmbouter 4 months ago

Workers now auto-name {pid}@{hostname}

Requiring users to specify a name for task workers was difficult and not productive. Having workers auto-name as {pid}@{hostname} is much easier to operate.

This also updates the docs to match.

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

(cherry picked from commit 76ad5759cd4fc2f5ebb9586e22945a451d0735b2)

History

#1 Updated by bmbouter 4 months ago

  • Sprint/Milestone set to 3.0.0

#2 Updated by daviddavis 4 months ago

Does this mean users can no longer set the worker name? I think that's ok. Just want to make sure I understand.

#3 Updated by bmbouter 4 months ago

Users can still set the worker name, and Pulp should recognize it as a worker and not a resource manager.

#4 Updated by daviddavis 4 months ago

  • Groomed changed from No to Yes
  • Sprint set to Sprint 62

+1 sounds good.

#5 Updated by bmbouter 4 months ago

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

#6 Updated by bmbouter 4 months ago

  • Status changed from ASSIGNED to POST

#7 Updated by fabricio.aguiar 4 months ago

  • Tracker changed from Issue to Task
  • % Done set to 0

#8 Updated by bmbouter 4 months ago

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

#9 Updated by bmbouter 4 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF