Commit 9dc09bc1 authored by Guillaume Pasero's avatar Guillaume Pasero

Merge branch 'ci_contributing_update' into 'develop'

Ci contributing update

See merge request !538
parents 78db248d 7a9552bd
Pipeline #2072 passed with stages
in 6 minutes and 18 seconds
......@@ -58,9 +58,10 @@ then send a merge request.
Note that we also accept PRs on our [GitHub mirror](
which we will manually merge.
Feature branches are tested on multiple platforms on the OTB test infrastructure (a.k.a the [Dashboard]( They appear in the FeatureBranches section.
Feature branches are tested on multiple platforms on the OTB
[CI infrastructure](
Caveat: even if the Dashboard build on develop branch is broken, it is not
Caveat: even if the CI build on develop branch is broken, it is not
allowed to push fixes directly on develop. The developer trying to fix the
build should create a merge request and submit it for review. Direct push to
develop without review must be avoided.
......@@ -100,33 +101,36 @@ OTB team.
* Merge requests **must receive at least 2 positives votes from core developers** (members of Main Repositories group in Gitlab with at least "Developer" level; this includes PSC members) before being merged
* The merger is responsible for checking that the branch is up-to-date with develop
* Merge requests can be merged by anyone (not just PSC or RM) with push access to develop
* Merge requests can be merged once the dashboard is proven green for this branch.
This condition is mandatory unless reviewers and authors explicitely agree that
it can be skipped (for instance in case of documentation merges or compilation
fixes on develop). Branches of that sort can be identified with the ~patch label,
which tells the reviewer that the author would like to merge without dashboard testing.
* Merge requests can be merged once the CI pipeline passes successfully. See
next section for details on the CI pipelines.
Branches can be registered for dashboard testing by adding one line in `Config/feature_branches.txt` in [otb-devutils repository](
For branches in the main repository, the syntax is the following:
### Using the CI platform
branch_name [otb-data_branch_name]
The CI pipelines are triggered automatically when pushing commits. If you push
to a fork, you will need a few settings to trigger properly the CI pipelines:
The second branch name is optional. It can be set if you need to modify [otb-data]( according to your changes.
* You must add Runners for your fork: the best way is to ask access to the
runners from main repository when doing your first MR. During code review,
someone from CI admins will assign the runners to your fork.
* [Optional] You can create a
[personal access token](
(choose the scope API) and add it as a secret variable: on the project page of
your fork, go to Settings -> CI/CD -> Variables, add the variable `K8S_SECRET_API_TOKEN`
with your token as value (you should mask this variable for security reasons).
For branches in forks, the syntax is the following:
user/branch_name [user/otb-data_branch_name]
Again, the second branch name is optional.
When your pipeline ends, there are two cases:
For users without push access to [otb-devutils repository](, the modification can be asked through a merge requests to this repository.
* if all the jobs succeed, you see a green pipeline, which means no problem was
found on your commit.
* if one job fails, you see a red pipeline, which means something is broken in
your commit. The pipeline widget on Gitlab will tell you which job failed. You
will also find special jobs "cdash:..." in the stage `external` that provide
a link to the [Dashboard](
where you can look more in details into compilation errors and failed tests.
Once the feature branch is registered for testing, it should appear in the *FeatureBranches* section of the [OTB dashboard]( next day (remember tests are run on a nightly basis).
Do not forget to remove the feature branch for testing once it has been merged.
More details on the CI platform can be found
### Contribution license agreement
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment