amcneil36.github.io

Local and Ci-Server Build Recommendations

The build must be able to be started from the command line

We want our build to be able to be started from the command line.1 This will help make the build able to be ran from any developer’s machine as well as on a ci-server.

Commit build recommendations

10-minute build

We want our build that is ran on commit to be able to be ran in 10 or fewer minutes.1,2 Having a build that takes longer than this makes it take a while to fix errors in the build since there has will be a long wait time to make sure that the build passes. People will also be less inclined to merge to trunk as frequently if the build has a long wait time.

Static analyzers to run

Unit tests

All unit tests should run on the commit build. All unit tests should run in under 1/10 of a second.3 Therefore the unit tests should add very little time to the build.

Acceptance tests

We would ideally run all acceptance tests on the commit build. However, we did say that we want a 10-minute build and in some cases, running all of the acceptance tests would take longer than 10 minutes. If our acceptance tests are causing our build to take longer than 10 minutes, then we should look into doing the following:

If the acceptance tests are still causing the build to take longer than 10 minutes after doing this, then we want to take the least important acceptance tests and have them run post-commit.1 The most important acceptance tests and the ones that are most likely to fail should be the ones ran on commit in this case. This would make the commit stage fast while having the post-commit build still be likely to pass. In either case, we should make sure that all acceptance tests pass prior to releasing.

Place all tests in the same repository as the code they test

Sometimes people will try to place acceptance tests in a different repository than the code they test. This makes it harder to set up a build that runs all the necessary tests. It also makes it possible for the production repository to be out of date with the corresponding acceptance testing repository. Having tests in the same repository as the code they test will make the build easier to set up and will make it less likely that people merge with tests that are failing elsewhere.

Ability to choose the environment to deploy to on the command line

We should be able to deploy to any environment we want by just running one command on the command line.1 This could be as simple as just passing in the environment name as a parameter to the command that is ran on the command line.

Containerization

Our build should be able to create a new environment, run all of the tests, and then destroy the environment.5 This is best done with containerization.5 The purpose of creating the new environment is so that our environment is always in a consistent state when running tests.

Smoke test deployments

When deploying to any environment, we should have smoke tests that run to make sure that the environment is configured properly.1 If any external service we depend on is not working or not configured properly, the corresponding smoke test should fail.

Nightly build

Nightly builds are builds that run once per day, usually around 3 am when no one is awake. These are most commonly done with tests that might use expensive resources or take a long time. The downside of nightly builds is that if something goes wrong, you don’t find out until the next day. It would be convenient if everything ran on commit so that you know sooner if something went wrong. However, some things are too expensive and just can’t be ran on commit.

Tests like load tests, stress tests, and longevity tests are all good candidates to have ran nightly on a ci-server.1 These tests would also ideally be ran in parallel.

Use the application’s API to put the environment in the correct state for tests

What some people will do to put the environment in the correct state needed for their test is to write their own code to set up the environment. The problem with this is that when the environment changes in production, we would have to update our test code to set up the environment properly for the new changes. We should use the application’s API to set up the state for our tests so that when changes to the environment are made in production, our tests will automatically set up the environment the new way.1 This will ensure that we are testing our application against the correct environment.

Favor push hooks over polling

Some people will set up their ci-server to poll their repository every so often for changes. This is not only expensive, but it also causes the ci-server to not necessarily build the project right away after receiving a commit. We should instead use a push hook to where the ci-server is notified by our version control system as soon as we push. This will make our project built as soon as a commit is pushed and will use up much less resources.

Pre-tested commit vs post-tested commit on ci-server

When developing on a branch, I like having the ci-server build my branch post commit. This means that my commit is made to my remote branch and then the ci-server builds. This does allow for my build to go red but I don’t find that as big of a deal on a development branch in comparison to trunk. I sometimes just want to be able to save my work remotely or be able to show a teammate my work remotely and I don’t want that attempt at committing to be rejected on my remote development branch due to a red build.

When it comes to merging code into trunk, however, I prefer the ci-server to build pre-commit. What I mean by this is that the ci-server will locally merge the development branch into trunk and build that. If the build succeeds, the ci-server will notify you that it succeeded. The ci-server could either automatically commit this merge to trunk upon a successful build or have you press a button or type a command to indicate that you want to complete the merge. This helps keep the trunk build passing.

Set up a timebox for fixing broken trunk builds before reverting

If a check-in to trunk breaks the build, everyone needs to stop what they are doing and try to fix the build. If no one can fix the build within ten minutes, revert the commit that broke the build.1 This helps keep the build passing and prevents people from getting blocked on merging.

Sources

  1. Humble, Jez and Farley, David. Continuous Delivery: Reliable Software Releases Through Build, Test And Deployment Automation. Addison-Wesley, 2010.
  2. Beck, Kent and Andres, Cynthia. Extreme Programming Explained. Addison-Wesley, 2004.
  3. Feathers, Michael. Working Effectively with Legacy Code. Pearson Education, 2005.
  4. Wolff, Eberhard. A Practical Guide to Continuous Delivery. Addison-Wesley, 2017.
  5. Cohn, Mike. Succeeding With Agile: Software Development Using Scrum. Addison-Wesley, 2013.