cloud_controller_ng
cloud_controller_ng copied to clipboard
Upload buildpack_name.zip from URL rather than downloading file locally first.
Please provide details on the following items. Failure to do so may result in deletion of your feature request.
What's the user value of this feature request? A clear and concise description of what the user value is.
The value here would be speed of uploading the latest buildpacks to the PaaS blobstore.
Who is the functionality for? Who will use this feature?
PaaS System Engineers / Automated buildpack updates to the PaaS blobstore.
How often will this functionality be used by the user? How often will they use this feature?
We specifically would use this weekly.
Who else is affected by the change? Does this feature cause a breaking change? Are other users affected by this change?
Ultimately the buildpacks being uploaded to the PaaS blobstore faster does allow for developers to test changes faster.
Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
The problem we're seeing here is speed. Whenever we update buildpacks to the latest version weekly, the cf-cli first downloads the buildpack locally (from our internal artifactory instance) and then pushes that local file to the PaaS blobstore.
Describe the solution you'd like A clear and concise description of what you want to happen.
I'd like an IF condition to be added to the code so that whenever we supply the path as a URL, just have the cli upload those bits directly to the PaaS blobstore instead of first downloading the file locally.
Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.
N/A
Additional context Add any other context or screenshots about the feature request here.
N/A
We have created an issue in Pivotal Tracker to manage this:
https://www.pivotaltracker.com/story/show/172230282
The labels on this github issue will be updated when the story is started.
Hello @johnewhitley!
Thank you for opening this pull request, I contacted the API team in slack (https://cloudfoundry.slack.com/archives/C07C04W4Q/p1589818344299100), we download the buildpack bits because the API does not support providing a url, to change this we would need to add this functionality on the API side. I would encourage opening this same github issue in the CAPI repo (https://github.com/cloudfoundry/cloud_controller_ng).
I am closing this now but feel free to re-open if you feel I've closed it in error, thank you and sorry I couldn't be more help!
Jenna
We have created an issue in Pivotal Tracker to manage this:
https://www.pivotaltracker.com/story/show/172896511
The labels on this github issue will be updated when the story is started.
We have created an issue in Pivotal Tracker to manage this:
https://www.pivotaltracker.com/story/show/172896595
The labels on this github issue will be updated when the story is started.
Got this moved to the right repo, handing off to the API team now, thanks @johnewhitley