Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I just did a beta2 -> rc1 upgrade and my main aegir site wasn't put offline. I looked at the code quickly, and I suspect this is the case for all migrations.
In platform/migrate.provision.inc:
drush_log(dt("Putting site under maintenance"));
drush_set_option('site_offline', 1);
_provision_drupal_create_settings_file();
drush_invoke('provision-backup');
<?php if (!$this->site_enabled) : ?>
$conf['site_offline'] = 1;
<?php endif ?>
I think we meant ->site_offline there, but that needs to be tested. Marking this as critical as it can result in dataloss if somebody posts content on the site after the backup is performed.
Comments
Comment #1
EugenMayer CreditAttribution: EugenMayer commentediam confirming this bug.
Comment #2
anarcat CreditAttribution: anarcat commentedI pushed a fixed to the master branch on git.drupal.org, please test master, then I will merge to 0.4, and we can make rc2.
I think this regression is related to #955018, because the "site_offline" variable wasn't propagated to the settings.php properly. I tried setting it in the site context (by using
drush_set_option('site_offline', 0, 'site')
) but that wouldn't work either. Oh and I tried of course that and using$this->site_offline
in the settings.php - no luck.I think that's becase
site_offline
is not a registered property of the site context... hence the relation with #955018: ability to save new arbitrary data to a context from outside the service.Comment #3
anarcat CreditAttribution: anarcat commented@EugenMayer - can you test if you still have this bug now, with the latest code on git.drupal.org?
Comment #4
Anonymous (not verified) CreditAttribution: Anonymous commentedI tested it and it works.
Comment #5
omega8cc CreditAttribution: omega8cc commentedI see this change in the master on http://drupalcode.org but this commit is not listed on http://drupal.org/node/196005/commits which is rather confusing.
[EDIT] screenshots added:
The code: https://skitch.com/omega8cc/r1bjn/drupalcode.org-git-project-provision.g...
The list of commits: https://skitch.com/omega8cc/r1be4/commits-for-provision-drupal.org
Comment #6
anarcat CreditAttribution: anarcat commentedalright, I merged the commit in the 0.4 branch - looks like we're mostly ready for rc2! :)
@omega8cc - i don't know what is the cause of that problem, but it has nothing to do with this issue. ;) i ping sdboyer about it though, we'll see...