search

gatsbyjs / gatsby-source-wordpress-experimental

The upcoming v4 of gatsby-source-wordpress, currently in beta
MIT License
385 stars 100 forks source link

Feat: add cli that uses a list of users to run a preview swarm #336

Closed TylerBarnes closed 3 years ago

TylerBarnes commented 4 years ago

This CLI is run like wpgatsby preview swarm --wp-url="http://previewtest.local" --users="./users.json". It accepts a list of users from that JSON file and then proceeds to start swarming the WP/Preview instances by repeatedly simultaneously creating new posts and previewing in many different headless chrome windows.

KyleAMathews commented 4 years ago

EMBRACE THE CHAOS!!!

acao commented 3 years ago

I am able to re-create this in linux locally, and now am seeing no 50x errors or PHP errors from the wordpress docker image, only 200 and 302, which makes me still wonder if i have the network properly configured

TylerBarnes commented 3 years ago

@acao you shouldn't get any 50x errors since the preview failure on linux is due to the webhook not being hit. So I'm thinking it's probably still the network 🤔 if we're still totally stuck on this we can put it on hold for a while until we can find someone that's dealt with these cross platform networking issues before. This would be really nice to have but isn't a showstopper.

acao commented 3 years ago

@TylerBarnes would you like me to update the PR as well? im guessing we want to add the preview suite to github actions? is it just me or is it missing?

TylerBarnes commented 3 years ago

@acao I think we're good for now. I disabled the e2e test and added an @todo there. This is useable as a cli for debugging multi-user preview issues so it's valuable as-is and we can revisit getting it working in CI in the future

acao commented 3 years ago

ahh i see ok, yes that sounds fine to me!

TylerBarnes commented 3 years ago

Sounds good :) thanks for the review and reconciliation @acao!