Project

Profile

Help

Send by e-mail

Task #4554

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

Added by ehelms@redhat.com over 2 years ago. Updated almost 2 years ago.

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

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 60
Quarter:

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 about 2 years ago

Rename pulp services

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

Revision 239d4ced View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision f093c99f View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision c69124af View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision c69124af View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision 9c6b98d8 View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision 6f3907a6 View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision e83b4689 View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision a186bd6d View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision 11479d7f View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision 4563dfc0 View on GitHub
Added by Lubos Mjachky about 2 years ago

Rename pulp services

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

Revision 42e08bb4 View on GitHub
Added by ttereshc about 2 years ago

Rename pulpcore-content-app to pulpcore-content

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

Revision 42e08bb4 View on GitHub
Added by ttereshc about 2 years ago

Rename pulpcore-content-app to pulpcore-content

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

Revision 1c8ed3d6 View on GitHub
Added by ttereshc about 2 years ago

Rename pulpcore-content-app to pulpcore-plugin

re #4554

Revision 6fa946bd View on GitHub
Added by ttereshc about 2 years ago

Rename pulpcore-content-app to pulpcore-content

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

Required PR: https://github.com/pulp/ansible-pulp/pull/170

History

#1 Updated by ttereshc over 2 years 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 over 2 years 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 over 2 years 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 over 2 years ago

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

#5 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

#6 Updated by rchan over 2 years ago

  • Sprint changed from Sprint 51 to Sprint 52

#7 Updated by rchan over 2 years ago

  • Sprint changed from Sprint 52 to Sprint 53

#8 Updated by ttereshc over 2 years ago

  • Platform Release set to 2.20.0

#9 Updated by amacdona@redhat.com over 2 years ago

  • Sprint changed from Sprint 53 to Sprint 54

#10 Updated by dkliban@redhat.com over 2 years ago

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

#12 Updated by ttereshc over 2 years ago

  • Sprint changed from Sprint 54 to Sprint 55

#13 Updated by ttereshc over 2 years ago

  • Platform Release deleted (2.20.0)

#14 Updated by ttereshc over 2 years ago

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

#15 Updated by ttereshc over 2 years 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 over 2 years ago

Pulp 3 services should be renamed to:

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

#18 Updated by dkliban@redhat.com over 2 years ago

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

#19 Updated by dkliban@redhat.com over 2 years ago

  • Sprint changed from Sprint 55 to Sprint 56

#20 Updated by dkliban@redhat.com over 2 years ago

  • Status changed from ASSIGNED to NEW

#21 Updated by rchan about 2 years ago

  • Sprint changed from Sprint 56 to Sprint 57

#22 Updated by dkliban@redhat.com about 2 years ago

  • Assignee deleted (dkliban@redhat.com)

#23 Updated by rchan about 2 years ago

  • Sprint changed from Sprint 57 to Sprint 58

#24 Updated by daviddavis about 2 years ago

  • Sprint/Milestone changed from 3.0.0 to 71

#25 Updated by bmbouter about 2 years ago

  • Sprint/Milestone changed from 71 to 3.0.0

#26 Updated by rchan about 2 years ago

  • Sprint changed from Sprint 58 to Sprint 59

#27 Updated by lmjachky about 2 years ago

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

#29 Updated by rchan about 2 years ago

  • Sprint changed from Sprint 59 to Sprint 60

#30 Updated by Anonymous about 2 years ago

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

#31 Updated by mdepaulo@redhat.com about 2 years 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 about 2 years 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 about 2 years ago

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

#34 Updated by bmbouter about 2 years ago

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

#35 Updated by dalley about 2 years ago

  • Status changed from MODIFIED to POST

#37 Updated by Anonymous about 2 years ago

  • Status changed from POST to MODIFIED

#38 Updated by bmbouter almost 2 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Send by e-mail

Also available in: Atom PDF