Project

Profile

Help

Task #4629

deduplicate systemd config examples with the ansible-pulp templates

Added by bmbouter about 1 year ago. Updated 6 months ago.

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

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Documentation
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 51

Description

The templates in our docs here ( https://docs.pulpproject.org/en/3.0/nightly/installation/instructions.html#systemd ) could instead refer the users to the pulp-ansible templates. We could have the user link to the ansible-pulp templates for the three systemd files they should be deploying:

pulp-content-app
pulp-worker
pulp-resource-manager

These should also link to the variable definitions, e.g. https://github.com/pulp/ansible-pulp/blob/master/roles/pulp/README.md#role-variables

Associated revisions

Revision 8e1c72dc View on GitHub
Added by bmbouter about 1 year ago

systemd docs use Ansible Installer Template

The systemd docs were incomplete in various ways and regularly out of date. This rewrites that section to link users to the Ansible Installer's templates for these files which are maintained and will be up to date.

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

Revision e1136174 View on GitHub
Added by bmbouter about 1 year ago

Add in pulp-api systemd template links

When fixing the systemd templates I did not include the pulp-api systemd template links. This was pointed out on pulp-list so I am fixing now.

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

Revision 51970592 View on GitHub
Added by bmbouter about 1 year ago

One liner systemd fix

The systemctl command was not updated correctly for the pulp-api service file change. This fixes that.

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

History

#1 Updated by bmbouter about 1 year ago

I think we should duplicate because our docs are long as it is, and the ansible-pulp is already a better reference and its maintained.

#2 Updated by amacdona@redhat.com about 1 year ago

The only problem with doing this today is that the templates are in jinja, and the user may not understand where the variables are set in ansible-pulp. (For instance, pulp_install_dir default is set in the pulp role, even though the template is defined in the pulp-worker (or other processes that you mentioned) role.

If we do link to the template, we should also link to the documentation that specifies the value's meaning and default. https://github.com/pulp/ansible-pulp/blob/master/roles/pulp/README.md#role-variables

This plan might actually make more sense than rendering the template with default values, since they might not align with what the user has installed.

#3 Updated by bmbouter about 1 year ago

  • Description updated (diff)

@asmacdo linking to their definitions I think will make this very clear for users. I edited it to include the linking to the definitions.

What about the default should we document those?

Also is taking away these copy/paste versions going to be a problem for our users?

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

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

I think we should not document the default, since it is already duplicated in ansible-pulp (where the default is set, and in the README).

IMO, the users not being able to copy/paste is acceptable. If they are using the default values and want this to be "out of the box", they should use ansible-pulp. If they are setting up their config to be custom, then they won't benefit from default values.

#5 Updated by bmbouter about 1 year ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter
  • Sprint set to Sprint 51

Adding to sprint as part of the docs push

#6 Updated by bmbouter about 1 year ago

  • Status changed from ASSIGNED to POST

#7 Updated by bmbouter about 1 year ago

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

#8 Updated by mdepaulo@redhat.com about 1 year ago

I just want to point out that another option is to install static .service files (containing both static settings and default settings) under /usr/lib/systemd/system/ , and put the customized settings as systemd override files like /etc/systemd/system/<service-name>/<arbitrary-name>.conf .

I'm not sure of PyPI can install & maintain the files under /usr/lib/systemd/system/ (RPM can), but this would simplify keeping the manual instructions & ansible-pulp in sync. All you'd need to do is specify the customized settings in both.

#9 Updated by bmbouter about 1 year ago

I believe with the PyPI packages we can't assume the user will be using systemd so I don't think we can deliver the unit files as part of PyPI. for example we don't ship dependencies on database drivers for he same reason, because we can't know if they want to use mysql or postgresql.

The installer can install the systemd files correctly because it knows specifically if the system should use systemd or not.

#10 Updated by bmbouter about 1 year ago

  • Status changed from MODIFIED to POST

Moving back to post so I can make one more PR that describes the pulp-api content service configuration need.

This should fix up: https://www.redhat.com/archives/pulp-list/2019-April/msg00026.html

#12 Updated by bmbouter about 1 year ago

  • Status changed from POST to MODIFIED

#13 Updated by bmbouter about 1 year ago

  • Status changed from MODIFIED to POST

Moving back to post for one final fix.

#15 Updated by bmbouter about 1 year ago

  • Status changed from POST to MODIFIED

#16 Updated by daviddavis about 1 year ago

  • Sprint/Milestone set to 3.0.0

#17 Updated by bmbouter about 1 year ago

  • Tags deleted (Pulp 3)

#18 Updated by bmbouter 6 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF