-
Notifications
You must be signed in to change notification settings - Fork 848
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
Bug: MongoDB error when attempting to use Tideways on fresh install #2697
Comments
Thanks for opening your first issue here! Be sure to follow the issue template and include your OS/Vagrant/VVV versions! Don't forget you can get support in the VVV slack at https://varyingvagrantvagrants.org/docs/en-US/slack/
|
My understanding of the Tideways provisioner was that if MongoDB wasn't installed that it would run the MongoDB provisioner to ensure this. @Mte90 do you have any thoughts? It's been a while since I used Tideways myself. The other thing that could have impacted this is PHP versions |
I've opened Varying-Vagrant-Vagrants/vvv-utilities#108 to handle Tideways in PHP v8.3, though the default PHP isn't 8.3 so it shouldn't impact this bug |
Actually this seems to be a known issue with php 8.1 (and, presumably, 8.2 and 8.3): Short StackOverflow discussion on the matter: I'm going to test with 8.0 and 7.x versions and see where I get.
|
So we have mongodb because it's how In the past we've explored this only to find that MongoDB is necessary for certain xhgui features, as were certain versions of PHP
|
it looks like this is the preferred storage mechanism now https://github.com/perftools/php-profiler?tab=readme-ov-file#upload-saver I also forgot that we'd already made the switch over and use PDO, so maybe mongodb isn't needed afterall? @Mte90 is removing mongodb feasible?
|
nevermind I realise now it's XHGUI that's the problem, not the collector, it supports MongoDB or PDO but using PDO removes features from the GUI, and composer expects the mongo PHP extension. On the plus side they explicitly support 8.0/8.1 now |
I'm seeing now that PDO is set as the default db option for tideways in VVV's xhgui config.
Meaning that what I have seen are separate errors. MongoDB warning
XHGUI/Tideways error/misbehaviour
|
My editors really struggle with that file but it's missing the error message and only contains the stack trace
This implies MongoDB can be jettisoned |
Ah! My bad. Here's the error belonging to that stack trace: |
0.21.3 is the latest version but our provisioner is pinned with:
does checkout out the 0.21.3 tag help with this? |
Varying-Vagrant-Vagrants/vvv-utilities#108 updates the PHP version used to install xhgui and the version it installs |
Unfortunately the above error occurs even after changing the provisioner script to pull xhgui 0.21.3 and when including the changes from Varying-Vagrant-Vagrants/vvv-utilities#108. Similarly, manually including the tableWatch parameter also still results in the same message ("Looks like you haven't done any profiling"). |
@arose-wt that's a shame, would you consider those changes an improvement though? Or did they introduce new issues? I'd like to see if they can be merged so we can continue debugging from a better place, and it sounds like we need to adjust the docs to remove the mongodb mention |
If you can do a quick PR upstream for that, I think the message about not getting any profiling is probably a separate issue related to our collector |
I've merged both PRs, does all work as expected for you? |
| I've merged both PRs, does all work as expected for you? I'm having unrelated system issues with my system's ssh/ssh-agent at the moment. |
Are you using the latest stable or develop branch version of VVV?
Yes (develop)
Is it a new VVV, or an existing VVV that used to work?
New fresh install
Did you use a CustomFile?
No (default)
Whats the problem?
Setup details:
config/config.yml
) but tried destroying and re-upping as wellChanges in
config/config.yml
:What occurs:
The
vagrant up --provision
output gets riddled with theUnable to load dynamic library 'mongodb.so'
warning shown below.Trying to access
tideways.vvv.test
results in the slim application error shown below.vagrant up --provision
log:full output
Slim Application Error when attempting to access the url for tideways:
Notice: Undefined index: tableWatch in /srv/www/default/xhgui/src/ServiceProvider/PdoStorageProvider.php on line 55
full output
Additional info:
That tideways error also occurred when I had uncommented
tideways
inextensions/core
but hadn't yet uncommentedmongodb
so, while it may well also be an issue, I am assuming it is unrelated to the mongodb.so issue.How do we reproduce it?
git clone -b stable https://github.com/Varying-Vagrant-Vagrants/VVV.git ~/vvv-local
cd ~/vvv-local
vagrant plugin install --local
- mongodb
and- tideways
(lines 87 and 88, respectively) inconfig/config.yml
vagrant up --provision
(Terminal displays error)
one.wordpress.test?enable-tideways=1
tideways.vvv.test
(Tideways error appears)
What is the output of
vagrant status
Which Operating System are you using?
Microsoft Windows
Which provider are you using?
VirtualBox 7
The text was updated successfully, but these errors were encountered: