Project

Profile

Help

Task #3078

pulp 3 ansible installer does not create a systemd unit file for the webserver

Added by dkliban@redhat.com almost 3 years ago. Updated 5 months ago.

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

100%

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

Description

The user should be able to start the webserver using systemd. The pulp3_systemd ansible role already creates a systemd file for the pulp_workers. However, it fails to create one for the web server.

Associated revisions

Revision cafd0171 View on GitHub
Added by Ichimonji10 almost 3 years ago

Run the development Pulp 3 webserver for testing

Make the pulp-3-dev-{os} jobs run the development webserver after Pulp 3 has been installed, so that it can be smashed. Do so by simply executing pulp-manager runserver … as the "pulp" user. This is a hacky and awful solution, because among other things, it makes process management a pain. Stopping the dev webserver requires manually hunting down and killing processes. Restarting the dev webserver is worse.

However, this is intended to be a temporary solution that's only in place until the devs figure out how Pulp 3 is supposed to run. When a solution that integrates with Apache/Nginx/LightTPD/etc is devised, this can be dropped.

See: https://pulp.plan.io/issues/3078

closes #3078

History

#1 Updated by dkliban@redhat.com almost 3 years ago

  • Subject changed from pulp 3 ansible installer does not create a systemd unit file the webserver to pulp 3 ansible installer does not create a systemd unit file for the webserver

#2 Updated by Ichimonji10 almost 3 years ago

This issue can be solved by creating a unit named e.g. pulp-manager-runserver in ~/.config/systemd/user/. It can then be managed with commands like systemctl --user start pulp-manager-runserver. Also see loginctl(1), especially the stuff related to lingering.

#3 Updated by amacdona@redhat.com almost 3 years ago

I'm not convinced we should run the django webserver with systemd. Maybe this should wait until we have apache or nginx.

#4 Updated by dkliban@redhat.com almost 3 years ago

We should do it this way for now. When we add nginx or httpd, we will create systemd unit files for them also. The user will have to choose which one she wants.

#5 Updated by amacdona@redhat.com almost 3 years ago

  • Groomed changed from No to Yes

I'm convinced. We aren't guaranteeing this for production.

#6 Updated by amacdona@redhat.com almost 3 years ago

  • Tracker changed from Issue to Task
  • % Done set to 0
  • Sprint Candidate changed from No to Yes

#7 Updated by jortel@redhat.com almost 3 years ago

  • Sprint/Milestone set to 46

#8 Updated by Ichimonji10 almost 3 years ago

  • Assignee set to Ichimonji10

I'm taking the liberty of assigning this to myself, both because I'm working on it, and also because this issue is on the sprint for the benefit of QE.

#9 Updated by amacdona@redhat.com almost 3 years ago

  • Status changed from NEW to ASSIGNED

#10 Updated by Ichimonji10 almost 3 years ago

See: https://github.com/pulp/pulp_packaging/pull/442

The proposed solution doesn't make use of systemd at all. It's totally doable, but the proposed solution is even simper. For an exmaple of what the systemd-based solution might look like, see master...Ichimonji10:pulp-3-webserver-systemd.

#11 Updated by Ichimonji10 almost 3 years ago

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

#12 Updated by bmbouter over 2 years ago

  • Sprint set to Sprint 27

#13 Updated by bmbouter over 2 years ago

  • Sprint/Milestone deleted (46)

#14 Updated by daviddavis over 1 year ago

  • Sprint/Milestone set to 3.0.0

#15 Updated by bmbouter over 1 year ago

  • Tags deleted (Pulp 3)

#16 Updated by bmbouter 10 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

#17 Updated by bmbouter 5 months ago

  • Category set to Installer
  • Tags deleted (Pulp 3 installer)

Please register to edit this issue

Also available in: Atom PDF