Skip to content

Add note about specifying stages when using include

Brie Carranza requested to merge bcarranza-stage-include-210781 into master

What does this MR do?

This MR updates the docs about stages to clarify what happens when you do specify stages.

We say:

If you do not define stages in the pipeline, you can use the 5 default stages

I'm adding a clarifying note about what happens if you do define stages. It may not be immediately clear to all that when you do define stages, the default stages are not added. This can lead to errors when you manually define stages and then include a template that attempts to add a job to a stage that you have not manually defined.

This is motivated by 🎫 a ticket that GitLab team members with access to ZenDesk can review for additional context.

We talk about manually adding a test stage when one is not declared in this troubleshooting section from the application security page but I think we should clarify it here where we talk about defining stages. People will encounter this outside of the CI templates we provide and so may not immediately draw a connection between what they are doing and the linked troubleshooting section.

Note: I am happy for any and all improvements to the wording I have proposed.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Brie Carranza

Merge request reports