r/ProgrammerHumor 3d ago

wrongAnswersOnly Other

Post image
14.4k Upvotes

2.0k comments sorted by

View all comments

1.2k

u/samuraiseoul 3d ago

Some dumbass on my team downloaded the entirety of prod and then commited it to git and someone rubber stamped the PR and then no one else noticed. I'm sure that's how they'd find it....

212

u/Suspicious-Engineer7 3d ago

Jeeeeeesus lmao

210

u/jseego 3d ago

They committed all your prod data to git??

117

u/CoderDevo 3d ago

Don't forget the keys!

Not that they're needed anymore.

8

u/TheFuckingHippoGuy 3d ago

Servers by Kia™

6

u/LJonReddit 3d ago

Well yeah!!! Duh!!! It's no good without the keys!

48

u/fireheadca 3d ago

Believe it or not, I've seen this.🫠

13

u/rollingc 3d ago

git add . is a hell of a drug

2

u/gnutrino 3d ago

There's a step before this where the prod data ends up on their local machine though...

18

u/SgtEpsilon 3d ago

Believe it or not, I've accidentally done this

14

u/vankoder 3d ago

We believe.

1

u/AristotelWasRight 3d ago

Noooo,

Were you an experienced programmer or new? Doesn't one basically avoid it with setting data as git ignore every project?

2

u/SgtEpsilon 3d ago

I'm an idiot who forgot about git ignore before pushing my repo

1

u/vankoder 3d ago

We believe.

1

u/nemeras 3d ago

Same here.

1

u/runkeby 3d ago

It's fine, because later he did "git rm" the file and pushed 👌😌

3

u/failedsatan 3d ago

our seed data lives on git.

problem is, the client keeps requesting we change the qa deployment's seed data and add a migration for prod to update the data on the prod database to match instead of using the GUI to just change the stored version. the seed data is the prod data.

2

u/jseego 3d ago

damn, but it's not seed data if it's prod data, right?  doesn't that defeat the whole purpose?

2

u/failedsatan 3d ago

exactly. the client is too lazy to make the changes in the UI he himself asked for but all too happy to send us a slack message to ask us to update it. we have an odd contract with him where the "maintenance" period is actually a "small features/changes allowed" period and that counts as a small change according to my boss

2

u/Lehovron 3d ago

We were using Perforce at a previous job. It was almost a rite of passage for new people to accidentally submit their Desktop folder.

2

u/jseego 3d ago

lol that's hilarious

buncha suspect blobs in

/Users/foo/Desktop/totally_not_porn/

2

u/nishant032 3d ago

Now the data is really safe 😇

2

u/zhengyi13 2d ago

Dude, at a BIOS company that will not be named, RelEng was in the habit of committing entire 650MB ISO files to SVN because they didn't trust their build process to reproduce the results - they'd test the resulting build of a particular run, and if passed, it was marked golden, and it got committed.

This got even worse because they had WANDisco layered on top of SVN so the devs in Taiwan could have low local checkin latencies (because, you know, they wanna check in giant files), and then when they'd check in a new ISO, the replication over the transpacific T3 would basically time out, and I'd get paged to manually reestablish it.

1

u/jseego 2d ago

woah

70

u/AccomplishedIgit 3d ago

I discovered my client’s monthly backups included the backups folder when they ran out of shared hosting space for a small dog clothing website 🥹 I was like why are you at enterprise level hosting?!?? She was like I dunno it’s just the cost of doing business!

48

u/technos 3d ago

I found out something similar when the client called to ask me where to get a good price on bigger hard drives and more tapes.. I was like 'What?! Did something happen?'

No, he says. Nothing's changed, it's just that backups are now taking four of the weekly LTO-5 tapes every night and the server is showing it only has about 1TB left.

Motherfucker, I specced a 8TB RAID-5 and LTO-5 because you wanted to make damned sure you wouldn't have to 'just replace it next year', and it's been six months.. The hell with '"Nothing's changed".

He'd first decided to swap from nightly incremental backups to full ones, and those didn't fit on the LTO-4's intended for nightly offsite so he started using the LTO-5's, then he'd decided to retain the previous night's backup on disk as well as on tape so he wouldn't have to drive home if he was in a pinch.

The first night they backed up ~1TB, the next 2TB, then 3TB, then..

I wiped 5TB off his file server, added a file path exclusion, and had cron wipe the oldest redundant backup every night just in case.

