[Discourse.ros.org] [General] ROS Wiki: announcing extended CI badges in package headers

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[Discourse.ros.org] [General] ROS Wiki: announcing extended CI badges in package headers

Tully Foote via ros-users


Greetings ROS users,


With this post we'd like to update you on a change to the *Package Summary* section on the ROS wiki that was recently merged and has been rolled out for packages with a *devel job* in Melodic.

### Overview

As you may have noticed, for packages in Melodic the *Continuous Integration* badge does not only show *whether* the source repository has been registered in the `distribution.yaml`, but it will also show the CI status of the repository. The colour of the badge and icons shown on the badge reflect the status and show a summary of the results of tests run during the last build.

Clicking on the *Continuous Integration* badge shows a drop-down menu with up to five previous runs of the devel job. Clicking on any of these entries will take you to the page of the respective build for that particular repository.

![actionlib_enhanced_ok_dropdown|564x365](upload://gkBg0BlAYh21ko3fpRbtHf3hkWK.png)

The idea behind this change is that the CI badge will reflect the actual state of a package like this, and that in turn this will facilitate assessing package state by giving the Jenkins job build status and test results a more prominent place on the wiki.


### Badges

There are three different icon-colour combinations, representing three different categories of packages.

#### Build ok, tests ok

Repositories for which all tests are succeeding and the build ended successfully will get a green badge and a check-mark:

![ok_tooltip](upload://syVrn3KDdmKJvwxn1tPPWXhskZV.png)

All tests pass and Jenkins is happy about the build. So are we.

#### Build ok, tests skipped

Repositories with packages that have skipped tests will show a grey badge with a dash:

![skipped](upload://t6AvP8XDv6Uw4fJ1YeqwdySOzLM.png)

While skipped tests do not necessarily indicate something is wrong with a package, they are typically skipped for a reason (they are expected to fail otherwise, require some elaborate setup, are in the process of being updated, etc). Packages with grey badges are most likely perfectly fine to use, but cautious users might want to verify *why* the tests are skipped.

#### Build failed, tests failed

Repositories with failed tests will get a red badge and a cross:

![fail](upload://gYunNCvhc9u5QYNIUF5e1ffSgoP.png)

A badge like this is an indication that something is not right and one or more tests are failing. Inspecting the build history might be a good idea to try and understand what failed and why.


### Status and future work

The current implementation shows this information per repository, and is enabled or disabled per ROS release (and currently only enabled for Melodic). The former is both a limitation of the way test results are gathered by the buildfarm as well as a limitation of the current implementation. The latter was determined to be a good first step, with a potential extension to allow opt-outs added in a future PR.

### Comment & Questions

How and whether to take this effort further is one of the topics being discussed in the Quality Working Group meetings held each month, as part of the [ROSIN project](http://rosin-project.eu). Comments, criticism and suggestions are more than welcomed: please open a topic in the [Quality](https://discourse.ros.org/c/quality) Discourse group to discuss.


### Links

PR against `ros-infrastructure/roswiki`: [roswiki#246](https://github.com/ros-infrastructure/roswiki/pulls/246).
PR against `ros-infrastructure/ros_buildfarm`: [ros_buildfarm#541](https://github.com/ros-infrastructure/ros_buildfarm/pull/541).





---
[Visit Topic](https://discourse.ros.org/t/ros-wiki-announcing-extended-ci-badges-in-package-headers/4930/1) or reply to this email to respond.


If you do not want to receive messages from ros-users please use the unsubscribe link below. If you use the one above, you will stop all of ros-users from receiving updates.
______________________________________________________________________________
ros-users mailing list
[hidden email]
http://lists.ros.org/mailman/listinfo/ros-users
Unsubscribe: <http://lists.ros.org/mailman//options/ros-users>
Reply | Threaded
Open this post in threaded view
|

[Discourse.ros.org] [General] ROS Wiki: announcing extended CI badges in package headers

Tully Foote via ros-users


The PR link for the roswiki modifications https://github.com/ros-infrastructure/roswiki/pull/246





---
[Visit Topic](https://discourse.ros.org/t/ros-wiki-announcing-extended-ci-badges-in-package-headers/4930/2) or reply to this email to respond.


If you do not want to receive messages from ros-users please use the unsubscribe link below. If you use the one above, you will stop all of ros-users from receiving updates.
______________________________________________________________________________
ros-users mailing list
[hidden email]
http://lists.ros.org/mailman/listinfo/ros-users
Unsubscribe: <http://lists.ros.org/mailman//options/ros-users>