查看: 33792|回复: 70
收起左侧

[新组上路] 新人就搞出了超级大bug该如何是好

    |只看干货
匿名用户-EE2  | 添加认证 | 2022-10-5 10:46:28 来自APP |阅读模式
本楼: 👍   98% (84)
 
 
1% (1)   👎

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看附件。没有帐号?注册账号

x
金融中小厂,NG毕业一年就把支付系统搞崩了
,真资本主义杀手,写了一行noop ..
最坏的情况下公司要陪用户1Mˊ_>ˋ, 是不是该连夜买机票回国了。

写的时候的问题:.--
1. 这是个不知道ownership的job,隔壁组叫我们改,我对payment一无所知,只改了自己组相关的
2. 在写的过程中没有一个test env能用,local只能跑1/6
3. 一些秘制功能没人知道干啥,问了pm和未来ownership nbcs
4. 隔壁组本来要帮我deploy但当时我代码有问题,直接给我了deploy的权限,叫我自己qa+deploy

事后发现
1.  现有unittest蜜汁bug,啥都pass
2. 没有一个monitor这么大一个job. 1point3acres

我的问题:
1. 测的时候光是把env搞好就浑身解数,没测全
2. 没人回我我就不问了,想着赶快deploy了事. check 1point3acres for more.

我的担心:. 1point 3acres
1. 被开掉/pip 在完全没准备的情况下打包回家流落太平洋
2. 以后也没办法发展了-baidu 1point3acres
3. 该积极甩锅还是隔壁组建monitor

看了一些帖子,有些说ok还有救好好posymodern有些说gg。 老板没表态,说先把事情解决再说,明天1-1但也不知道说什么。 之前还在准备升职plan现在就因为一个没人接的side project打入地府我真的想放弃

评分

参与人数 11大米 +24 收起 理由
不成功便成仁 + 1 赞一个
fmusk + 1 赞一个
zazahan10 + 1 赞一个!
Sulerdey + 1 给你点个赞!
sidisun + 1 赞一个

查看全部评分


上一篇:今天meta 官宣了15% need support
下一篇:现在的各个大厂分分裁员加冻结招人
big_news 2022-10-5 11:15:20 | 显示全部楼层
本楼: 👍   100% (194)
 
 
0% (0)   👎
全局: 👍   100% (203)
 
 
0% (0)    👎
你只是引发这个事故的一个小小的trigger,问题在于整个业务流程有系统性风险。把这个业务流程中的每一个节点,都拉出来一个背锅的。有理有据有节地帮助公司解决问题。每个NG都是很可能犯错的,关键是这个系统为啥没有阻止这种错误的发生。

评分

参与人数 6大米 +10 收起 理由
ebca0000 + 1 给你点个赞!
希望能留美的猫 + 1 赞一个
fmusk + 1 赞一个
akdhfikbk + 1 赞一个
QueenieV + 5 很有用的信息!

查看全部评分

回复

使用道具 举报

地里匿名用户
匿名用户-D22  | 添加认证 | 2022-10-5 13:13:45
本楼: 👍   100% (125)
 
 
0% (0)   👎
给楼主的建议,赶紧写postmortem,把所有出问题的环节都列出来,给management生动展示一下什么叫十个避孕套没一个不带眼儿的,然后再把自己的责任轻描淡写地提一下。要么就都没事,要么就要死一起死,把这群不好好维护系统不认真review的人一起拉下水。
-baidu 1point3acres
我的第一份工作就和同事push了一个empty ML model进production,我大概要付一半的责任吧,半个小时就损失了几十万,老板一点都没有怪罪的意思,他的看法就是,一个empty model能通过这么多integration test,显然是系统的问题啊。

评分

参与人数 3大米 +4 收起 理由
ebca0000 + 1 给你点个赞!
samscartz + 1 给你点个赞!
duanmupeiyi + 2 很有用的信息!

查看全部评分

回复

使用道具 举报

clarity 2022-10-5 10:56:38 来自APP | 显示全部楼层
本楼: 👍   100% (88)
 
 
0% (0)   👎
全局: 👍   82% (1837)
 
 
17% (383)    👎
你当时应该多拉几个垫背的,不review不批准你就不deploy那种。你也不用管他们实际上有没有review,只要一个documented approval。也许别的组不想deploy背后是有原因的,只是不想告诉你。。。

评分

参与人数 1大米 +1 收起 理由
nanako + 1 赞一个

查看全部评分

回复

使用道具 举报

jagerSie 2022-10-5 11:00:55 来自APP | 显示全部楼层
本楼: 👍   100% (14)
 
 
0% (0)   👎
全局: 👍   98% (49)
 
 
2% (1)    👎
听起来不大合规     你写的代码必须让人approve才行
回复

使用道具 举报

地里匿名用户
匿名用户-EE2  | 添加认证 | 2022-10-5 11:02:16 来自APP
本楼: 👍   100% (7)
 
 
0% (0)   👎
隔壁组是review了,但完全没看的那种ˊ_>ˋ 我没给自己的组看,因为这是完全隔壁组的项目,wich我的错
回复

使用道具 举报

地里匿名用户
匿名用户-EE2  | 添加认证 | 2022-10-5 11:04:03 来自APP
本楼: 👍   0% (0)
 
 
0% (0)   👎
本帖最后由 匿名 于 2022-10-4 23:08 编辑
.1point3acres
隔壁组approve的。但还是我写的,并且没有test全… 只跑了unittest 和基本检查能不能在server上run,而且model setup之前的人写的还有问题我没意识到
回复

使用道具 举报

浪漫歪歪 2022-10-5 11:11:27 | 显示全部楼层
本楼: 👍   100% (4)
 
 
0% (0)   👎
全局: 👍   95% (1678)
 
 
4% (72)    👎
基础设施的问题,赶紧甩锅
回复

使用道具 举报

地里匿名用户
匿名用户-EE2  | 添加认证 | 2022-10-5 11:16:56 来自APP
本楼: 👍   100% (1)
 
 
0% (0)   👎
哎,但是还是我写的,就甩干净好难啊,. 1point3acres
主要问题太简单了… 肉眼可见的那种typo…
我太焦虑了… 哭了
回复

使用道具 举报

doudouzhuang 2022-10-5 11:32:43 | 显示全部楼层
本楼: 👍   0% (0)
 
 
0% (0)   👎
全局: 👍   100% (3)
 
 
0% (0)    👎
匿名用户 发表于 2022-10-4 22:16
哎,但是还是我写的,就甩干净好难啊,
主要问题太简单了… 肉眼可见的那种typo… ..
我太焦虑了… 哭了

你的意思是你自己的code有typo, 如果这个typo当时眼睛灵一点,发现了,就不会有后续的问题?
回复

使用道具 举报

地里匿名用户
匿名用户-EE2  | 添加认证 | 2022-10-5 11:40:27
本楼: 👍   0% (0)
 
 
0% (0)   👎
doudouzhuang 发表于 2022-10-4 23:32. Waral dи,
你的意思是你自己的code有typo, 如果这个typo当时眼睛灵一点,发现了,就不会有后续的问题?

基本上就是以为调用完原本会变忘赋了,我记错了,不过后来发现codebase还有别人犯一样的错误,也可能我当时就是直接参考的😂
当时谁也没发现,测这个的test的setup正好有问题,诸如此类的叠加。
但我真的平时写自己组的代码写test都尽量写全,结果翻车在这种事情如此猛烈
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
职场达人
  • ↑ 本版用于讨论职场各种干货话题,闲聊请去🔗聊聊或者🔗匿名版
  • ❌ 本版严禁水贴,引战,发布广告,拉群,贴个人联系方式,扣分无警告
  • ☑ 求职、面经等去 🔗北美求职和 🔗回国求职大区,刷题和学习请去 🔗终身学习大区
  • ☑ 请去专版发布 🔗内推, 🔗招聘信息,和讨论 🔗创业内容
  • ☑ PIP / DevList/ Need Support 等话题也已开设 🔗专版

本版积分规则

>
快速回复 返回顶部 返回列表