Closed (fixed)
Project:
Panopoly
Version:
7.x-1.x-dev
Component:
Admin
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Dec 2013 at 16:23 UTC
Updated:
9 Feb 2016 at 06:12 UTC
Jump to comment: Most recent
Here is the full log from Travis-CI:
https://travis-ci.org/lsolesen/panopoly/jobs/16105934
And here is the important error:
WD menu: PDOException: SQLSTATE[42S02]: Base table or view not found:[error]
1146 Table 'drupal.pathauto_state' doesn't exist: SELECT entity_id,
pathauto FROM {pathauto_state} WHERE entity_type = :entity_type AND
entity_id IN (:entity_ids_0, :entity_ids_1); Array
(
[:entity_type] => search_api_index
[:entity_ids_0] => 1
[:entity_ids_1] => 2
)
in pathauto_entity_state_load_multiple() (line 436 of
/home/travis/build/lsolesen/drupal/profiles/panopoly/modules/contrib/pathauto/pathauto.module).
This may be related to: #1979558: Update pathauto patch
Comments
Comment #1
dsnopekImplemented
hook_update_dependencies()in panopoly_core and panopoly_user to make sure update functions happen in the correct order.Comment #2
dsnopekUgh, this appears to still happen on Travis-CI for 1.0-rc3 (but strangely not 1.0-rc2) and I can't recreate it locally - works fine for me locally. :-/
Here is the Travis-CI log:
https://travis-ci.org/lsolesen/panopoly/jobs/16833273
Comment #3
dsnopekAdded a hook_update_dependencies() hack to panopoly_widgets and now it's getting past the update functions finally! :-) Marking as Fixed again.
Comment #5
mducharme commentedSorry to have to reopen this. This is still happening for my upgrade from rc5 to 7.x-1.1+23-dev
Comment #6
dsnopekHrm. We have tests that attempt to upgrade from 1.0-rc5 on every commit and they're running fine:
https://travis-ci.org/lsolesen/panopoly/jobs/19265484
Are you running straight Panopoly or is a distribution built on Panopoly? Can you give some more information about your site? Are you using any patches to Panopoly or Pathauto (beyond what Panopoly includes)?
Comment #7
mducharme commentedIt's a straight panopoly installation without any other distributions, profiles and the only patched panopoly module is entityreference (to fix a field_collection issue). I do have a lot of additional contrib in sites/all/modules/contrib that are enabled on my site but this pathauto_state problem is the only error I get when running updates.
I don't mind trying something drastic if you have any ideas. I have lots of backups.
Comment #8
mducharme commentedOK, I got this working but it did require the following workaround:
sites/all/modules/contrib/pathautosites/all/modules/contrib/version of pathautosites/all/modules/contrib/pathautoI'll let you decide whether the issue should be closed again or if you'd like to further review the issue based on this result.
Comment #9
bojanz commentedYou are running into #2230419: Distribution updates run in an unpredictable order.
Comment #10
dsnopekSince @mducharme found a solution to their problem and we can't reproduce this in our tests (well, not anymore, since we fixed the issue originally), I'm setting this back to fixed.
Comment #11
lunazoid commentedI just wanted to post my experience with this issue. We are updating from Panopoly rc4 to 1.2, and encountered this problem (it appears as though rc5 is when the table name changed from 'pathauto' to 'pathauto_state'.) The workaround used by @mducharme in #8 didn't work for me, as the table still wasn't created. What I wound up doing was hacking the code a bit to add a conditional statement to the 3 functions that access the database table. If the 'pathauto_state' table existed, it would use it, otherwise, it would use 'pathauto.' This was only necessary to keep the site alive so I could run drush updb. Once the database updates ran, the table was created, and the conditionals could be removed.
I could post a patch if anyone needs it, but this is really just a simple hack to allow the database updates to run and probably won't affect many users.
Comment #12
electrokate commented#8 worked for me, thanks for posting, mducharme!
Comment #13
dsnopekWow, this is more delicate than I thought! Appearantly, adding an update function to a completly unrelated module can reorder the update functions in such a way that this error happens. Here's an example from our upgrade tests on Travis-CI:
https://travis-ci.org/panopoly/panopoly/jobs/23551978
This commit added
panopoly_images_update_7102(), and now upgrades from 1.0-rc4 to latest -dev break.So, basically, having any contrib modules beyond what's bundled in Panopoly could break upgrade.
The only real fix would be a change to Drupal core to make update order predictable per @bojanz's issue: #2230419: Distribution updates run in an unpredictable order
For now, I'll probably add another
hook_update_dependencies()hack...Comment #15
liza commentedugh... i just got this message while trying to finish an update. how can i tell the order of the updates? i have a quite a number of modules outside of the Panopoly bundle :P
Comment #16
dsnopek@liza: If you run the updates via drush (ie.
drush updb) then it'll print out each update function as it runs them. You can copy and paste that output and that'll tell us the order that the update functions are running in.It's unfortunate that this issue is still happening. I was kinda hoping that the latest updates to the pathauto patch would prevent this, but it looks like no. :-/
Comment #17
liza commented*sigh* still have this problem with 1.4.
here's the print out to the update order
will try out #8 and report back
*siiiiiiiiiiiiiiiihg*
Comment #18
liza commentedtried a few variations of #8 and nothing works. suggestions?
Comment #19
liza commentednot solved for me wit 1.4
Comment #20
dsnopekJust to be clear, this means you're updating from Panopoly 1.4 to 1.14? Or from 1.0-rc5 to 1.14 (and "1.4" was a type-o)?
Comment #21
manishs commentedRunning /update.php fixed this error for me.