Skip to content

Commit d3e9fb5

Browse files
laura-naopawiecz
authored andcommitted
config: jobs-chromeos: Fix rules override for Tast decoder tests
The current rules override in the Tast decoder test job definition only considers the minimum kernel version, ignoring the existing tree-specific rules. Reintroduce the tree rules. Signed-off-by: Laura Nao <laura.nao@collabora.com>
1 parent da314f7 commit d3e9fb5

1 file changed

Lines changed: 55 additions & 21 deletions

File tree

config/jobs-chromeos.yaml

Lines changed: 55 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -742,25 +742,35 @@ jobs:
742742

743743
tast-decoder-chromestack-arm64-qualcomm:
744744
<<: *tast-decoder-chromestack-job
745-
rules: *min-6_7-rules
745+
rules:
746+
<<: *tast-decoder-rules
747+
<<: *min-6_7-rules
746748

747749
tast-debian-decoder-chromestack-arm64-qualcomm:
748750
<<: *tast-debian-decoder-chromestack-job
749-
rules: *min-6_7-rules
751+
rules:
752+
<<: *tast-decoder-rules
753+
<<: *min-6_7-rules
750754

751755
tast-decoder-chromestack-verification-arm64-qualcomm:
752756
<<: *tast-decoder-chromestack-verification-job
753-
rules: *min-6_7-rules
757+
rules:
758+
<<: *tast-decoder-rules
759+
<<: *min-6_7-rules
754760

755761
tast-debian-decoder-chromestack-verification-arm64-qualcomm:
756762
<<: *tast-debian-decoder-chromestack-verification-job
757-
rules: *min-6_7-rules
763+
rules:
764+
<<: *tast-decoder-rules
765+
<<: *min-6_7-rules
758766

759767
tast-decoder-chromestack-arm64-qualcomm-pre6_7:
760768
<<: *tast-decoder-chromestack-job
761769
params:
762770
<<: *tast-decoder-chromestack-params
763-
rules: *max-6_6-rules
771+
rules:
772+
<<: *tast-decoder-rules
773+
<<: *max-6_6-rules
764774

765775
tast-decoder-chromestack-verification-arm64-qualcomm-pre6_7:
766776
<<: *tast-decoder-chromestack-verification-job
@@ -773,27 +783,35 @@ jobs:
773783
# Qualcomm-specific: those always fail with pre-6.7 kernels
774784
- video.ChromeStackDecoderVerification.vp9_0_group1_frm_resize
775785
- video.ChromeStackDecoderVerification.vp9_0_group1_sub8x8_sf
776-
rules: *max-6_6-rules
786+
rules:
787+
<<: *tast-decoder-rules
788+
<<: *max-6_6-rules
777789

778790
tast-debian-decoder-chromestack-arm64-qualcomm-pre6_7:
779791
<<: *tast-debian-decoder-chromestack-job
780792
params:
781793
<<: *tast-decoder-chromestack-params
782-
rules: *max-6_6-rules
794+
rules:
795+
<<: *tast-decoder-rules
796+
<<: *max-6_6-rules
783797

784798
tast-debian-decoder-chromestack-verification-arm64-qualcomm-pre6_7:
785799
<<: *tast-debian-decoder-chromestack-verification-job
786800
params:
787801
<<: *tast-decoder-chromestack-verification-params
788802
excluded_tests: *tast-decoder-chromestack-verification-arm64-qualcomm-pre6_7-excluded_tests
789-
rules: *max-6_6-rules
803+
rules:
804+
<<: *tast-decoder-rules
805+
<<: *max-6_6-rules
790806

791807
tast-debian-decoder-chromestack-verification-arm64-qualcomm-pre6_7:
792808
<<: *tast-debian-decoder-chromestack-verification-job
793809
params:
794810
<<: *tast-decoder-chromestack-verification-params
795811
excluded_tests: *tast-decoder-chromestack-verification-arm64-qualcomm-pre6_7-excluded_tests
796-
rules: *max-6_6-rules
812+
rules:
813+
<<: *tast-decoder-rules
814+
<<: *max-6_6-rules
797815

798816
tast-decoder-chromestack-x86-amd: *tast-decoder-chromestack-job
799817
tast-decoder-chromestack-x86-intel: *tast-decoder-chromestack-job
@@ -817,11 +835,15 @@ jobs:
817835

818836
tast-decoder-v4l2-sf-vp9-arm64-qualcomm:
819837
<<: *tast-decoder-v4l2-sf-vp9-job
820-
rules: *min-6_7-rules
838+
rules:
839+
<<: *tast-decoder-rules
840+
<<: *min-6_7-rules
821841

822842
tast-debian-decoder-v4l2-sf-vp9-arm64-qualcomm:
823843
<<: *tast-debian-decoder-v4l2-sf-vp9-job
824-
rules: *min-6_7-rules
844+
rules:
845+
<<: *tast-decoder-rules
846+
<<: *min-6_7-rules
825847

826848
tast-decoder-v4l2-sf-vp9-arm64-qualcomm-pre6_7:
827849
<<: *tast-decoder-v4l2-sf-vp9-job
@@ -836,38 +858,50 @@ jobs:
836858
- video.PlatformDecoding.v4l2_stateful_vp9_0_group3_sub8x8_sf
837859
- video.PlatformDecoding.v4l2_stateful_vp9_0_group4_frm_resize
838860
- video.PlatformDecoding.v4l2_stateful_vp9_0_group4_sub8x8_sf
839-
rules: *max-6_6-rules
840-
861+
rules:
862+
<<: *tast-decoder-rules
863+
<<: *max-6_6-rules
864+
841865
tast-debian-decoder-v4l2-sf-vp9-arm64-qualcomm-pre6_7:
842866
<<: *tast-debian-decoder-v4l2-sf-vp9-job
843867
params:
844868
<<: *tast-decoder-v4l2-sf-vp9-params
845869
excluded_tests: *tast-decoder-v4l2-sf-vp9-arm64-qualcomm-pre6_7-excluded_tests
846-
rules: *max-6_6-rules
847-
870+
rules:
871+
<<: *tast-decoder-rules
872+
<<: *max-6_6-rules
873+
848874
tast-decoder-v4l2-sf-vp9-extra-arm64-qualcomm:
849875
<<: *tast-decoder-v4l2-sf-vp9-extra-job
850-
rules: *min-6_7-rules
876+
rules:
877+
<<: *tast-decoder-rules
878+
<<: *min-6_7-rules
851879

852880
tast-debian-decoder-v4l2-sf-vp9-extra-arm64-qualcomm:
853881
<<: *tast-debian-decoder-v4l2-sf-vp9-extra-job
854-
rules: *min-6_7-rules
855-
882+
rules:
883+
<<: *tast-decoder-rules
884+
<<: *min-6_7-rules
885+
856886
tast-decoder-v4l2-sf-vp9-extra-arm64-qualcomm-pre6_7:
857887
<<: *tast-decoder-v4l2-sf-vp9-extra-job
858888
params:
859889
<<: *tast-decoder-v4l2-sf-vp9-extra-params
860890
excluded_tests: &tast-decoder-v4l2-sf-vp9-extra-arm64-qualcomm-pre6_7-excluded_tests
861891
- video.PlatformDecoding.v4l2_stateful_vp9_0_level5_0_frm_resize
862892
- video.PlatformDecoding.v4l2_stateful_vp9_0_level5_0_sub8x8_sf
863-
rules: *max-6_6-rules
864-
893+
rules:
894+
<<: *tast-decoder-rules
895+
<<: *max-6_6-rules
896+
865897
tast-debian-decoder-v4l2-sf-vp9-extra-arm64-qualcomm-pre6_7:
866898
<<: *tast-decoder-v4l2-sf-vp9-extra-job
867899
params:
868900
<<: *tast-decoder-v4l2-sf-vp9-extra-params
869901
excluded_tests: *tast-decoder-v4l2-sf-vp9-extra-arm64-qualcomm-pre6_7-excluded_tests
870-
rules: *max-6_6-rules
902+
rules:
903+
<<: *tast-decoder-rules
904+
<<: *max-6_6-rules
871905

872906
tast-decoder-v4l2-sl-av1-arm64-mediatek: *tast-decoder-v4l2-sl-av1-job
873907
tast-decoder-v4l2-sl-h264-arm64-mediatek: *tast-decoder-v4l2-sl-h264-job

0 commit comments

Comments
 (0)