Skip to content
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

Issue with the CLI removing a non-empty directory #1158

Closed
cfjedimaster opened this issue Jun 2, 2017 · 4 comments
Closed

Issue with the CLI removing a non-empty directory #1158

cfjedimaster opened this issue Jun 2, 2017 · 4 comments
Labels

Comments

@cfjedimaster
Copy link

Command

cf push

What occurred

Uploading app files from: /mnt/c/Users/ray/Desktop/nodeexample2
Uploading 10.6M, 10980 files
FAILED
Error processing app files: Error uploading application.
remove /tmp/apps473228129/node_modules/eslint/lib/rules: directory not empty

What you expected to occur

For it not to error. ;)

CLI Version

cf version 6.26.0+9c9a261.2017-04-06

CC API Endpoint Version

api endpoint: https://api.ng.bluemix.net
api version: 2.54.0

CF Trace

Nothing seemed helpful here.

Platform & Shell Details

Windows Subsystem for Linux

Any other relevant information

If I remove my own node_modules locally, it works.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/146528371

The labels on this github issue will be updated when the story is started.

@dkoper
Copy link

dkoper commented Jun 3, 2017

Hi @cfjedimaster

We've seen issues on Windows with deep paths. They seem to have been resolved in Golang 1.8, with which we are currently building (post cf CLI 6.26.0 release).
You could try downloading an edge release and see if you can reproduce the issue.
If the issue is still there, we're refactoring the push command and trying to avoid the creation of all those temporary files, which should definitely fix it. But that's still some time out.

Regards,
Dies Koper
CF CLI PM

@dkoper dkoper added the bug label Jun 3, 2017
@cfjedimaster
Copy link
Author

I can confirm the edge release helped - thanks.

@dkoper
Copy link

dkoper commented Jun 9, 2017

Resolved in 6.27.0. Thanks for confirming on the edge release.
https://github.com/cloudfoundry/cli#downloads

@dkoper dkoper closed this as completed Jun 9, 2017
@cf-gitbot cf-gitbot removed the accepted label Jun 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants