Project

Profile

Help

Story #4239

Story #3693: Lazy for Pulp3

merge the content-app & streamer

Added by bmbouter over 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:
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Sprint:

Description

This proposal was originally proposed here by @jortel.

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

Associated revisions

Revision 8219efd5 View on GitHub
Added by bmbouter over 1 year ago

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

Revision 8219efd5 View on GitHub
Added by bmbouter over 1 year ago

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

Revision 3a1bea73 View on GitHub
Added by mdellweg over 1 year ago

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

Revision 79492ca5 View on GitHub
Added by mdellweg over 1 year ago

Align default settings with new content app

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

Revision 79492ca5 View on GitHub
Added by mdellweg over 1 year ago

Align default settings with new content app

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

History

#1 Updated by bmbouter over 1 year ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter
  • Parent task set to #3693

#2 Updated by bmbouter over 1 year ago

  • Status changed from ASSIGNED to POST

#3 Updated by bmbouter over 1 year ago

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

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

  • Status changed from MODIFIED to POST

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

#5 Updated by bmbouter over 1 year ago

  • Status changed from POST to MODIFIED

PR was merged, back to MODIFIED

#6 Updated by daviddavis about 1 year ago

  • Sprint/Milestone set to 3.0.0

#7 Updated by bmbouter about 1 year ago

  • Tags deleted (Pulp 3)

#8 Updated by bmbouter 6 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF