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
Update CI + enable https #304
Conversation
|
WARNING!!! This PR is not attached to an issue. In most cases this is not advisable. Please see our PR docs for more information about how to attach this PR to an issue. |
0b0e00d
to
b65c6fc
Compare
|
🎉 |
|
Blocked by pulp/pulp-oci-images#73 or pulp/pulp-oci-images#84 |
c0e40b9
to
ae411a0
Compare
|
@pulp/container-plugin I merged the SSL changes, I rebased this PR, but it is broken. |
|
"413 Request Entity Too Large" There is some |
it 'is |
|
from SSL single container image: location /pulp/api/v3/ {
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
proxy_set_header X-Forwarded-Proto $scheme;
proxy_set_header Host $http_host;
# we don't want nginx trying to do something clever with
# redirects, we set the Host: header above already.
proxy_redirect off;
proxy_pass http://pulp-api;
client_max_body_size 10m;
}https://github.com/pulp/pulp-oci-images/blob/latest/assets/ssl_nginx.conf#L75 |
c5d0937
to
902a593
Compare
[noissue]
|
The issue is here: https://github.com/pulp/plugin_template/blob/master/templates/github/.ci/ansible/Containerfile.j2.copy#L26-L27 |
|
it is ready for review now! |
[noissue]