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

Bugs during recursive DELETE #752

Closed
adavydow opened this issue Apr 26, 2019 · 9 comments
Closed

Bugs during recursive DELETE #752

adavydow opened this issue Apr 26, 2019 · 9 comments
Assignees
Labels

Comments

@adavydow
Copy link
Contributor

I tried to remove a directory, but the following error occurred:

 neuro rm storage:results/synthetic-reverse/synth_scheduler/lr02_gamma05step30
Using path 'storage://adavydow/results/synthetic-reverse/synth_scheduler/lr02_gamma05step30'

ERROR: Illegal argument(s) ({"error": "Unexpected exception: [Errno 39] Directory not empty: 'logs'. Path with query: /api/v1/storage/adavydow/results/synthetic-reverse/synth_scheduler/lr02_gamma05step30?op=DELETE."})

All future attempts to remove the directory result in:

neuro rm storage:results/synthetic-reverse/synth_scheduler/lr02_gamma05step30
Using path 'storage://adavydow/results/synthetic-reverse/synth_scheduler/lr02_gamma05step30'

ERROR: Illegal argument(s) ({"error": "Unexpected exception: [Errno 16] Device or resource busy: '.nfs0000000006fa64e000000001'. Path with query: /api/v1/storage/adavydow/results/synthetic-reverse/synth_scheduler/lr02_gamma05step30?op=DELETE."})
)
@adavydow
Copy link
Contributor Author

adavydow commented Apr 26, 2019

Same bug with another directory.

(synthetic-reverse) adavydow-mac:synthetic-reverse adavydow$ neuro rm storage:results/synthetic-reverse/albu/basic
Using path 'storage://adavydow/results/synthetic-reverse/albu/basic'

ERROR: Illegal argument(s) ({"error": "Unexpected exception: [Errno 39] Directory not empty: 'logs'. Path with query: /api/v1/storage/adavydow/results/synthetic-reverse/albu/basic?op=DELETE."})

(synthetic-reverse) adavydow-mac:synthetic-reverse adavydow$ neuro rm storage:results/synthetic-reverse/albu/basic
Using path 'storage://adavydow/results/synthetic-reverse/albu/basic'

ERROR: Illegal argument(s) ({"error": "Unexpected exception: [Errno 16] Device or resource busy: '.nfs0000000006fa64f700000002'. Path with query: /api/v1/storage/adavydow/results/synthetic-reverse/albu/basic?op=DELETE."})

@dalazx
Copy link
Contributor

dalazx commented Apr 30, 2019

thanks @adavydow. we'll take a look shortly.

@anayden anayden self-assigned this May 7, 2019
@anayden
Copy link
Contributor

anayden commented May 7, 2019

@adavydow how reproducible is that for you? every time or very occasionally?

I have two theories about the issue:

  • Some files within the directory are marked read-only
  • Files are being added/modified/synchronized via NFS while deletion is happening (i.e., there's a race between list+rm operations vs. changes in the filesystem).

@anayden
Copy link
Contributor

anayden commented May 13, 2019

I discussed the issue with Alexey Davydow and he's unable to reproduce it now. I added extra logging for the exception and we'll be able to troubleshoot it better when/if it happens again.

@anayden anayden closed this as completed May 28, 2019
@anayden anayden reopened this May 28, 2019
@anayden
Copy link
Contributor

anayden commented May 28, 2019

@adavydow could you please update the issue with the problem you encountered last week?

@adavydow
Copy link
Contributor Author

adavydow commented Jun 5, 2019

Encountered the problem again.
Additional information: this problem occurs more often under a high load.
It’s not a neuro rm problem. Deletion from inside the container doesn’t work too.

@asvetlov
Copy link
Contributor

asvetlov commented Jun 5, 2019

Note, the latest client allows to delete directories with --recursive flag only.
It doesn't close the problem, though. The deletion code is the same still

@dalazx
Copy link
Contributor

dalazx commented Jul 1, 2019

the sticky sessions functionality should've resolved the issue. closing for now.

cc @asvetlov @anayden

@dalazx dalazx closed this as completed Jul 1, 2019
@serhiy-storchaka
Copy link
Contributor

I have encountered the same issue in #918. It was fixed by repeating the removing. It would be better to repeat it in platform-storage instead of client.

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

6 participants