Project

Profile

Help

Task #4554

Change naming of Pulp 3 services to differentiate them from Pulp 2 services

Added by ehelms@redhat.com about 1 year 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:
Yes
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 60

Description

When Pulp 2 and Pulp 3 are ran on the same system differentiating the services is difficult which affects the ability to debug, restart the proper services and overall manage both applications during transition. Based on some pulp-dev email conversations one of two options should be taken:

#1: Rename Pulp 3 services to all new names to clearly differentiate

For example:

a) pulp-task-manager
b) pulp-tasking-manager
c) pulp-queue-manager
d) pulp-arborist
e) arborist
f) pulp-task-runner
g) pulp-task-worker
h) pulp-async-worker
i) pulp-tasking-worker
j) pulp-seedling

#2: Rename Pulp 2 services to include version

-a) pulp2-resource-manager
b) pulp2-worker-

As per this thread, it's decided to go with option 2 - rename pulp2 services.

As per this thread, it's decided to go with option 1 a rename pulp3 services to pulpcore-*

Associated revisions

Revision 366dda09 View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision 239d4ced View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision f093c99f View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision c69124af View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision c69124af View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision 9c6b98d8 View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision e83b4689 View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision a186bd6d View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision 11479d7f View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision 4563dfc0 View on GitHub
Added by Lubos Mjachky 6 months ago

Rename pulp services

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

Revision 42e08bb4 View on GitHub
Added by ttereshc 6 months ago

Rename pulpcore-content-app to pulpcore-content

re #4554 https://pulp.plan.io/issues/4554

Revision 42e08bb4 View on GitHub
Added by ttereshc 6 months ago

Rename pulpcore-content-app to pulpcore-content

re #4554 https://pulp.plan.io/issues/4554

Revision 1c8ed3d6 View on GitHub
Added by ttereshc 6 months ago

Rename pulpcore-content-app to pulpcore-plugin

re #4554

History

#1 Updated by ttereshc about 1 year ago

  • Tracker changed from Issue to Task
  • Subject changed from Change naming of Pulp 3 services or version name Pulp 2 services to differentiate themm to Change naming of Pulp 3 services or version name Pulp 2 services to differentiate them
  • % Done set to 0

TBD on this thread https://www.redhat.com/archives/pulp-dev/2019-March/msg00085.html

Option 1 is under question at the moment As per this thread ^, no changes to Pulp3.
Option 2 is agreed to happen anyway

#2 Updated by ttereshc about 1 year ago

  • Subject changed from Change naming of Pulp 3 services or version name Pulp 2 services to differentiate them to Change naming of Pulp 2 services to differentiate them from Pulp 3 services
  • Description updated (diff)
  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes

#3 Updated by dkliban@redhat.com about 1 year ago

  • Sprint set to Sprint 51

This change needs to be advertised on pulp-list because this goes against semantic versioning.

#4 Updated by ehelms@redhat.com about 1 year ago

We could consider adding aliases to the old names to help with backwards compatibility of scripts.

#5 Updated by bmbouter 12 months ago

  • Tags Pulp 2 added

#6 Updated by rchan 12 months ago

  • Sprint changed from Sprint 51 to Sprint 52

#7 Updated by rchan 11 months ago

  • Sprint changed from Sprint 52 to Sprint 53

#8 Updated by ttereshc 11 months ago

  • Platform Release set to 2.20.0

#9 Updated by amacdona@redhat.com 10 months ago

  • Sprint changed from Sprint 53 to Sprint 54

#10 Updated by dkliban@redhat.com 10 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com

#12 Updated by ttereshc 10 months ago

  • Sprint changed from Sprint 54 to Sprint 55

#13 Updated by ttereshc 9 months ago

  • Platform Release deleted (2.20.0)

#14 Updated by ttereshc 9 months ago

  • Status changed from POST to NEW
  • Assignee deleted (dkliban@redhat.com)

#15 Updated by ttereshc 9 months ago

  • Subject changed from Change naming of Pulp 2 services to differentiate them from Pulp 3 services to Change naming of Pulp 3 services to differentiate them from Pulp 2 services
  • Description updated (diff)
  • Sprint/Milestone set to 3.0.0
  • Tags deleted (Pulp 2)

#16 Updated by ttereshc 9 months ago

Pulp 3 services should be renamed to:

pulpcore-resource-manager
pulpcore-workers
pulpcore-worker-{n}

#17 Updated by ttereshc 9 months ago

  • Backwards Incompatible changed from No to Yes

#18 Updated by dkliban@redhat.com 9 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com

#19 Updated by dkliban@redhat.com 9 months ago

  • Sprint changed from Sprint 55 to Sprint 56

#20 Updated by dkliban@redhat.com 9 months ago

  • Status changed from ASSIGNED to NEW

#21 Updated by rchan 8 months ago

  • Sprint changed from Sprint 56 to Sprint 57

#22 Updated by dkliban@redhat.com 8 months ago

  • Assignee deleted (dkliban@redhat.com)

#23 Updated by rchan 7 months ago

  • Sprint changed from Sprint 57 to Sprint 58

#24 Updated by daviddavis 7 months ago

  • Sprint/Milestone changed from 3.0.0 to 71

#25 Updated by bmbouter 7 months ago

  • Sprint/Milestone changed from 71 to 3.0.0

#26 Updated by rchan 7 months ago

  • Sprint changed from Sprint 58 to Sprint 59

#27 Updated by lmjachky 7 months ago

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

#29 Updated by rchan 6 months ago

  • Sprint changed from Sprint 59 to Sprint 60

#30 Updated by Anonymous 6 months ago

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

#31 Updated by mdepaulo@redhat.com 6 months ago

I made a big review comment, that affects every other PR:
https://github.com/pulp/ansible-pulp/pull/161#pullrequestreview-296305649

#32 Updated by mdepaulo@redhat.com 6 months ago

While we're renaming all the services, I request that one be renamed differently:
Instead of:
pulp-content-app -> pulpcore-content-app

Please do:
pulp-content-app -> pulpcore-content

The reason is for brevity, and consistency:
1. The script on disk is "$PREFIX/bin/pulp-content"
2. The ansible role is "pulp-content"
3. The pulp/pulpcore container images, which only ever run pulp3 (not pulp2) in it and which uses command names as an interface with pulp-operator, use "pulp-content" as well.

#33 Updated by dkliban@redhat.com 6 months ago

+1 to renaming pulp-content-app to pulpcore-content

#34 Updated by bmbouter 6 months ago

+1 to renaming pulp-content-app to pulpcore-content

#35 Updated by dalley 6 months ago

  • Status changed from MODIFIED to POST

#37 Updated by Anonymous 6 months ago

  • Status changed from POST to MODIFIED

#38 Updated by bmbouter 4 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF