Project

Profile

Help

Issue #4187

Pulp3's systemd unit names should not conflict with Pulp2's systemd unit names

Added by bmbouter 11 months ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Pulp 3 installer
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 49

Description

Problem

Pulp2 uses the following systemd unit names: 'pulp_resource_manager', 'pulp_workers' among others.
Pulp3 also needs those same components and uses 'pulp_resource_manager' and 'pulp_workers' also. per the docs

Solution: Have Pulp3 change

Pulp would switch to 'pulp3_resource_manager' and 'pulp3_worker@1', pulp3_worker@2.

Solution: Have Pulp2 change

Pulp would switch to 'pulp2_resource_manager' and 'pulp2_workers'.

Solution: Leave them conflicting and the user avoids through configuration

This option would have a documentation note that if these systemd names conflict with your system, perhaps due to a Pulp2 install already there, that you can rename the unit names and that is OK.

Also we would need to add a variable for this to the Ansible installer so automated installs could also do the same thing.

Associated revisions

Revision 6038d55a View on GitHub
Added by ipanova@redhat.com 8 months ago

Use hyphens for Pulp3' system units name.

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

Revision bfa27591 View on GitHub
Added by ipanova@redhat.com 8 months ago

Use hyphens for Pulp3' system units name.

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

Revision bed44546 View on GitHub
Added by ipanova@redhat.com 8 months ago

Use hyphens for Pulp3' system units name.

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

Revision 9394e92b View on GitHub
Added by ipanova@redhat.com 8 months ago

Use hyphens for Pulp3' system units name.
closes #4187
https://pulp.plan.io/issues/4187

Revision 9394e92b View on GitHub
Added by ipanova@redhat.com 8 months ago

Use hyphens for Pulp3' system units name.
closes #4187
https://pulp.plan.io/issues/4187

Revision 4a9b071f View on GitHub
Added by bmbouter 8 months ago

Update to latest ansible-pulp3

This is primarily to pick up the systemd name changes that came with
this change https://github.com/pulp/ansible-pulp3/pull/75/files

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

History

#1 Updated by CodeHeeler 11 months ago

  • Triaged changed from No to Yes

#2 Updated by bmbouter 8 months ago

  • Tags Pulp 3 RC Blocker, Pulp 3 installer added

I think this should be an RC blocker since it was identified as preventing Pulp2 and Pulp3 from running on the same system. Changing it would also be backwards incompatible.

#3 Updated by amacdona@redhat.com 8 months ago

IMO we should do both Option 1 and Option 3.

The default should not conflict with Pulp2, but users should also be able configure it. Completing Option 1 should be an RC blocker, but completing Option 3 should be added as a separate story and does not need to be an RC blocker, and will be backwards compatible.

#4 Updated by daviddavis 8 months ago

+1 to asmacdo's proposal.

#5 Updated by bmbouter 8 months ago

#6 Updated by ipanova@redhat.com 8 months ago

  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes

#7 Updated by jortel@redhat.com 8 months ago

Using hyphens for names in Linux is far more conventional and convenient that underscores. Let's resolve the conflict and improve the naming all at once by naming the pulp3 services: pulp-resource-manager and pulp-worker@1 and not rename pulp2 services.

#8 Updated by daviddavis 8 months ago

It seems like other systemd units use hyphens as well. The only downside is there's nothing to indicate for users whether pulp-resource-manager or pulp_resource_manager is pulp 2 or 3.

Regardless, I think I am +1 to using hyphens.

#9 Updated by ttereshc 8 months ago

  • Sprint set to Sprint 49

#10 Updated by ipanova@redhat.com 8 months ago

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

#11 Updated by bmbouter 8 months ago

+1 to hyphens for Pulp3

#12 Updated by ipanova@redhat.com 8 months ago

issue for option3 created https://pulp.plan.io/issues/4429

#14 Updated by ipanova@redhat.com 8 months ago

  • Status changed from POST to ASSIGNED

#15 Updated by ipanova@redhat.com 8 months ago

  • Status changed from ASSIGNED to POST

#16 Updated by ipanova@redhat.com 8 months ago

  • Status changed from POST to MODIFIED

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

  • Status changed from MODIFIED to ASSIGNED

#23 Updated by ipanova@redhat.com 8 months ago

  • Status changed from ASSIGNED to POST

#24 Updated by ipanova@redhat.com 8 months ago

  • Status changed from POST to MODIFIED

#25 Updated by ipanova@redhat.com 8 months ago

  • Status changed from MODIFIED to POST

#26 Updated by ipanova@redhat.com 8 months ago

  • Status changed from POST to MODIFIED

#28 Updated by ipanova@redhat.com 8 months ago

Applied in changeset plugin_template:commit:bfa275912708357be33b66047544cebd30cd982d.

#32 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#33 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3, Pulp 3 RC Blocker)

Please register to edit this issue

Also available in: Atom PDF