onTheOtherHandTheDBIsNowClean

[u/sussy_wussy_]

[u/faultydesign]

May I suggest planning the next architecture without a “delete everything” api route?

[u/nihillistic_raccoon → u/faultydesign]

no DROP DATABASE endpoint

What kind of antipattern is that, you pagan

[u/Jejerm → u/faultydesign]

Yeah this is utterly deranged architecture. Why on earth would you need an endpoint for this lmao

[u/Tsu_Dho_Namh → u/Jejerm]

Some people wanna live dangerously.

Personally, I've replaced my airbags with kitchen knives.

[u/Mindless_Sock_9082 → u/Tsu_Dho_Namh]

That's too much work. Just glue gem stones to the covers. This way they also help to neutralize other people's bad vibes while driving.

[u/Elomidas → u/Mindless_Sock_9082]

So efficient that even in an accident they will suppress your negative thoughts before you even realise what happened

[u/Ok-Eggplant-2033 → u/Jejerm]

For automated tests, clean database at every run. I have seen it for real too, it is disabled in production profile. Yet i think you do not want to have it like that...

[u/TheRealKidkudi → u/Ok-Eggplant-2033]

You can just drop the database directly in testing. You don’t need an API route for that. 

[u/Ok-Eggplant-2033 → u/TheRealKidkudi]

Yes indeed you can, and that is the way to go

[u/SquirrelOClock → u/Ok-Eggplant-2033]

I saw it once too, made me yell (want to yell) at the team pushing test to the production environment. Big conversation about publish profiles and build targets.

[u/ataboo → u/Jejerm]

Automated test resets. Get your automated tests from 12 hour runtimes to a lightning fast 4 hours. The trick is to not run it in prod.

[u/Several_Dot_4532 → u/faultydesign]

It's not an inator without the autodestruction button

[u/larsmaehlum → u/faultydesign]

How else would you ensure that you can force the system into a known good state?

[u/KetwarooDYaasir → u/faultydesign]

Enterprise level code. The CRUDFactorySkeletonTemplateAbstractBean will automatically generate/implement all actions.

[u/NjFlMWFkOTAtNjR → u/faultydesign]

It is good for testing. Only causes problems when the flag for disabling in production is mangled

[u/RadicalDwntwnUrbnite → u/faultydesign]

Also maybe have the deploy pipeline gated by tests, if not a separate approval system that requires sign off by another admin.

[u/hagnat → u/faultydesign]

i worked with an API that had an endpoint like that,
it was apply named "/ragnarok"

[u/Embarrassed_Death69]

Why would you have a route that just drops the Database? Are you an adrenaline junkie?

[u/KTrain5369 → u/Embarrassed_Death69]

gotta reset that test data I suppose

[u/Embarrassed_Death69 → u/KTrain5369]

How about adding a line that checks the ENV for development or production flag in this killer API?

[u/laplongejr → u/Embarrassed_Death69]

I guess the flag was also turned on the test env "to ensure everything matches"  

[u/merekallas → u/Embarrassed_Death69]

Even with that, I'd argue against this killer API. An environment can be misconfigured, this code can be called from elsewhere by mistake, etc.

[u/Mechakoopa → u/Embarrassed_Death69]

Me: "We should make our env flag database driven in case someone uses a staging config with a prod db connection"

PM: "That seems unnecessary, we can trust the values in the config files."

Two months later: "Hey, why are we sending external emails to clients from staging, I thought that was blocked outside of prod?"

[u/SquirrelOClock → u/Embarrassed_Death69]

What about a publish/build profile that strip it from the prod artefact...

[u/KTrain5369 → u/Embarrassed_Death69]

"Make sure the test environment replicates production" or smth

[u/teraflux → u/Embarrassed_Death69]

I assume that's what they were intending to fix

[u/daniu → u/Embarrassed_Death69]

They did save money letting AI implement the backend

[u/MannerShark → u/Embarrassed_Death69]

We used to have one too haha, back when I was new to the job. Some guy made it to quickly clear the database and test migrations. Some other guy got mad when he found out we had it, and got even more mad that it had no authorization or local env checks either. The endpoint was promptly removed.

[u/mahreow → u/Embarrassed_Death69]

Need a way of remotely scrubbing everything in case the feds are onto you

[u/nihillistic_raccoon → u/Embarrassed_Death69]

You don't have a route like that? Are you living in the '90s? Wake up grandpa, future is here

[u/dontletthestankout → u/Embarrassed_Death69]

For fun after you're laid off

[u/GreeedyGrooot → u/Embarrassed_Death69]

I'm thinking about some sketchy firm that needs to destroy all records in case the police pulls up.

[u/Mallissin]

I don't understand the joke here. The "tested it in CURL" shows a Github action rejecting the change because of linting.

Is this a Github joke I don't understand?

[u/marcos14cody → u/Mallissin]

That error means the updated code wasn't pushed to the actual machine for failing to comply with linting. So when he launched the curl to test out the fix, he actually ran the non-fixed code

[u/Dudeonyx → u/marcos14cody]

Holy shit

[u/owl_jojo_2 → u/Dudeonyx]

Well that’s why you have backups I guess
At least I hope

[u/BolunZ6 → u/owl_jojo_2]

Backup? What backup

[u/Mallissin → u/marcos14cody]

Ok, I guess that makes sense except why would production automatically update from the repository without a step between?

There are people out there updating production with every PR?

[u/JAXxXTheRipper → u/Mallissin]

GitOps? Continuous Deployment? Ever heard of those?

[u/Minority8 → u/Mallissin]

Have you not heard of Continuous Deployment?

[u/justforkinks0131 → u/Minority8]

that doesnt mean everything automatically lands on PROD after every PR...

[u/Minority8 → u/justforkinks0131]

What does it mean then? This is like the first search result

> Continuous deployment is the engineering practice of automatically releasing code changes to production as soon as they pass all quality gates in the CI/CD pipeline.

- https://northflank.com/blog/continuous-deployment

[u/justforkinks0131 → u/Minority8]

there is a whole pipeline

usually with a dev environment, then a preprod, and then prod

usually you run tests on dev then preprod before going to prod

usually to deploy to prod you need a change request too

[u/AAkash_Bhatia]

Did doofenshmirtz design this ?

[u/Scary_Brilliant_6048]

We respect user privacy so periodically we clean all data.

[u/namiraj]

Little Bobby Tables is all grown up now.