Story #4239
Story #3693: Lazy for Pulp3
merge the content-app & streamer
100%
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
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
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.
Revision 79492ca5
View on GitHub
Align default settings with new content app
Revision 79492ca5
View on GitHub
Align default settings with new content app
History
#1
Updated by bmbouter about 2 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to bmbouter
- Parent task set to #3693
#2
Updated by bmbouter about 2 years ago
- Status changed from ASSIGNED to POST
PR available at: https://github.com/pulp/pulp/pull/3779
also here: https://github.com/pulp/pulp_file/pull/140
#3
Updated by bmbouter about 2 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp|8219efd59098fc5e883ea831d4fb38b7ae15422d.
#4
Updated by amacdona@redhat.com about 2 years 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 about 2 years ago
- Status changed from POST to MODIFIED
PR was merged, back to MODIFIED
#6
Updated by daviddavis almost 2 years ago
- Sprint/Milestone set to 3.0.0
#7
Updated by bmbouter almost 2 years ago
- Tags deleted (
Pulp 3)
#8
Updated by bmbouter about 1 year ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Please register to edit this issue
.
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 2This 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