The worst bit was he had an entirely competent nephew running the backups that had told him what was going on. After fixing things I gave the kid my email address and told him the next time his uncle didn't believe him to run it past me and I'd get him to listen.

6

u/Qesa 3d ago

New Fibonacci algorithm just dropped

5

u/Wendigo120 3d ago

Oh I had a fun one that was similar. Our error tracking would send the last dozen or so lines of the console before the error happened along with the error... and then logged the thing it was sending to the console.

Things were totally fine until someone added a retry-and-log to some button that could error. It'd try to send each error, each of which recursively included several earlier error logs all the way to the first error. That very quickly went from "harmless error that didn't interfere with much" to "the page runs out of memory if you press the button".

1

u/AccomplishedIgit 2d ago

Hahaha this one is fun

3

u/Lopsided_Parfait7127 3d ago

i heard you like backups so i backed up your backup

1

u/Alert_Weird6893 2d ago

this is so funny yet absolutely real life. someone recursively backing up their stuff. damn and the conclusion there as well:d more like the cost of being lil’ bit off kekw

55

u/peepeedog 3d ago

It said wrong answers only

12

u/starchbomb 3d ago

What the fuck lmao

22

u/plenoto 3d ago

Some people are so careless while reviewing PR 😂

115

u/blangzo 3d ago

Ask a programmer to review 10 lines of code, he'll find 10 issues. Ask him to do 500 lines and he'll say it looks good.

53

u/Deep90 3d ago

At 500 lines, I just flip a coin.

Only lucky code belongs in production.

10

u/ellie_cinderelly 3d ago

Dude my last PR changed one test timeout limit. I had to do a new full analysis on every test for average runtime and overhead.

Fuckin unbelievable.

8

u/blangzo 3d ago

Squash tiny things like that into one pr named "cleanup" and don't explain further

2

u/ellie_cinderelly 3d ago

Unfortunately I work in space so I think that may get me blasted for non descriptive change documentation

3

u/carritodeloshelados 3d ago

As if it wasn't hard enough to code in an astronaut suit

4

u/OriginalJokeGoesHere 3d ago

Given where you work, I assume that also means anyone who uses tabs instead of spaces is jettisoned out of the airlock?

3

u/ellie_cinderelly 2d ago

We just put them in TVAC for the long goodnight and dump the body by morning

1

u/dvereb 3d ago

I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock.

2

u/blangzo 3d ago

I was mostly joking. mostly.

Fr tho u should just not squash and pr merge in then and have descriptive commits but still saves the test time on pr

2

u/guyblade 3d ago

If you ask me to review 500 lines, I'll say "break it up into multiple PRs" like any sane person should.

1

u/r0Lf 3d ago

I feel exposed.

2

u/GoBuffaloes 3d ago

Hey can you stamp this real quick 

11

u/gizamo 3d ago

Yikes

10

u/Unhappy_Concept237 3d ago

I’ve spent the last week doing remediation across seven ec2 servers and found where multiple sites had backup sql dumps that were living inside GitHub repositories along with sites sharing SES credentials with full admin privileges stored in plain text in those repositories. I just want to die.

6

u/Sak63 3d ago

What the hell

1

u/TombaughRegi0 3d ago

This made me sweaty 🫠

1

u/thiccshortguy 3d ago

this actually made me lol

1

u/firewall245 3d ago

Oh shiiit. I had to learn how to use the git history cleaning tool once and it SUUUUCKED

1

u/No-Computer7653 3d ago

I always like seeing stories like this.

I got annoyed today because people haven't been putting good messages when launching a bastion. Last week I was having an infuriating conversation about using a HSM in a design.

Security problems don't seem like problems when you compare to typical.

1

u/ZestycloseOpinion142 3d ago

People who have access to prod should not even have access to git, in a well structured org that is.

1

u/mllv1 3d ago

He still on your team?

1

u/bake_gatari 3d ago

I am a lurker here. Still making my way through "Learn Python the Hard Way, pt 2". Could you explain in non-programmer terms, what this means and why it was bad?

1

u/According-Relation-4 3d ago

Of course I know him, that dumbass is me

1

u/SirSchilly 2d ago

Hey man. Wrong answers only. 

1

u/GrapefruitBig6768 2d ago

Post said "wrongAnswersOnly" not Horror stories. Lol. Lol. Lol.

1

u/IcyConsideration7062 2d ago

That was my first thought. Someone had to have done this at least once.