cloud_controller_ng icon indicating copy to clipboard operation
cloud_controller_ng copied to clipboard

Upload buildpack_name.zip from URL rather than downloading file locally first.

Open johnewhitley opened this issue 4 years ago • 5 comments

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

johnewhitley avatar Apr 08 '20 17:04 johnewhitley

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.

cf-gitbot avatar Apr 08 '20 17:04 cf-gitbot

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

JenGoldstrich avatar May 18 '20 16:05 JenGoldstrich

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.

cf-gitbot avatar May 18 '20 16:05 cf-gitbot

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.

cf-gitbot avatar May 18 '20 16:05 cf-gitbot

Got this moved to the right repo, handing off to the API team now, thanks @johnewhitley

JenGoldstrich avatar May 18 '20 16:05 JenGoldstrich