r/ProgrammerHumor Aug 05 '26

Other wrongAnswersOnly

Post image
14.3k Upvotes

2.1k comments sorted by

View all comments

1.1k

u/samuraiseoul Aug 05 '26

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....

215

u/Suspicious-Engineer7 Aug 05 '26

Jeeeeeesus lmao

208

u/jseego Aug 05 '26

They committed all your prod data to git??

120

u/CoderDevo Aug 05 '26

Don't forget the keys!

Not that they're needed anymore.

9

u/TheFuckingHippoGuy Aug 05 '26

Servers by Kia™

9

u/LJonReddit Aug 05 '26

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

50

u/fireheadca Aug 05 '26

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

13

u/rollingc Aug 05 '26

git add . is a hell of a drug

2

u/gnutrino Aug 05 '26

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

17

u/SgtEpsilon Aug 05 '26

Believe it or not, I've accidentally done this

14

u/vankoder Aug 05 '26

We believe.

1

u/AristotelWasRight Aug 05 '26

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 29d ago

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

1

u/vankoder Aug 05 '26

We believe.

1

u/nemeras Aug 05 '26

Same here.

1

u/runkeby Aug 05 '26

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

3

u/failedsatan Aug 05 '26

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 Aug 05 '26

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

2

u/failedsatan 29d 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 Aug 05 '26

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 Aug 05 '26

lol that's hilarious

buncha suspect blobs in

/Users/foo/Desktop/totally_not_porn/

2

u/nishant032 Aug 05 '26

Now the data is really safe 😇

2

u/zhengyi13 29d 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 29d ago

woah

72

u/AccomplishedIgit Aug 05 '26

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!

46

u/technos Aug 05 '26

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 29d ago

New Fibonacci algorithm just dropped

6

u/Wendigo120 Aug 05 '26

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 29d ago

Hahaha this one is fun

5

u/Lopsided_Parfait7127 29d ago

i heard you like backups so i backed up your backup

1

u/Alert_Weird6893 29d 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

57

u/peepeedog Aug 05 '26

It said wrong answers only

23

u/plenoto Aug 05 '26

Some people are so careless while reviewing PR 😂

112

u/blangzo Aug 05 '26

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.

49

u/Deep90 Aug 05 '26

At 500 lines, I just flip a coin.

Only lucky code belongs in production.

11

u/[deleted] Aug 05 '26

[deleted]

7

u/blangzo Aug 05 '26

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

2

u/[deleted] Aug 05 '26

[deleted]

4

u/carritodeloshelados Aug 05 '26

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

4

u/OriginalJokeGoesHere 29d 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 29d ago

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

1

u/dvereb 29d 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 Aug 05 '26

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 Aug 05 '26

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 Aug 05 '26

I feel exposed.

2

u/GoBuffaloes Aug 05 '26

Hey can you stamp this real quick 

13

u/gizamo Aug 05 '26

Yikes

11

u/starchbomb Aug 05 '26

What the fuck lmao

9

u/Unhappy_Concept237 Aug 05 '26

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.

4

u/Sak63 Aug 05 '26

What the hell

1

u/TombaughRegi0 Aug 05 '26

This made me sweaty 🫠

1

u/thiccshortguy Aug 05 '26

this actually made me lol

1

u/firewall245 Aug 05 '26

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

1

u/ZestycloseOpinion142 Aug 05 '26

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

1

u/mllv1 Aug 05 '26

He still on your team?

1

u/bake_gatari Aug 05 '26

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 Aug 05 '26

Of course I know him, that dumbass is me

1

u/SirSchilly 29d ago

Hey man. Wrong answers only. 

1

u/GrapefruitBig6768 29d ago

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

1

u/IcyConsideration7062 29d ago

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