V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
diggzhang
V2EX  ›  职场话题

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

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

    同事是架构师,我们目的是做新旧系统的对比测试。 为了对比新旧消息处理是否异同,希望可以转发生产环境的流量做转发测试,侵入代码写了一个 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   ·   实用小工具   ·   1063 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 25ms · UTC 19:32 · PVG 03:32 · LAX 11:32 · JFK 14:32
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.