Project

Profile

Help

Story #4239

closed

Story #3693: Lazy for Pulp3

merge the content-app & streamer

Added by bmbouter over 5 years ago. Updated over 4 years ago.

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

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

This proposal was originally proposed here by @jortel.

https://www.redhat.com/archives/pulp-dev/2018-November/msg00065.html

Actions #1

Updated by bmbouter over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter
  • Parent issue set to #3693
Actions #2

Updated by bmbouter over 5 years ago

  • Status changed from ASSIGNED to POST

Added by bmbouter over 5 years ago

Revision 8219efd5 | View on GitHub

Adds the content app to pulpcore.content

The Settings can be a lot simpler now that the architecture no longer required redirection. It does need to account for the content being served on a different HOST. This PR handles that.

Since the settings are now single values, they are much easier to override using Dynaconf, which is great.

The content app can be removed now that the streamer code is handling it all.

This PR is a prototype and has no docs. Use Pulp as normal , but run the streamer from github.com/bmbouter/pulp_streamer/ and run the streamer with:

gunicorn pulpcore.streamer:server --bind localhost:8080 --worker-class aiohttp.GunicornWebWorker -w 2

This PR needs pulp-smash changes from pulp_file so it requires the PR below:

Required PR: https://github.com/pulp/pulp_file/pull/140

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

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

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

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

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

Added by bmbouter over 5 years ago

Revision 8219efd5 | View on GitHub

Adds the content app to pulpcore.content

The Settings can be a lot simpler now that the architecture no longer required redirection. It does need to account for the content being served on a different HOST. This PR handles that.

Since the settings are now single values, they are much easier to override using Dynaconf, which is great.

The content app can be removed now that the streamer code is handling it all.

This PR is a prototype and has no docs. Use Pulp as normal , but run the streamer from github.com/bmbouter/pulp_streamer/ and run the streamer with:

gunicorn pulpcore.streamer:server --bind localhost:8080 --worker-class aiohttp.GunicornWebWorker -w 2

This PR needs pulp-smash changes from pulp_file so it requires the PR below:

Required PR: https://github.com/pulp/pulp_file/pull/140

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

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

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

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

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

Actions #3

Updated by bmbouter over 5 years ago

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

Added by mdellweg over 5 years ago

Revision 3a1bea73 | View on GitHub

Add content app service

Add the systemd service file, add pulp_content_app to bash helpers and configure pulp to redirect to the content app.

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

Added by mdellweg over 5 years ago

Revision 79492ca5 | View on GitHub

Align default settings with new content app

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

Added by mdellweg over 5 years ago

Revision 79492ca5 | View on GitHub

Align default settings with new content app

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

Actions #4

Updated by amacdona@redhat.com over 5 years ago

  • Status changed from MODIFIED to POST

Switching back to post to make room for https://github.com/pulp/pulp/pull/3802

Actions #5

Updated by bmbouter over 5 years ago

  • Status changed from POST to MODIFIED

PR was merged, back to MODIFIED

Actions #6

Updated by daviddavis almost 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #7

Updated by bmbouter almost 5 years ago

  • Tags deleted (Pulp 3)
Actions #8

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF