同事未经同意修改我的线上代码,该怎么办 T_T - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
diggzhang
V2EX    职场话题

同事未经同意修改我的线上代码,该怎么办 T_T

  •  
  •   diggzhang 2016-10-16 07:51:06 +08:00 6256 次点击
    这是一个创建于 3293 天前的主题,其中的信息可能已经有所发展或是发生改变。

    同事是架构师,我们目的是做新旧系统的对比测试。 为了对比新旧消息处理是否异同,希望可以转发生产环境的流量做转发测试,侵入代码写了一个 proxy 。架构师大大在没有跟我打招呼的情况下,直接修改了线上代码在生产环境做测试。生产环境代码啊!两个人消息居然没有同步!觉得好不爽啊!测试期间出现一些 bug 导致线上环境不断重启。如果不是运维发现监控异常提醒,大家都没有发现。

    其实应该有 gor 或 duplicator 或 tcpcopy 可以做到不侵入转发。架构师大大之前也让我去做过非常深度的测试,我甚至给出了图表,还横向对比了几款常用的流量转发工具。觉得自己非常尴尬,自知工作一年能力欠佳,表达能力欠佳,多次想表达自己,都被截断。已经非常努力的去适应架构师大大的工作模式,可是还是做不到他满意。我很尊敬技术人,遇到大神眼里发光,可是同事之间为什么偏偏觉得多了一层“傲慢”。很迷茫,现在都害怕跟他交流,三句不到就换来责问的语句,迷之尴尬。

    35 条回复    2016-10-17 14:36:56 +08:00
    Hallujah
        1
    Hallujah  
       2016-10-16 08:10:57 +08:00 via Android   1
    为了解恨:你把他的代码也改了。
    为了面子:宽容地原谅他。
    为了以后的工作:狠狠地艹他。
    wsy2220
        2
    wsy2220  
       2016-10-16 08:11:29 +08:00 via Android
    谁改谁负责嘛
    bsidb
        3
    bsidb  
       2016-10-16 08:36:20 +08:00 via Android   1
    和非常忙的大大们交流,我自己的一点体会,也不知道适用不适用,权当一点经验交流吧~

    跟非常忙的 boss 交流,要先抛出他最关心的问题的结论出来,如果他感兴趣再解释细节,如果他不感兴趣,直接忽略就好了。

    虽然详细介绍一个事情的前因后果的过程是: 先 blablabla 介绍背景,再分析原因,最后得到结论。 但是 boss 一般都挺忙的,大多数时间只想听一个结论,不想听比较冗杂的分析过程。 所以和他们交流,先抛出你的结论,然后问他“您想听一下具体的原因么?” 如果他有兴趣,再简单介绍一下很概括的原因。如果他还有兴趣,再详细介绍一些细节的原因。如果他没兴趣,那就不用说了,他也不关心。

    另外有人是天生的责问体质,不光对你,对所有的下属都是责问的状态,自己不要往心里去就好了~
    shoaly
        4
    shoaly  
       2016-10-16 08:53:56 +08:00
    我建议还是要求他之后跟你多沟通, 或者让他写一个接口给你 你去调用..把他需要知道的信息都通过接口传进去, 至少这样才能让你自己的代码可控.
    随便被插入一行"其他人"的代码, 先不说尊重不尊重, 出了问题 debug 上都是一个麻烦. 莫非你要先全部下载线上版本, 然后跟你自己的做 diff 然后再来找原因? 如果都已经开始线上线下 diff 了, 不如邀请你们同事一起跟你在开发版本中开一个 branch. 你来负责合并
    binux
        5
    binux  
       2016-10-16 08:54:01 +08:00
    在线上做这种事情,责任比怎么做更重要,这种活,责任重产出还低,躲还躲不急呢。
    作为小兵,做了足够的调研线下测试,更多的时候不是说保证万无一失,而是表明一个态度,你看我做了那么充足的准备,出错了责任也没那么大。

    现在好了,有人愿意担这个责任,干这个活,那就让他干去呗。
    ebony0319
        6
    ebony0319  
       2016-10-16 08:57:51 +08:00 via Android
    喂,你的代码。
    不是公司的代码。
    ebony0319
        7
    ebony0319  
       2016-10-16 08:58:46 +08:00 via Android
    少了一个逗号。
    喂,你的代码。
    不,是公司的代码。
    laoyuan
        8
    laoyuan  
       2016-10-16 09:00:33 +08:00
    你也傲慢呗,来啊互相伤害
    songchao
        9
    songchao  
       2016-10-16 09:40:16 +08:00 via Android
    怎么没有 Git ?
    hasbug
        10
    hasbug  
       2016-10-16 09:51:02 +08:00
    1 楼已经说了
    tabris17
        11
    tabris17  
       2016-10-16 09:51:37 +08:00 via iPhone
    撕逼呗
    newworld
        12
    newworld  
       2016-10-16 10:13:28 +08:00
    不要怂 就是艹他 干得他不要不要的!
    开玩笑 O(∩_∩)O~
    沟通很重要 一定要沟通。。。
    kouryu
        13
    kouryu  
       2016-10-16 10:39:57 +08:00 via iPhone
    生产环境这种改动没有操作规范吗?
    报备一下,下次出问题就直接说好像代码被改了,给自己争取缓冲时间 23333333
    sutra
        14
    sutra  
       2016-10-16 10:40:02 +08:00
    没有经过 QA 部门签字的代码也能进入生产环境?药丸。
    ldbC5uTBj11yaeh5
        15
    ldbC5uTBj11yaeh5  
       2016-10-16 10:40:43 +08:00 via Android
    不要怂,干他啊。

    具体干的姿势温和一点就行了。
    ooh
        16
    ooh  
       2016-10-16 10:50:23 +08:00
    扔杯子
    geew
        17
    geew  
       2016-10-16 10:53:30 +08:00
    同意五楼

    反正有人愿意来背锅求之不得呢 爱改就改呗 谁改谁负责
    davidlau
        18
    davidlau  
       2016-10-16 11:04:39 +08:00
    code review 很重要
    xw
        19
    xw  
       2016-10-16 11:04:40 +08:00
    (吐槽下--)
    之前生产环境代码都是 sftp 手动传上去,然后上头领导(以前是开发,现在不写了)经常私自改动,而且也不把改动提交到 svn 或 git (只开发环境用了版本控制,生产环境一直是手动传上去),也不告知几个开发人员,经常出现莫名其妙情况,也是无语了、、、
    goodryb
        20
    goodryb  
       2016-10-16 11:09:18 +08:00
    这种情况难道不是运维去叼他么,线上的东西开发怎么会有权限动
    zonghua
        21
    zonghua  
       2016-10-16 11:10:55 +08:00
    DevOps
    youxiachai
        22
    youxiachai  
       2016-10-16 11:14:23 +08:00
    好同事啊......
    出了..问题..负责任的是他啊....
    k9982874
        23
    k9982874  
       2016-10-16 11:29:02 +08:00
    他是你的上司,他有这个权限,他改你的代码自然有他的道理。
    至于跟不跟你打招呼或者给不给你说明,看你在他眼中值不值的这么做。

    对于这个问题,你也不用很纠结,毕竟是工作,做好本职工作,这是本职工作外的事了,不要让这些事乱了心情。
    他改出问题他背锅就是。你的工资奖金不能少。
    forestyuan
        24
    forestyuan  
       2016-10-16 12:00:17 +08:00
    这个架构师很不专业
    bk201
        25
    bk201  
       2016-10-16 15:17:23 +08:00 via iPhone
    相处不长不要对别人指手画脚,好好沟通都是能沟通的,能坐上架构师自然不会太差.
    SmiteChow
        26
    SmiteChow  
       2016-10-16 16:02:08 +08:00
    沟通出现问题就改沟通方式方法,同时用自动化工具来保证以后不会出现这种 situation
    jhaohai
        27
    jhaohai  
       2016-10-16 18:29:16 +08:00 via iPhone
    谁改谁负责,出了问题让他背锅就行了
    millson
        28
    millson  
       2016-10-16 19:13:48 +08:00
    php ,曾经也有这样的问题,现在线上都是 ZendGuard 加密的代码 :lol
    neoblackcap
        29
    neoblackcap  
       2016-10-16 19:18:16 +08:00
    你们撕逼直接在生产环节撕逼也是厉害,我们撕逼最多也就是在 code review 的环节
    chunqiuyiyu
        30
    chunqiuyiyu  
       2016-10-16 20:20:17 +08:00
    谁修改,谁负责。
    iyaozhen
        31
    iyaozhen  
       2016-10-16 20:26:20 +08:00 via Android
    也是 6 啊,居然能直接修过线上代码。
    楼主不要激动,这是公司的代码,有责任心很好,但这个时候不要强出头, QA 找过来你就实话实说,你们架构师胆子真大,这情况 QA 还不肛死他。
    chilaoqi
        32
    chilaoqi  
       2016-10-16 21:12:45 +08:00
    有人想抗雷,直接禅让吧 :)
    Niphor
        33
    Niphor  
       2016-10-16 23:34:25 +08:00
    外包 /小公司吧... 居然有权限该线上代码

    谁改谁负责 很简单的
    没楼主什么事
    zjuster
        34
    zjuster  
       2016-10-17 10:13:04 +08:00
    1 、这是公司的代码。
    2 、谁改谁负责。

    你就看看是不是这样改比自己写的要有效率,可别再改回来哦。
    PEP4JASON
        35
    PEP4JASON  
       2016-10-17 14:36:56 +08:00
    能动手的尽量动手
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     1114 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 28ms UTC 23:17 PVG 07:17 JFK 19:17
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86