remote_directory fails trying to purge a non-empty directory

Details

Description

remote_directory fails trying to purge a non-empty directory
remote_directory fails if "purge" is set to true and the destination directory is not empty. e.g.
trying to copy to 

A
|_B
  |_C

fails stating "B" is not empty. The failure is primarily because Directory.rm does not force delete.
FileUtils.rm_rf may be a better bet.

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. CHEF-3028 remote_directory purge fails with managed folders only containing folders

  • Unknown - Issue untriaged, priority unassigned or questioned
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Prashant Srivastava added a comment -

Damn you wiki markup!
The folder structure intended was

A -> B -> C

Show
Prashant Srivastava added a comment - Damn you wiki markup! The folder structure intended was A -> B -> C
Hide
Permalink
Laurent Désarmes added a comment -

confirmed here. it fails with the directory resource attribute recursive set to true as well.

Show
Laurent Désarmes added a comment - confirmed here. it fails with the directory resource attribute recursive set to true as well.
Laurent Désarmes made changes -
Field Original Value New Value
Assignee Laurent Désarmes [ laurent ]
Laurent Désarmes made changes -
Status Open [ 1 ] In Progress [ 3 ]
Hide
Permalink
Laurent Désarmes added a comment -

proposed fix in llaurent/CHEF-2599

Show
Laurent Désarmes added a comment - proposed fix in llaurent/ CHEF-2599
Laurent Désarmes made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Hide
Permalink
Laurent Désarmes added a comment -

i haven't fixed it by forcing the rmdir.

A/B/C/file being in the remote directory
the provider was only removing A/B/C and A/B/C/file from the purge_list.
the fix i propose also removes A/B and A from the purge list.
If A/B/C/file is in the remote directory, there's no reason for the provider to remove A (or A/B)

Can you confirm it really fixes the issue you're having ?

Show
Laurent Désarmes added a comment - i haven't fixed it by forcing the rmdir. A/B/C/file being in the remote directory the provider was only removing A/B/C and A/B/C/file from the purge_list. the fix i propose also removes A/B and A from the purge list. If A/B/C/file is in the remote directory, there's no reason for the provider to remove A (or A/B) Can you confirm it really fixes the issue you're having ?
Daniel DeLeo made changes -
Description remote_directory fails trying to purge a non-empty directory
remote_directory fails if "purge" is set to true and the destination directory is not empty. e.g.
trying to copy to
A
|_B
  |_C

fails stating "B" is not empty. The failure is primarily because Directory.rm does not force delete.
FileUtils.rm_rf may be a better bet.
  		remote_directory fails trying to purge a non-empty directory
remote_directory fails if "purge" is set to true and the destination directory is not empty. e.g.
trying to copy to
{noformat}
A
|_B
  |_C
{noformat}

fails stating "B" is not empty. The failure is primarily because Directory.rm does not force delete.
FileUtils.rm_rf may be a better bet.
 
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

This should of course be tested, but we also should really have a unit test to verify and protect against regression here.

Show
Bryan McLellan [Opscode] added a comment - This should of course be tested, but we also should really have a unit test to verify and protect against regression here.
Bryan McLellan [Opscode] made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Hide
Permalink
Laurent Désarmes added a comment -

Ok, i've updated llaurent/CHEF-2599 with a simple testcase, that's enough to fail 2 existing tests.
https://github.com/llaurent/chef/commits/CHEF-2599/

Show
Laurent Désarmes added a comment - Ok, i've updated llaurent/ CHEF-2599 with a simple testcase, that's enough to fail 2 existing tests. https://github.com/llaurent/chef/commits/CHEF-2599/
Hide
Permalink
Laurent Désarmes added a comment -

i'm resolving it, i don't want it to be missed

Show
Laurent Désarmes added a comment - i'm resolving it, i don't want it to be missed
Laurent Désarmes made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Marked to be merged.

Show
Bryan McLellan [Opscode] added a comment - Marked to be merged.
Bryan McLellan [Opscode] made changes -
Triage Status [Triaged]
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Merged to master, thanks Laurent!

Show
Bryan McLellan [Opscode] added a comment - Merged to master, thanks Laurent!
Bryan McLellan [Opscode] made changes -
Status Resolved [ 5 ] Closed [ 6 ]
Fix Version/s 0.10.10 [ 10702 ]
Bryan McLellan [Opscode] made changes -
Link This issue is duplicated by CHEF-3028 [ CHEF-3028 ]
Bryan McLellan [Opscode] made changes -
Component/s Chef Client [ 10000 ]
Component/s Resources [ 10004 ]
Bryan McLellan [Opscode] made changes -
Workflow jira [ 14321 ] New OSS [ 20208 ]
Bryan McLellan [Opscode] made changes -
Workflow New OSS [ 20208 ] OSS: Product Management [ 28123 ]
Transition Time In Source Status Execution Times Last Executer Last Execution Date
Open Open In Progress In Progress
154d 18h 6m 1 Laurent Désarmes 09/Feb/12 1:02 PM
In Progress In Progress Resolved Resolved
18m 6s 1 Laurent Désarmes 09/Feb/12 1:20 PM
Resolved Resolved Reopened Reopened
6d 5h 52m 1 Bryan McLellan [Opscode] 15/Feb/12 7:13 PM
Reopened Reopened Resolved Resolved
15d 5h 12m 1 Laurent Désarmes 02/Mar/12 12:25 AM
Resolved Resolved Closed Closed
32d 19h 5m 1 Bryan McLellan [Opscode] 03/Apr/12 7:30 PM

People

  • Assignee:
    Laurent Désarmes
    Reporter:
    Prashant Srivastava
Vote (1)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: