Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for failing integration tests. #76

Merged
merged 2 commits into from
Nov 12, 2024
Merged

Conversation

nigelgbanks
Copy link
Contributor

I was able to debug this, thanks to the SQL dump provided by @joecorall on #71.

Though, this is just a patch over the problem to account for drush's behaviour. The site seems to install correctly regardless, but afterwards the cache is in a bad state, but only occasionally. Why, I do not know. I've not been able to catch it in the act. Though not relying on drush but rather drupal to do the initial rebuild seems to deal with the problem (for a stretched definition of deal). Something about drush install is not deterministic, or at the very least, is negatively impacted by access to resources or something of the like.

@nigelgbanks nigelgbanks marked this pull request as ready for review November 12, 2024 16:46
@joecorall joecorall merged commit 0a24db0 into main Nov 12, 2024
3 checks passed
@joecorall joecorall deleted the issue-66-bad-cache-state branch November 12, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants