ci: switch rules:changes to be using paths: subkeyword

This is (should) be functionally identical to the existing rules.

When not specified, items added to the changes: are automatically passed
to changes:paths [1], however we are going to introduce another keyword
(changes:compare_to) in the followup commit. Thus switch already from changes: to
changes:paths so it is easier to test the changes standalone.

[1]: https://docs.gitlab.com/ee/ci/yaml/index.html#ruleschanges

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/7507>
This commit is contained in:
Jordan Petridis 2024-09-10 00:02:47 +03:00 committed by GStreamer Marge Bot
parent d7e8f0e1ca
commit d8421a1f42

View file

@ -324,9 +324,10 @@ commitlint:
.gtk-build-rules: .gtk-build-rules:
rules: rules:
- changes: - changes:
- .gitlab-ci.yml paths:
- meson.build - ".gitlab-ci.yml"
- subprojects/*.wrap - "meson.build"
- "subprojects/*.wrap"
variables: variables:
GTK_ARGS: "-Dgtk=enabled" GTK_ARGS: "-Dgtk=enabled"
@ -359,9 +360,11 @@ commitlint:
rules: rules:
- !reference [.upstream-branch-rules, rules] - !reference [.upstream-branch-rules, rules]
- changes: - changes:
*modules_changes paths:
*modules_changes
- changes: - changes:
- subprojects/gstreamer-vaapi/**/* paths:
- "subprojects/gstreamer-vaapi/**/*"
.build simple: .build simple:
extends: extends:
@ -479,9 +482,11 @@ build debian x86_64:
- !reference [.gtk-build-rules, rules] - !reference [.gtk-build-rules, rules]
- !reference [.upstream-branch-rules, rules] - !reference [.upstream-branch-rules, rules]
- changes: - changes:
*modules_changes paths:
*modules_changes
- changes: - changes:
- subprojects/win-*/* paths:
- "subprojects/win-*/*"
script: script:
# Make sure powershell exits on errors # Make sure powershell exits on errors
# https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_preference_variables?view=powershell-6 # https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_preference_variables?view=powershell-6
@ -600,9 +605,11 @@ build macos:
- !reference [.gtk-build-rules, rules] - !reference [.gtk-build-rules, rules]
- !reference [.upstream-branch-rules, rules] - !reference [.upstream-branch-rules, rules]
- changes: - changes:
*modules_changes paths:
*modules_changes
- changes: - changes:
- subprojects/macos-*/* paths:
- "subprojects/macos-*/*"
parallel: parallel:
matrix: matrix:
- BUILD_TYPE: ["--default-library=both"] - BUILD_TYPE: ["--default-library=both"]
@ -647,7 +654,8 @@ build macos:
PYTHONFAULTHANDLER: "enabled" PYTHONFAULTHANDLER: "enabled"
rules: rules:
- changes: - changes:
*modules_changes paths:
*modules_changes
script: script:
# Disable gdb debugdir safeload # Disable gdb debugdir safeload
# The CI will execute any code submitted to it anyway # The CI will execute any code submitted to it anyway
@ -684,9 +692,11 @@ check fedora:
TEST_SUITE: "check.gst*" TEST_SUITE: "check.gst*"
rules: rules:
- changes: - changes:
*modules_changes paths:
*modules_changes
- changes: - changes:
- subprojects/gstreamer-vaapi/**/* paths:
- "subprojects/gstreamer-vaapi/**/*"
integration testsuites fedora: integration testsuites fedora:
extends: '.test fedora x86_64' extends: '.test fedora x86_64'
@ -734,8 +744,9 @@ check video formats:
- $CI_PROJECT_DIR/ci/scripts/check-video-formats.sh - $CI_PROJECT_DIR/ci/scripts/check-video-formats.sh
rules: rules:
- changes: - changes:
- ${VIDEO_HEADER} paths:
- ${WL_HEADER} - "${VIDEO_HEADER}"
- "${WL_HEADER}"
playbin3 integration: playbin3 integration:
extends: 'integration testsuites fedora' extends: 'integration testsuites fedora'
@ -826,15 +837,16 @@ fluster v4l2-stateless on visl:
junit: build/fluster-results-*.xml junit: build/fluster-results-*.xml
rules: rules:
- changes: - changes:
- .gitlab-ci.yml paths:
- .gitlab-image-tags.yml - ".gitlab-ci.yml"
- subprojects/gst-plugins-bad/sys/v4l2codecs/* - ".gitlab-image-tags.yml"
- subprojects/gst-plugins-bad/gst-libs/gst/codecparsers/* - "subprojects/gst-plugins-bad/sys/v4l2codecs/*"
- subprojects/gst-plugins-bad/gst-libs/gst/codecs/* - "subprojects/gst-plugins-bad/gst-libs/gst/codecparsers/*"
- subprojects/gst-plugins-bad/gst/videoparsers/* - "subprojects/gst-plugins-bad/gst-libs/gst/codecs/*"
- tests/virtme/* - "subprojects/gst-plugins-bad/gst/videoparsers/*"
- ci/scripts/build-linux.sh - "tests/virtme/*"
- ci/docker/fedora/* - "ci/scripts/build-linux.sh"
- "ci/docker/fedora/*"
# Valgrind # Valgrind
.valgrind fedora x86_64: .valgrind fedora x86_64:
@ -844,9 +856,11 @@ fluster v4l2-stateless on visl:
EXTRA_VALIDATE_ARGS: "--valgrind" EXTRA_VALIDATE_ARGS: "--valgrind"
rules: rules:
- changes: - changes:
*modules_changes paths:
*modules_changes
- changes: - changes:
- subprojects/gstreamer-vaapi/**/* paths:
- "subprojects/gstreamer-vaapi/**/*"
valgrind core: valgrind core:
extends: '.valgrind fedora x86_64' extends: '.valgrind fedora x86_64'
@ -854,11 +868,12 @@ valgrind core:
TEST_SUITE: "check.gstreamer\\..*" TEST_SUITE: "check.gstreamer\\..*"
rules: rules:
- changes: - changes:
- "*" paths:
- scripts/* - "*"
- ci/**/* - "scripts/*"
- subprojects/gst-devtools/**/* - "ci/**/*"
- subprojects/gstreamer/**/* - "subprojects/gst-devtools/**/*"
- "subprojects/gstreamer/**/*"
valgrind base: valgrind base:
extends: '.valgrind fedora x86_64' extends: '.valgrind fedora x86_64'
@ -866,12 +881,13 @@ valgrind base:
TEST_SUITE: "check.gst-plugins-base\\..*" TEST_SUITE: "check.gst-plugins-base\\..*"
rules: rules:
- changes: - changes:
- "*" paths:
- scripts/* - "*"
- ci//**/* - "scripts/*"
- subprojects/gst-devtools/**/* - "ci/**/*"
- subprojects/gstreamer/**/* - "subprojects/gst-devtools/**/*"
- subprojects/gst-plugins-base/**/* - "subprojects/gstreamer/**/*"
- "subprojects/gst-plugins-base/**/*"
valgrind good: valgrind good:
extends: '.valgrind fedora x86_64' extends: '.valgrind fedora x86_64'
@ -881,13 +897,14 @@ valgrind good:
TIMEOUT_FACTOR: "4" TIMEOUT_FACTOR: "4"
rules: rules:
- changes: - changes:
- "*" paths:
- scripts/* - "*"
- ci/**/* - "scripts/*"
- subprojects/gst-devtools/**/* - "ci/**/*"
- subprojects/gstreamer/**/* - "subprojects/gst-devtools/**/*"
- subprojects/gst-plugins-base/**/* - "subprojects/gstreamer/**/*"
- subprojects/gst-plugins-good/**/* - "subprojects/gst-plugins-base/**/*"
- "subprojects/gst-plugins-good/**/*"
valgrind ugly: valgrind ugly:
extends: '.valgrind fedora x86_64' extends: '.valgrind fedora x86_64'
@ -895,14 +912,15 @@ valgrind ugly:
TEST_SUITE: "check.gst-plugins-ugly\\..*" TEST_SUITE: "check.gst-plugins-ugly\\..*"
rules: rules:
- changes: - changes:
- "*" paths:
- scripts/* - "*"
- ci/**/* - "scripts/*"
- subprojects/gst-devtools/**/* - "ci/**/*"
- subprojects/gstreamer/**/* - "subprojects/gst-devtools/**/*"
- subprojects/gst-plugins-base/**/* - "subprojects/gstreamer/**/*"
- subprojects/gst-plugins-good/**/* - "subprojects/gst-plugins-base/**/*"
- subprojects/gst-plugins-ugly/**/* - "subprojects/gst-plugins-good/**/*"
- "subprojects/gst-plugins-ugly/**/*"
valgrind bad: valgrind bad:
extends: '.valgrind fedora x86_64' extends: '.valgrind fedora x86_64'
@ -910,14 +928,15 @@ valgrind bad:
TEST_SUITE: "check.gst-plugins-bad\\..*" TEST_SUITE: "check.gst-plugins-bad\\..*"
rules: rules:
- changes: - changes:
- "*" paths:
- scripts/* - "*"
- ci/**/* - "scripts/*"
- subprojects/gst-devtools/**/* - "ci/**/*"
- subprojects/gstreamer/**/* - "subprojects/gst-devtools/**/*"
- subprojects/gst-plugins-base/**/* - "subprojects/gstreamer/**/*"
- subprojects/gst-plugins-good/**/* - "subprojects/gst-plugins-base/**/*"
- subprojects/gst-plugins-bad/**/* - "subprojects/gst-plugins-good/**/*"
- "subprojects/gst-plugins-bad/**/*"
valgrind ges: valgrind ges:
extends: '.valgrind fedora x86_64' extends: '.valgrind fedora x86_64'
@ -925,16 +944,17 @@ valgrind ges:
TEST_SUITE: "check.gst-editing-services\\..*" TEST_SUITE: "check.gst-editing-services\\..*"
rules: rules:
- changes: - changes:
- "*" paths:
- scripts/* - "*"
- ci/**/* - "scripts/*"
- subprojects/gst-devtools/**/* - "ci/**/*"
- subprojects/gstreamer/**/* - "subprojects/gst-devtools/**/*"
- subprojects/gst-plugins-base/**/* - "subprojects/gstreamer/**/*"
- subprojects/gst-plugins-good/**/* - "subprojects/gst-plugins-base/**/*"
- subprojects/gst-plugins-bad/**/* - "subprojects/gst-plugins-good/**/*"
- subprojects/gst-editing-services/**/* - "subprojects/gst-plugins-bad/**/*"
- subprojects/gst-python/**/* - "subprojects/gst-editing-services/**/*"
- "subprojects/gst-python/**/*"
# valgrind ges jobs are racy across the board and simply fail too often. # valgrind ges jobs are racy across the board and simply fail too often.
# Someone needs to go through job logs and figure out which jobs to mark # Someone needs to go through job logs and figure out which jobs to mark
# as flaky. Until then, just ignore failures. # as flaky. Until then, just ignore failures.
@ -995,21 +1015,22 @@ build documentation:
- if: '$CI_PROJECT_NAMESPACE == "gstreamer" && $CI_COMMIT_BRANCH == $GST_UPSTREAM_BRANCH' - if: '$CI_PROJECT_NAMESPACE == "gstreamer" && $CI_COMMIT_BRANCH == $GST_UPSTREAM_BRANCH'
when: never when: never
- changes: - changes:
- .gitlab-ci.yml paths:
- .gitlab-image-tags.yml - ".gitlab-ci.yml"
- ci/gitlab/*.py - ".gitlab-image-tags.yml"
- subprojects/*.wrap - "ci/gitlab/*.py"
- subprojects/gst-docs/**/* - "subprojects/*.wrap"
- subprojects/gst-devtools/**/* - "subprojects/gst-docs/**/*"
- subprojects/gst-editing-services/**/* - "subprojects/gst-devtools/**/*"
- subprojects/gst-libav/**/* - "subprojects/gst-editing-services/**/*"
- subprojects/gst-plugins-bad/**/* - "subprojects/gst-libav/**/*"
- subprojects/gst-plugins-base/**/* - "subprojects/gst-plugins-bad/**/*"
- subprojects/gst-plugins-good/**/* - "subprojects/gst-plugins-base/**/*"
- subprojects/gst-plugins-ugly/**/* - "subprojects/gst-plugins-good/**/*"
- subprojects/gstreamer/**/* - "subprojects/gst-plugins-ugly/**/*"
- subprojects/gstreamer-vaapi/**/* - "subprojects/gstreamer/**/*"
- subprojects/gst-rtsp-server/**/* - "subprojects/gstreamer-vaapi/**/*"
- "subprojects/gst-rtsp-server/**/*"
# FIXME: Using trigger: causes permission issues, workaround using old REST API. # FIXME: Using trigger: causes permission issues, workaround using old REST API.
# https://gitlab.com/gitlab-org/gitlab/-/issues/341737 # https://gitlab.com/gitlab-org/gitlab/-/issues/341737
@ -1033,16 +1054,17 @@ cerbero trigger:
- if: '$CI_PROJECT_NAMESPACE == "gstreamer" && $CI_COMMIT_BRANCH == $GST_UPSTREAM_BRANCH' - if: '$CI_PROJECT_NAMESPACE == "gstreamer" && $CI_COMMIT_BRANCH == $GST_UPSTREAM_BRANCH'
when: never when: never
- changes: - changes:
- .gitlab-ci.yml paths:
- ci/gitlab/*.py - ".gitlab-ci.yml"
- subprojects/gst-devtools/**/* - "ci/gitlab/*.py"
- subprojects/gst-editing-services/**/* - "subprojects/gst-devtools/**/*"
- subprojects/gst-libav/**/* - "subprojects/gst-editing-services/**/*"
- subprojects/gst-plugins-bad/**/* - "subprojects/gst-libav/**/*"
- subprojects/gst-plugins-base/**/* - "subprojects/gst-plugins-bad/**/*"
- subprojects/gst-plugins-good/**/* - "subprojects/gst-plugins-base/**/*"
- subprojects/gst-plugins-ugly/**/* - "subprojects/gst-plugins-good/**/*"
- subprojects/gst-python/**/* - "subprojects/gst-plugins-ugly/**/*"
- subprojects/gstreamer/**/* - "subprojects/gst-python/**/*"
- subprojects/gst-rtsp-server/**/* - "subprojects/gstreamer/**/*"
- subprojects/gst-examples/**/* - "subprojects/gst-rtsp-server/**/*"
- "subprojects/gst-examples/**/*"