We no longer support using upgrade.php for major version upgrades. Accordingly, the section of UPGRADE.txt that deals with major version upgrades needs to be revised.

Actually, most of the text should be removed, and replaced with one sentence saying that it is not possible to upgrade a site from one major version to another in place, and that what you have to do instead is build a new Drupal 8 site, and then use the Migrate module to migrate your data from the old version of Drupal to the new site.

The rest of this is the original issue summary, which is largely irrelevant now
Relevant text, under major version upgrades:

1. Check on the Drupal 8 status of your contributed and custom modules and
   themes. See http://drupal.org/node/948216 for information on upgrading
   contributed modules and themes. See http://drupal.org/node/895314 for a list
   of modules that have been moved into core for Drupal 8, and instructions on
   how to update them. See http://drupal.org/update/modules for information on
   how to update your custom modules, and http://drupal.org/update/theme for
   custom themes.

   You may decide at this point that you cannot upgrade your site because
   needed modules or themes are not ready for Drupal 8

There are a few problems with this:

1) Makes reference to http://drupal.org/node/948216 and http://drupal.org/node/895314 but these pages are only for D6 -> D7, not D7 -> D8. We need new equivalents.

2) Makes reference to http://drupal.org/update/modules and http://drupal.org/update/themes, which we don't use anymore, in favour of http://drupal.org/list-changes instead.

3) That last sentence doesn't end in a period and it DRIVES ME TO THE BRINK OF MAAAAAADNESSSSS!

:)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

A little bit more:

16. Replace and update your non-core modules and themes, following the
   procedures at http://drupal.org/node/948216

...also makes reference to that link.

References to sites/*/modules and the like should probably be checked as well since in D8 the recommended practice is to use the top-level directories.

jhodgdon’s picture

This seems like a release blocker but not urgent for the moment. Do we have a tag for that?

star-szr’s picture

Issue tags: +revisit before beta

@jhodgdon - I think this fits for #1393358: [META-12] Review 'revisit before release candidate' tag, tagging.

catch’s picture

Category: Bug report » Task
Priority: Normal » Critical
Issue summary: View changes
Issue tags: -revisit before beta

This now needs all reference to major upgrades completely removed, we can mention migrate in a single sentence somewhere.

jhodgdon’s picture

"mention migrate in a single sentence somewhere"... Um... So you mean we are not going to have a README that gives an outline of how to take a D6/7 site and migrate it to 8?!? That seems like a major omission.

catch’s picture

We're not going to have that until there's a release that includes migrate, no. That release may well be 8.1.x, but we need to remove references to the old major upgrade path before release. So yes eventually, but not in this issue no.

jhodgdon’s picture

Issue summary: View changes
Issue tags: +Novice

How fun! This is most likely a good Novice issue, and a Critical. Good combination. :)

Updated issue summary to reflect what we actually want to do here.

paulh’s picture

I'll give it a shot.

paulh’s picture

Assigned: Unassigned » paulh
paulh’s picture

Okay, doco updated and patched.

paulh’s picture

Status: Active » Needs review
jhodgdon’s picture

Status: Needs review » Needs work

Thanks for the patch!

Let's clean up the verb tenses here a bit. The patch says things like "will not be possible"; the fact is, it is already not possible. I also don't think we want to talk about how support for upgrade.php has been "withdrawn". Let's just document that the process for major versions is migration, in contrast to the process for minor versions, which uses update.php.

Really, we just want a sentence saying that and saying that migration support is currently only partially implemented. Don't make any references to "withdrawing" or promises for the future. Thanks!

paulh’s picture

Got a bit carried away with the previous one :).

Updated as per suggestions and re-rolled the patch.

paulh’s picture

Status: Needs work » Needs review
jhodgdon’s picture

Status: Needs review » Needs work

This looks great to me! Except we (tradition) use American spelling/grammar in Drupal docs, so it should say "utilizing" instead of "utilising".

paulh’s picture

British English spell checkers :). Updated and re-rolled.

paulh’s picture

Status: Needs work » Needs review
jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

That looks good to me, thanks!

webchick’s picture

I'm skipping over this one in my RTBC queue sprint because I presume jhodgdon would like to commit it. If that's wrong, let me know!

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 8.x -- thanks again all!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.