Closed (fixed)
Project:
Hosting
Version:
6.x-0.4-alpha3
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 Sep 2010 at 01:40 UTC
Updated:
30 Sep 2010 at 09:10 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Anonymous (not verified) commentedThe 'deploying over existing site' logic introduced in this commit breaks that functionality (somehow).
Rather than investigate why the conditional logic doesn't ring true, it occurs to me that the new logic there is actually unnecessary: in every case of Migrate, you would *want* to delete the old directory in the post hook like we used to. Whether the site is being migrated or renamed, in both cases the old dir should be removed.
Clone is exempt because it doesn't call the migrate hooks, but its own invocation provision-deploy, so it's still safe.
I think we can remove the 'deploy_replace_site' conditional, but I missed a lot and may not be aware of something else that requires it here, so I'll wait for Vertice to chime in.
Comment #2
Anonymous (not verified) commentedThe problem is that the 'deploy_replace_site' logic is for when only the db server is changing. The logic relies on the idea that the platforms are identical. However, platforms *are* identical when simply renaming the site, so the potential for renaming the site and thus the requirement for deleting the old directory, is getting skipped.
Attached patch adds a 'deploy_rename_site' option if we are renaming. The 'deploy_replace_site' is not set if this is the case, allowing the old dir to be removed.
I'm worried if this potentially screws up if we *are* renaming the site and *also* changing the db server though :/
Comment #3
Anonymous (not verified) commentedConfirmed that Vertice has fixed this in HEAD.