Closed (fixed)
Project:
Provision
Version:
6.x-1.0-rc2
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
25 Jan 2011 at 07:48 UTC
Updated:
31 Mar 2011 at 10:09 UTC
Jump to comment: Most recent
Comments
Comment #1
Anonymous (not verified) commentedThe only reason this doesn't work is because we don't queue up a verify of the sites on that platform afterward (all that is missing is that the DocumentRoot isn't rewritten for the site vhost. The platform apache config is rewritten as expected)
But I think we decided a while ago that we didn't want to auto-invoke verifications of sites if a platform node is edited, so in that case, yes we should make this uneditable.
Comment #2
Anonymous (not verified) commentedActually this problem is bigger:
Even if the platform name is changed, the context changes, and the Site drush aliases would reference the wrong platform.
So we should in fact auto-re-verify all sites, or make the platform node totally uneditable possibly.
EDIT: I am on crack, it seems we don't even change the context name if the platform name changes.So the alias remains the same as it was (this could even be a bug in itself or by design, don't know)
Comment #3
Anonymous (not verified) commentednow in git, publish path is not changeable for existing platforms.
Comment #5
alfthecat commentedSorry for editing a closed issue, however on RC1 I ran into the problem that, after making a typo in the publish path, there is no way for me to either get the platform out of the system, or correct the publish path.
So I had to change the platform directory to match my typo in order to be able to verify, delete and restore things.
Seems to me that this is a (minor) disadvantage to this method.
Comment #6
dsobon commentedMaybe it should be possible to change the publish path only if there are 0 sites or 1 site attached to the platform.
Or if the platform fails to verify.
Comment #7
anarcat commentedPlease open a new feature request for separate requests, and do not set issues to "needs review" unless there's a patch.
Comment #8
alfthecat commentedhttp://drupal.org/node/1111754