• MajorHavoc@lemmy.world
    link
    fedilink
    arrow-up
    4
    ·
    1 year ago

    It’s not about how hard the problem is to reverse, it’s about respecting the team enough not to call them on Saturday.

    • rglullis@communick.newsOP
      link
      fedilink
      English
      arrow-up
      1
      arrow-down
      4
      ·
      edit-2
      1 year ago

      Again: if the changes are small enough and you have automated checks in place, they should not require manual intervention.

      Plus, what happens if a deploy on Thursday has a bug which only is manifested on a Saturday?

      • MajorHavoc@lemmy.world
        link
        fedilink
        arrow-up
        7
        ·
        1 year ago

        Again: if the changes are small enough and you have automated checks in place, they should not require manual intervention.

        You’ve used the magic word “should”. “Should is famous last words.” The trick to keeping developer talent is not to risk the developer’s weekend plans on “should”.

        And yes, maybe I’m only risking our cloud ops person’s weekend plans. Same principle applies.

        Every change that isn’t already an active disaster recovery can wait for Monday.

        • rglullis@communick.newsOP
          link
          fedilink
          English
          arrow-up
          1
          arrow-down
          3
          ·
          1 year ago

          Every change that isn’t already an active disaster recovery can wait for Monday.

          I honestly fail to see the difference between “don’t deploy on Friday if this can wait until Monday” and “don’t deploy on the evening if it can wait until the next morning”.

          The idea of CD is that changes are small and cheap. No one is saying “it’s okay to push huge PRs with huge database migrations on a Friday”, what is being said is “if your team is used to ship frequently and incrementally, it won’t matter when you ship and your risk will always be small.”

          • MajorHavoc@lemmy.world
            link
            fedilink
            arrow-up
            4
            ·
            1 year ago

            I honestly fail to see the difference between “don’t deploy on Friday if this can wait until Monday” and “don’t deploy on the evening if it can wait until the next morning”.

            Both are top tier practices.

            If your team is used to ship frequently and incrementally, it won’t matter when you ship and your risk will always be small."

            Yep. That’s all great advice.

            But I’m just a veteran saying that all the preparation in the world doesn’t compare with simply not inviting trouble right before the evening or the weekend.

            Organizations that feel that they desperately need to take that risk, are doing it because they disrespect their team’s time.

            It can be the smallest risk in the world, but it’s still a risk, and it’s a completely unnecessary one (outside of an active in progress disaster recovery).

            • rglullis@communick.newsOP
              link
              fedilink
              English
              arrow-up
              1
              arrow-down
              2
              ·
              1 year ago

              Organizations that feel that they desperately need to take that risk, are doing it because they disrespect their team’s time.

              Or they are aware they are not in a position to block deployments for 60 hours every week? I’ve felt more discouraged working at companies that blocked Friday deployments because “it could wait until Monday”, and then when Monday came half of the team was blocked or waiting for some new data report that could have been running during the weekend.

              It can be the smallest risk in the world, but it’s still a risk.

              And it’s up to the Engineering manager (or at least the Release Manager in places where that role still exists) to evaluate what would be the trade-off. If you say that a bug coming from a Thursday deployment could’ve waited until Monday, why can’t a bug that has come from a Friday deployment?

              I guess my issue is not in saying “Some things should not be deployed on a Friday”, but with the generalization. Of course there are things that should be okay to deploy on a Friday, or a Thursday night, or when the manager is on vacation… Being strict about it seems anything but “respect for the team”, but a general distrust of the people and the process.

      • MajorHavoc@lemmy.world
        link
        fedilink
        arrow-up
        5
        ·
        1 year ago

        Good question.

        Since we’re doing a deep dive, I’ll share some additonal context. I’m the manager of the developers. On my team, that means the call comes to me first.

        I have had Thursday deploys that resulted in bugs discovered on Saturday. Here’s how the conversation on Saturday went:

        “Thanks for letting me know. So we didn’t notice this on Friday?”

        “No, it’s subtle.” Or “We noticed, but didn’t get around to letting you know until now.”

        “Okay. I’ll let the team know to plan to rollback at 0900 on Monday, then we will start fixing any damage that happened on Friday, and the weekend.”