Mathieu Lamiot
Mathieu Lamiot
Indeed, it looks like the root cause could be the more detailed coverage. It's too bad there is not a way to keep the "simple" version for performance when the...
Discussed during the daily: This PR is needed for 3.17.2 as the rollback of the regression for SaaS visist was not done. So we should merge this to develop to...
Skipping grooming as it's pretty straightforward.
I would say we can clean them from the DB, if they exist, when updating to 3.19.1?
Ahah, if you think that's not a dumb idea, then let's go for it and implement it in this PR 👍 I'm just not sure 100% that we indeed don't...
Yes, correct @Mai-Saad , good catch. It might be better to keep this for 3.20 then. Let's get @DahmaniAdame's decision here
What I tested: Clearing all PerformanceHints tables manually (not triggering the warm-up) and then going to the URL visited by the SaaS ([http://mathieu.e2e.rocketlabsqa.ovh?nowprocket=1&no_optimize=1&wpr_imagedimensions=1](http://mathieu.e2e.rocketlabsqa.ovh/?nowprocket=1&no_optimize=1&wpr_imagedimensions=1)), I could see the hashes not added....
Ok thanks, so probably not related. It's a bit hard to test with manual visits because the hashes and beacon are added only if there is no data in the...
Created #6909 to cover issue n°1. Then we can circle back on this one for further investigation.
After #6909, I can't reproduce the issue n°2 on mathieu.e2e: I consistently have hashes for all entries except contact-us on mobile, which seems normal.