Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Support gunicorn worker recycling" #581

Merged

Conversation

mdellweg
Copy link
Member

Reverts #573

This was a breaking change and is not compatible with all versions of
pulpcore. The options introduced here are not guaranteed to exist. And
so starting the container fails.

[noissue]
@mdellweg mdellweg force-pushed the revert-573-feature/support-gunicorn-worker-recycling branch from 6f00023 to 77101db Compare January 23, 2024 10:11
@mikedep333
Copy link
Member

Sorry @markafarrell , we need to revert this until a new PR is opened up.

The problem is that our CI for plugins uses the current Dockerfiles (pulp_ci_centos) to test installing older versions of pulpcore as well.

The entrypoint changed from gunicorn / pulp-content to pulpcore-content / pulpcore-api in 3.35. But the old binaries do not support these arguments, thus the CI breaks.

So we need to only add these arguments if the new binaries are in use.

@mikedep333 mikedep333 merged commit a7b1cd3 into latest Jan 23, 2024
12 of 15 checks passed
@mikedep333 mikedep333 deleted the revert-573-feature/support-gunicorn-worker-recycling branch January 23, 2024 17:41
@mdellweg
Copy link
Member Author

I do believe the thing is that these specific options were added even later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants