Uh oh!
There was an error while loading. Please reload this page.
Add GCP fake credentials - #720
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
ipanova
commented
Jan 9, 2023
@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 |
589a43c to
c83956bCompare[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. |
fao89
commented
Jan 20, 2023
actually, this PR is more related to @pulp/continuous-integration team |
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