Project

Profile

Help

Actions
Send by e-mail Copy link

Task #4554

closed

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

Added by ehelms@redhat.com over 4 years ago. Updated almost 4 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-*

Actions
Copy link
 #1

Updated by ttereshc over 4 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

Actions
Copy link
 #2

Updated by ttereshc over 4 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
Actions
Copy link
 #3

Updated by dkliban@redhat.com over 4 years ago

  • Sprint set to Sprint 51

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

Actions
Copy link
 #4

Updated by ehelms@redhat.com over 4 years ago

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

Actions
Copy link
 #5

Updated by bmbouter over 4 years ago

  • Tags Pulp 2 added
Actions
Copy link
 #6

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 51 to Sprint 52
Actions
Copy link
 #7

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 52 to Sprint 53
Actions
Copy link
 #8

Updated by ttereshc over 4 years ago

  • Platform Release set to 2.20.0
Actions
Copy link
 #9

Updated by amacdona@redhat.com over 4 years ago

  • Sprint changed from Sprint 53 to Sprint 54
Actions
Copy link
 #10

Updated by dkliban@redhat.com over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com
Actions
Copy link
 #12

Updated by ttereshc over 4 years ago

  • Sprint changed from Sprint 54 to Sprint 55
Actions
Copy link
 #13

Updated by ttereshc over 4 years ago

  • Platform Release deleted (2.20.0)
Actions
Copy link
 #14

Updated by ttereshc over 4 years ago

  • Status changed from POST to NEW
  • Assignee deleted (dkliban@redhat.com)
Actions
Copy link
 #15

Updated by ttereshc over 4 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)
Actions
Copy link
 #16

Updated by ttereshc over 4 years ago

Pulp 3 services should be renamed to:

pulpcore-resource-manager
pulpcore-workers
pulpcore-worker-{n}
Actions
Copy link
 #18

Updated by dkliban@redhat.com over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com
Actions
Copy link
 #19

Updated by dkliban@redhat.com over 4 years ago

  • Sprint changed from Sprint 55 to Sprint 56
Actions
Copy link
 #20

Updated by dkliban@redhat.com over 4 years ago

  • Status changed from ASSIGNED to NEW
Actions
Copy link
 #21

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 56 to Sprint 57
Actions
Copy link
 #22

Updated by dkliban@redhat.com over 4 years ago

  • Assignee deleted (dkliban@redhat.com)
Actions
Copy link
 #23

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 57 to Sprint 58
Actions
Copy link
 #24

Updated by daviddavis about 4 years ago

  • Sprint/Milestone changed from 3.0.0 to 71
Actions
Copy link
 #25

Updated by bmbouter about 4 years ago

  • Sprint/Milestone changed from 71 to 3.0.0
Actions
Copy link
 #26

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 58 to Sprint 59
Actions
Copy link
 #27

Updated by lmjachky about 4 years ago

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

Added by Lubos Mjachky about 4 years ago

Revision 366dda09 | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision 239d4ced | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision f093c99f | View on GitHub

Rename pulp services

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

Actions
Copy link
 #29

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 59 to Sprint 60

Added by Lubos Mjachky about 4 years ago

Revision c69124af | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision c69124af | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision 9c6b98d8 | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision 6f3907a6 | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision 6f3907a6 | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision 6f3907a6 | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision 6f3907a6 | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision e83b4689 | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision a186bd6d | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision 11479d7f | View on GitHub

Rename pulp services

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

Added by Lubos Mjachky about 4 years ago

Revision 4563dfc0 | View on GitHub

Rename pulp services

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

Actions
Copy link
 #30

Updated by Anonymous about 4 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions
Copy link
 #31

Updated by mdepaulo@redhat.com about 4 years ago

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

Actions
Copy link
 #32

Updated by mdepaulo@redhat.com about 4 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.

Actions
Copy link
 #33

Updated by dkliban@redhat.com about 4 years ago

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

Actions
Copy link
 #34

Updated by bmbouter about 4 years ago

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

Added by ttereshc about 4 years ago

Revision 42e08bb4 | View on GitHub

Rename pulpcore-content-app to pulpcore-content

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

Added by ttereshc about 4 years ago

Revision 42e08bb4 | View on GitHub

Rename pulpcore-content-app to pulpcore-content

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

Added by ttereshc about 4 years ago

Revision 1c8ed3d6 | View on GitHub

Rename pulpcore-content-app to pulpcore-plugin

re #4554

Actions
Copy link
 #35

Updated by dalley about 4 years ago

  • Status changed from MODIFIED to POST
Actions
Copy link
 #37

Updated by Anonymous about 4 years ago

  • Status changed from POST to MODIFIED
Actions
Copy link
 #38

Updated by bmbouter almost 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Send by e-mail Copy link

Also available in: Atom PDF