Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove the upstream pypi reverse proxy
Trying to reverse proxy upstream pypi has not turned out to be very stable, or very useful. We've had many, many reports of stability issues and the additional complexity for offline and proxy usage is just not worth it. Given we already have a mechanism in place to handle using upstream pypi if the repo server is not there yet, disabling this should just result in that mechanism kicking in and all will be well again. Once the repo is built, the reverse proxy to pypiserver will then be exclusively used and the upstream pypi proxy is not necessary anyway. Depends-On: https://review.openstack.org/584394 Change-Id: Ie407c6a346de6b46c8f4d30caea8664a7f6bd341 (cherry picked from commit dab934b)
- Loading branch information
Jesse Pretorius
committed
Jul 23, 2018
1 parent
c97a300
commit 1bc9519
Showing
4 changed files
with
16 additions
and
54 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
10 changes: 10 additions & 0 deletions
10
releasenotes/notes/pypi-cache-removed-c03a9a0658c9d89f.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
deprecations: | ||
- | | ||
The repo server's reverse proxy for pypi has now been removed, | ||
leaving only the pypiserver to serve packages already on the | ||
repo server. The attempt to reverse proxy upstream pypi turned | ||
out to be very unstable with increased complexity for deployers | ||
using proxies or offline installs. With this, the variables | ||
``repo_nginx_pypi_upstream`` and ``repo_nginx_proxy_cache_path`` | ||
have also been removed. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,5 @@ | ||
# {{ ansible_managed }} | ||
|
||
proxy_cache_path {{ repo_nginx_proxy_cache_path }}; | ||
|
||
upstream pypiserver { | ||
server localhost:{{ repo_pypiserver_port }}; | ||
} | ||
|
||
upstream pypi { | ||
server {{ repo_nginx_pypi_upstream }}; | ||
keepalive 16; | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters