-
Notifications
You must be signed in to change notification settings - Fork 45
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
Adding GCP support #1123
Adding GCP support #1123
Conversation
jschneier/django-storages#1193 Required PR: pulp/pulpcore#3367 [noissue]
|
||
def redirect_to_object_storage(self, artifact, return_media_type): | ||
""" | ||
Redirect to the passed artifact's file stored in the Azure storage. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Azure/GCP
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we have a pulpcore function for this now.
edit: Maybe it's not exactly compatible yet...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me, please once django-storages releases and pulpco PR is merged undraft this one and merge it. thanks!
And this needs issue please + changelog entry
it was just a draft |
why closing it if the code does not need any changes? |
without CI I don't know if it needs more changes, better to start a PR when we have a CI job for it |
jschneier/django-storages#1193
Required PR: pulp/pulpcore#3367
No Issue! Just a draft PR to have an idea on what is needed for GCP support