and then the CI fails because the fucking test env collapsed since the billion dollar company won't cough up the cash to fix their CI/Testing infrastructure.

[–] BassTurd@lemmy.world 13 points 1 week ago

Kind of related, but I spent about 5 weeks straight a couple months back working on our Playwright tests at work. Lots and lots of race conditions, many because of poor code since nobody knew the framework well, but many were just uncontrollable server side and network issues. I modified so much code and optimized it to the point that the entire pipeline was running in about half the original time, and I was getting successful consecutive pipeline runs. I found out that the test server is a VM with 1 CPU core and 8gb of RAM. Playwright by default will run all test files in parallel but each individual test in each file in serial, so we had about 10 tests trying to run frontend tests at the same time on that hardware. I think per Playwrights recommendations, it should be something like half as many cores as concurrent tests or similar, so we weren't close.

Anyway all of that text to say I was able to get that increased from 1 to 2 cores, and 8 to 16gb of RAM. It's still getting hammered, and it's still having issues, but it's no longer my problem, so the company gets what they're paying for.

[–] one_old_coder@piefed.social 21 points 1 week ago (5 children)

Disable the CI when a PR is still a draft.

[–] death_to_carrots@feddit.org 21 points 1 week ago (1 children)

Actual helpful advise? In my meme community?

[–] MaggiWuerze@feddit.org 4 points 1 week ago

In this part of the country?

[–] Skullgrid@lemmy.world 15 points 1 week ago

yeah, OP has those kinds of privileges I'm sure.

[–] MostlyBlindGamer@rblind.com 9 points 1 week ago (1 children)

And force me to compile on my own computer!? I’ll find out I broke treated features via email, thank you very much!

[–] one_old_coder@piefed.social 5 points 1 week ago (1 children)

You can have a special button to force the build.

[–] MostlyBlindGamer@rblind.com 2 points 1 week ago

No. I want it to build every time I push. I already have a button for that on my computer and I don’t want to click that either. Don’t take this away from me!

[–] jtrek@startrek.website 6 points 1 week ago

I leave my PR in draft until the checks pass. I don't want anyone reviewing it when it's broken.

[–] kibblebits@quokk.au 4 points 1 week ago

Make things less complicated.

[–] rizzothesmall@sh.itjust.works 8 points 1 week ago

The way it should be. You change a line, you test everything. If you can't test everything yourself, the CI does it for you.

[–] ryathal@sh.itjust.works 5 points 1 week ago

You aren't going to benefit from fixing it anyway so who cares.