Conversation
| {%- if test_gcp %} | ||
|
|
||
| cat >> credentials.json << GCP_JSON | ||
| { |
There was a problem hiding this comment.
just to understand this better, where this is taken from?
There was a problem hiding this comment.
There was a problem hiding this comment.
I pulled in these changes into my pr and the issue seems to persist pulp/pulp_container#1196
There was a problem hiding this comment.
it seems it needs more hacks:
https://github.com/fsouza/fake-gcs-server#using-with-signed-urls
|
@fao89 I don't think your changes are still enough, because pulp-container CI fails. Will you be able to test these changes on the example of pulp-container pulp/pulp_container#1196 |
fao89
force-pushed
the
gcpcred
branch
8 times, most recently
from
589a43c to
c83956b
Compare
[noissue] Required PR: pulp/pulpcore#3481 Signed-off-by: Fabricio Aguiar <fabricio.aguiar@gmail.com>
|
@pulp/deployments if someone can take this over and finalize, it would be great. We have already merged the corresponding PR in core that states GCP is supported pulp/pulpcore#3481 hovewer we do not have a proof of a successful flow because the CI part was not done. |
|
actually, this PR is more related to @pulp/continuous-integration team |
2 participants
https://django-storages.readthedocs.io/en/latest/backends/gcloud.html#authentication
Required PR: pulp/pulpcore#3481
Signed-off-by: Fabricio Aguiar fabricio.aguiar@gmail.com