Skip to content

Populate CI_MERGE_REQUEST_CHANGED_PAGE_* variables

drew stachon requested to merge changed-pages-ci-var into master

What does this MR do?

Adds two predefined CI variables, CI_MERGE_REQUEST_CHANGED_PAGE_PATHS and CI_MERGE_REQUEST_CHANGED_PAGE_URLS. They are only populated on build with deployed environments for merge request Pipelines.

Why?

When a Review App is deployed, the configuration writer can automatically feed the pages known to be different post-merge in a Browser Performance or Accessibility test.

What does this MR not do?

  • Implement any testing features. This solely populated the variables for use in arbitrary configuration.
  • Populate changes for Review Apps outside of a merge request context. Because the merge request diff is necessary to determine what has changed, these variables are limited to that scope.

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 Yorick Peterse

Merge request reports