lunes, febrero 25, 2013

Code Quality stage using Jenkins


In Continuous Delivery each build is potentially shippable. This fact implies among a lot of other things, to assign a none snapshot version to your components as fast as possible so you can refer them through all the process.

Usually automated software delivery process consist of several stages like Commit stage, Code Quality, Acceptance Tests, Manual Test, Deployment, ... But let's focusing on second stage related to code quality. Note that in my previous post (http://www.lordofthejars.com/2013/02/conditional-buildstep-jenkins-plugin.html) there are some concepts that are being used here.

Second stage in continuous delivery is the code quality. This step is very important because is where we are running static code analysis for detecting possible defects (mostly possible NPE), code conventions or unnecessary object creation. Some of projects that are typically used are Checkstyle, PMD or FindBugs among others. In this case we are going to see how to use Checkstyle, but of course it is very similar in any other tool.

So the first thing to do is configure Checkstyle into our build tool (in this case Maven). Because we only want to run the static analysis in second stage of our pipeline we are going to register the Checkstyle Maven plugin into a metrics profile. Keep in mind that all plugins run for code analysis should be added into that profile.


Now that we have our pom configured with Checkstyle, we can configure Jenkins to run Code Quality stage after the first stage (explained in my previous post).

In this case we are going to use Trigger Parameterized Build plugin to execute code quality job from commit stage.

Because code of current build version has been pushed into a release branch (see my previous post) during commit stage, we need to set branch name as parameter for the code quality Jenkins job, so code can be downloaded and then run the static analysis.

In build job of our first stage, we add a Post-build Action of type Trigger parameterized build on other projects. First we open the Configure menu of first build job of pipeline and we configure it so next build job of the pipeline (helloworld-code-quality) is executed only if current job is stable. Also we define the RELEASE_BRANCH_NAME parameter with branch name.



Then let's create a new build job that will be in charge of running static code analysis, we are going to name it helloworld-code-quality.

And we configure the new build job. First of all check the option "This build is parameterized", and add a String parameter and set the name RELEASE_BRANCH_NAME. After that we can use RELEASE_BRANCH_NAME parameter in current job. So at Source Code Management section we add the repository URL and in Branches to build we set origin/${RELEASE_BRANCH_NAME}.

Then at Build section we add a Maven build step, which executes Checkstyle goal: checkstyle:checkstyle -P metrics.

And finally to have a better visibility of the result, we can install Checkstyle Jenkins plugin and publish the report. After plugin is installed, we can add a new Post-build Actions with name "Publish Checkstyle analysis result". In our case report is located at **/target/checkstyle-result.xml.



And that's all for current stage, next stage is the responsible of executing the acceptance tests, but this would be in another post.

So in summary we have learned how after code is compiled and some tests are executed (in first stage of pipeline), the Code Quality stage is run into Jenkins using Checkstyle Maven plugin.

We keep learning,
Alex
En algun lugar de un gran pais, Olvidaron construir, Un hogar donde no queme el sol, Y al nacer no haya que morir… (En Algún Lugar - Dunncan Dhu)
Music: http://www.youtube.com/watch?v=Myn7ghLQltI

5 comentarios:

Unknown dijo...

Thank you very much for your post. Very interesting.

I think add these tools to our software development ecosystems help to build better products and become more aware about the "quality" early. However, we must not forget that these tools do not validate that the design or our architecture is adequate. Just help us study the last part of our process, the coding.

---------------------------

Muchas gracias por tu post, muy interesante. Creo que agregar este tipo de herramientas a nuestros ecosistemas de desarrollo de software ayudan a obtener mejores resultados y a tomar mas conciencia sobre la "calidad" en etapas tempranas. No obstante, no debemos olvidar que este tipo de herramientas no validan que nuestro diseño o nuestra arquitectura sea adecuada tecnica y economicamente. Tan solo nos ayudan a estudiar la ultima parte de nuestro proceso, la codificacion.

Antonio Jose Molina Gutierrez

Jarosław Bajak dijo...

If you can use external tool think about Sonar - many analysis tool combined into one.

Unknown dijo...

yes totally I agree that with Sonar you get a full report, but for this academic example I wanted to keep things simple. ;) Thanks for reading my blog both of you.

Unknown dijo...

hi,

I have configured the checkstyle in jenkins. I have maven3 project. everything works find when i execute project from command line.
if I run mvn checkstyle:checkstyle it generates the xml file in target directory

if i run mvn site it generates HTML report.

however when i run checkstyle:checksstyle from Jenkins it fails to show reports.

Console output says following, which mean plugin is working, however all the reports are coming empty
[INFO] There are 13 checkstyle errors.

Unknown dijo...


Magnificent items from you, man. I have bear in mind your stuff prior to and you're just too great. I actually like what you've got right here, really like what you're stating and the way in which you assert it. You make it entertaining and you still take care of to stay it sensible. I cant wait to learn much more from you. That is actually a terrific web site. outlook 365 sign in