Project

Profile

Help

Task #4554

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

Added by ehelms@redhat.com 7 months ago. Updated 13 days ago.

Status:
MODIFIED
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 24 days ago

Rename pulp services

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

Revision 239d4ced View on GitHub
Added by Lubos Mjachky 24 days ago

Rename pulp services

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

Revision f093c99f View on GitHub
Added by Lubos Mjachky 24 days ago

Rename pulp services

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

Revision c69124af View on GitHub
Added by Lubos Mjachky 16 days ago

Rename pulp services

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

Revision 9c6b98d8 View on GitHub
Added by Lubos Mjachky 16 days ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky 16 days ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky 16 days ago

Rename pulp services

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

Revision e83b4689 View on GitHub
Added by Lubos Mjachky 16 days ago

Rename pulp services

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

Revision a186bd6d View on GitHub
Added by Lubos Mjachky 16 days ago

Rename pulp services

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

Revision 11479d7f View on GitHub
Added by Lubos Mjachky 16 days ago

Rename pulp services

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

Revision 4563dfc0 View on GitHub
Added by Lubos Mjachky 16 days ago

Rename pulp services

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

Revision 42e08bb4 View on GitHub
Added by ttereshc 14 days ago

Rename pulpcore-content-app to pulpcore-content

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

Revision 1c8ed3d6 View on GitHub
Added by ttereshc 14 days ago

Rename pulpcore-content-app to pulpcore-plugin

re #4554

History

#1 Updated by ttereshc 7 months 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 7 months 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 7 months 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 7 months ago

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

#5 Updated by bmbouter 6 months ago

  • Tags Pulp 2 added

#6 Updated by rchan 6 months ago

  • Sprint changed from Sprint 51 to Sprint 52

#7 Updated by rchan 5 months ago

  • Sprint changed from Sprint 52 to Sprint 53

#8 Updated by ttereshc 5 months ago

  • Platform Release set to 2.20.0

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

  • Sprint changed from Sprint 53 to Sprint 54

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

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

#12 Updated by ttereshc 4 months ago

  • Sprint changed from Sprint 54 to Sprint 55

#13 Updated by ttereshc 4 months ago

  • Platform Release deleted (2.20.0)

#14 Updated by ttereshc 4 months ago

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

#15 Updated by ttereshc 4 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
  • Tags deleted (Pulp 2)

#16 Updated by ttereshc 4 months ago

Pulp 3 services should be renamed to:

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

#17 Updated by ttereshc 4 months ago

  • Backwards Incompatible changed from No to Yes

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

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

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

  • Sprint changed from Sprint 55 to Sprint 56

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

  • Status changed from ASSIGNED to NEW

#21 Updated by rchan 3 months ago

  • Sprint changed from Sprint 56 to Sprint 57

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

  • Assignee deleted (dkliban@redhat.com)

#23 Updated by rchan about 2 months ago

  • Sprint changed from Sprint 57 to Sprint 58

#24 Updated by daviddavis about 1 month ago

  • Sprint/Milestone changed from 3.0 to 71

#25 Updated by bmbouter about 1 month ago

  • Sprint/Milestone changed from 71 to 3.0

#26 Updated by rchan 30 days ago

  • Sprint changed from Sprint 58 to Sprint 59

#27 Updated by lmjachky 28 days ago

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

#29 Updated by rchan 20 days ago

  • Sprint changed from Sprint 59 to Sprint 60

#30 Updated by Anonymous 15 days ago

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

#31 Updated by mdepaulo@redhat.com 15 days 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 15 days 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 15 days ago

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

#34 Updated by bmbouter 15 days ago

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

#35 Updated by dalley 14 days ago

  • Status changed from MODIFIED to POST

#37 Updated by Anonymous 13 days ago

  • Status changed from POST to MODIFIED

Please register to edit this issue

Also available in: Atom PDF