Removed noisy yamls, updated sh script to handle vp9 download only webm clip removed ivf dep, updated lib gstreamer with addon local clip function for vp9 test

Signed-off-by: Nitin Nakka <nitinn@qti.qualcomm.com>

Author: nitinn22

Runner/suites/Multimedia/GSTreamer/Display/Waylandsink_Playback/Waylandsink_Playback.yaml

@@ -10,40 +10,19 @@ metadata:
     - linux
   scope:
     - functional
+
 params:
-  VIDEO_DURATION: "30" # seconds
-  VIDEO_PATTERN: "smpte" # smpte|snow|black|white|red|green|blue|checkers-1|checkers-2|ball
-  VIDEO_WIDTH: "1920" # pixels
-  VIDEO_HEIGHT: "1080" # pixels
-  VIDEO_FRAMERATE: "30" # fps
-  VIDEO_GST_DEBUG: "2" # 1-9
+  VIDEO_DURATION: "30"    # seconds
+  VIDEO_PATTERN: "smpte"  # smpte|snow|black|white|red|green|blue|checkers-1|checkers-2|ball
+  VIDEO_WIDTH: "1920"     # pixels
+  VIDEO_HEIGHT: "1080"    # pixels
+  VIDEO_FRAMERATE: "30"   # fps
+  VIDEO_GST_DEBUG: "2"    # 1-9
+
 run:
   steps:
-    - |
-        REPO_PATH=$PWD
-        cd Runner/suites/Multimedia/GSTreamer/Display/Waylandsink_Playback || exit 0
-
-        CMD="./run.sh"
-
-        # Use CLI args only (no exports). Script defaults apply if params are empty.
-        if [ -n "${VIDEO_WIDTH}" ] && [ -n "${VIDEO_HEIGHT}" ]; then
-          CMD="${CMD} --resolution ${VIDEO_WIDTH}x${VIDEO_HEIGHT}"
-        fi
-        if [ -n "${VIDEO_PATTERN}" ]; then
-          CMD="${CMD} --pattern ${VIDEO_PATTERN}"
-        fi
-        if [ -n "${VIDEO_DURATION}" ]; then
-          CMD="${CMD} --duration ${VIDEO_DURATION}"
-        fi
-        if [ -n "${VIDEO_FRAMERATE}" ]; then
-          CMD="${CMD} --framerate ${VIDEO_FRAMERATE}"
-        fi
-        if [ -n "${VIDEO_GST_DEBUG}" ]; then
-          CMD="${CMD} --gst-debug ${VIDEO_GST_DEBUG}"
-        fi
-
-        echo "[LAVA] Running: ${CMD}"
-        sh -c "${CMD}" || true
-
-        # Send result to LAVA (single command line to avoid YAML parsing issues)
-        ${REPO_PATH}/Runner/utils/send-to-lava.sh Waylandsink_Playback.res
+    - REPO_PATH="$PWD"
+    - cd Runner/suites/Multimedia/GSTreamer/Display/Waylandsink_Playback/
+    - export VIDEO_DURATION VIDEO_PATTERN VIDEO_WIDTH VIDEO_HEIGHT VIDEO_FRAMERATE VIDEO_GST_DEBUG
+    - ./run.sh --resolution "${VIDEO_WIDTH}x${VIDEO_HEIGHT}" --pattern "${VIDEO_PATTERN}" --duration "${VIDEO_DURATION}" --framerate "${VIDEO_FRAMERATE}" --gst-debug "${VIDEO_GST_DEBUG}" || true
+    - $REPO_PATH/Runner/utils/send-to-lava.sh Waylandsink_Playback.res || true

Runner/suites/Multimedia/GSTreamer/Video/Video_Encode_Decode/README.md

@@ -5,14 +5,14 @@ This directory contains the **Video_Encode_Decode** validation test for Qualcomm
 It validates video **encoding and decoding** using **GStreamer (`gst-launch-1.0`)** with V4L2 hardware-accelerated codecs:
 - **v4l2h264enc** / **v4l2h264dec** (H.264/AVC)
 - **v4l2h265enc** / **v4l2h265dec** (H.265/HEVC)
-- **v4l2vp9dec** (VP9 decode only - uses pre-downloaded clips converted to WebM)
+- **v4l2vp9dec** (VP9 decode only - uses pre-downloaded WebM clips)
 
 The script is designed to be **CI/LAVA-friendly**:
 - Writes **PASS/FAIL/SKIP** into `Video_Encode_Decode.res`
 - Always **exits 0** (even on FAIL/SKIP) to avoid terminating LAVA jobs early
 - Logs the **final `gst-launch-1.0` command** to console and to log files
 - Uses **videotestsrc** plugin to generate test patterns for H.264/H.265 (no external video files needed)
-- For VP9: Downloads pre-encoded clips from git repo (requires network connectivity)
+- For VP9: Downloads WebM clips from git repo (requires network connectivity)
 
 ---
 
@@ -113,14 +113,13 @@ By default, logs are written relative to the script working directory:
   decode_h264_4k.log
   decode_h265_480p.log
   decode_h265_4k.log
-  decode_vp9_320p.log        # VP9 decode test log
+  decode_vp9_480p.log        # VP9 decode test log
   encoded/                   # Encoded video files
     encode_h264_480p.mp4
     encode_h264_4k.mp4
     encode_h265_480p.mp4
     encode_h265_4k.mp4
-  320_240_10fps.ivf          # Downloaded VP9 clip (IVF format)
-  vp9_test_320p.webm         # Converted VP9 clip (WebM format - used for decode test)
+  VP9_640x480_10s.webm      # Downloaded VP9 clip (WebM format)
   dmesg/                     # dmesg scan outputs (if available)
 ```
 
@@ -144,7 +143,7 @@ By default, logs are written relative to the script working directory:
 ### Parser Elements
 - `h264parse` - H.264 stream parser
 - `h265parse` - H.265 stream parser
-- `ivfparse` - IVF container parser (for VP9)
+- `matroskademux` - WebM/Matroska container demuxer (for VP9)
 
 ### Network Requirements (for VP9)
 - Network connectivity (Ethernet or WiFi)
@@ -335,27 +334,10 @@ Where:
 - Parser ensures proper stream format
 - `fakesink` discards output (no display needed for validation)
 
-### VP9 Clip Conversion Pipeline
-
-Before decoding, the downloaded IVF file is converted to WebM (Matroska) container:
-
-```
-filesrc location=320_240_10fps.ivf 
-  ! ivfparse 
-  ! matroskamux 
-  ! filesink location=vp9_test_320p.webm
-```
-
-Where:
-- `ivfparse` parses the downloaded IVF container
-- `matroskamux` remuxes to WebM/Matroska container
-- **If conversion fails**: Test is skipped with reason "GST conversion failure"
-- **No IVF fallback**: The test will not use IVF directly if conversion fails
-
 ### Decoding Pipeline (VP9)
 
 ```
-filesrc location=vp9_test_320p.webm 
+filesrc location=VP9_640x480_10s.webm 
   ! matroskademux 
   ! v4l2vp9dec 
   ! videoconvert 
@@ -364,9 +346,8 @@ filesrc location=vp9_test_320p.webm
 
 Where:
 - `matroskademux` parses WebM/Matroska container format
-- Input file is the converted WebM file (not IVF directly)
-- Resolution: 320x240
-- **Important**: Test skips if WebM conversion failed (no IVF fallback)
+- Input file is the downloaded WebM file
+- Resolution: 640x480
 
 ---
 
@@ -416,45 +397,19 @@ Where:
 
 ### H) VP9 decode fails with "Input file not found"
 - Ensure network connectivity is available
-- Check if clip was downloaded: `ls -l logs/Video_Encode_Decode/320_240_10fps.ivf`
+- Check if clip was downloaded: `ls -l logs/Video_Encode_Decode/VP9_640x480_10s.webm`
 - Manually download if needed:
   ```bash
   cd logs/Video_Encode_Decode/
-  wget https://github.com/qualcomm-linux/qcom-linux-testkit/releases/download/IRIS-Video-Files-v1.0/video_clips_iris.tar.gz
-  tar -xzf video_clips_iris.tar.gz
-  ```
-- Check network connectivity:
-  ```bash
-  ping -c 3 github.com
-  ```
-
-### I) VP9 decode fails with "ivfparse not found"
-- Ensure `ivfparse` GStreamer plugin is installed:
-  ```bash
-  gst-inspect-1.0 ivfparse
-  ```
-- This is typically part of `gst-plugins-bad` package
-
-### J) VP9 test skips with "GST conversion failure"
-- The IVF to WebM conversion failed
-- Check if `matroskamux` plugin is available:
-  ```bash
-  gst-inspect-1.0 matroskamux
-  ```
-- Check if `ivfparse` plugin is available:
-  ```bash
-  gst-inspect-1.0 ivfparse
-  ```
-- Manually test the conversion:
-  ```bash
-  cd logs/Video_Encode_Decode/
-  gst-launch-1.0 filesrc location=320_240_10fps.ivf ! ivfparse ! matroskamux ! filesink location=test.webm
+  wget https://github.com/qualcomm-linux/qcom-linux-testkit/releases/download/GST-Video-Files-v1.0/video_clips_gst.tar.gz
+  tar -xzf video_clips_gst.tar.gz
   ```
-- Check GStreamer debug output for errors:
+### I) VP9 decode fails with "matroskademux not found"
+- Ensure `matroskademux` GStreamer plugin is installed:
   ```bash
-  GST_DEBUG=3 gst-launch-1.0 filesrc location=320_240_10fps.ivf ! ivfparse ! matroskamux ! filesink location=test.webm
+  gst-inspect-1.0 matroskademux
   ```
-- **Note**: The test will NOT fall back to using IVF directly. If conversion fails, the test skips to ensure proper container format validation
+- This is typically part of `gst-plugins-good` package
 
 ---
 
@@ -639,6 +594,6 @@ The test supports these environment variables (can be set in LAVA job definition
 - The test uses `ensure_network_online()` to establish connectivity automatically
 - If network is unavailable, VP9 tests will SKIP (not FAIL)
 - Downloaded clips are cached in the output directory for subsequent runs
-- VP9 clip: 320_240_10fps.ivf (320x240 resolution, IVF container)
+- VP9 clip: VP9_640x480_10s.webm (640x480 resolution, WebM container)
 
 ---

Runner/suites/Multimedia/GSTreamer/Video/Video_Encode_Decode/Video_Encode_Decode.yaml

@@ -2,61 +2,29 @@ metadata:
   name: gstreamer-video-encode-decode
   format: "Lava-Test Test Definition 1.0"
   description: >
-    Video encode/decode validation using GStreamer (gst-launch-1.0) with V4L2 hardware-accelerated codecs
-    on Qualcomm Linux platforms. Supports v4l2h264enc, v4l2h265enc, v4l2h264dec, v4l2h265dec, v4l2vp9dec.
-    Uses videotestsrc to generate test patterns for H.264/H.265 (no external video files needed).
-    For VP9 decode, downloads pre-encoded IVF clips from git repo, converts them to WebM container format,
-    then decodes using v4l2vp9dec. Test skips if IVF to WebM conversion fails (no IVF fallback).
-    Tests encoding at 480p and 4K resolutions, then decodes the encoded files.
+    Video encode/decode validation using GStreamer (gst-launch-1.0) with V4L2 hardware-accelerated codecs.
+    VP9 clips can be provided via VIDEO_CLIP_PATH (local dir/file/archive) or downloaded via VIDEO_CLIP_URL.
   os:
     - linux
   scope:
     - functional
+ 
 params:
-  VIDEO_TEST_MODE: "all" # all|encode|decode
-  VIDEO_CODECS: "h264,h265,vp9" # h264,h265,vp9
-  VIDEO_RESOLUTIONS: "4k" # 480p,720p,1080p,4k
-  VIDEO_DURATION: "30" # seconds
-  VIDEO_FRAMERATE: "30" # fps
-  VIDEO_STACK: "auto" # auto|upstream|downstream
-  VIDEO_GST_DEBUG: "2" # 1-9
-  VIDEO_CLIP_URL: "" # if set, overrides default GitHub URL
+  VIDEO_TEST_MODE: "all"
+  VIDEO_CODECS: "h264,h265,vp9"
+  VIDEO_RESOLUTIONS: "480p"
+  VIDEO_DURATION: "30"
+  VIDEO_FRAMERATE: "30"
+  VIDEO_STACK: "auto"
+  VIDEO_GST_DEBUG: "2"
+  VIDEO_CLIP_URL: ""
+  VIDEO_CLIP_PATH: ""   # dir/file/archive (tar/tar.gz/tar.xz/xz)
+
 run:
   steps:
-    - |
-        REPO_PATH=$PWD
-        cd Runner/suites/Multimedia/GSTreamer/Video/Video_Encode_Decode/ || exit 0
-
-        CMD="./run.sh"
-
-        # Use CLI args only (no exports). Script defaults apply if params are empty.
-        if [ -n "${VIDEO_TEST_MODE}" ]; then
-          CMD="${CMD} --mode ${VIDEO_TEST_MODE}"
-        fi
-        if [ -n "${VIDEO_CODECS}" ]; then
-          CMD="${CMD} --codecs ${VIDEO_CODECS}"
-        fi
-        if [ -n "${VIDEO_RESOLUTIONS}" ]; then
-          CMD="${CMD} --resolutions ${VIDEO_RESOLUTIONS}"
-        fi
-        if [ -n "${VIDEO_DURATION}" ]; then
-          CMD="${CMD} --duration ${VIDEO_DURATION}"
-        fi
-        if [ -n "${VIDEO_FRAMERATE}" ]; then
-          CMD="${CMD} --framerate ${VIDEO_FRAMERATE}"
-        fi
-        if [ -n "${VIDEO_STACK}" ]; then
-          CMD="${CMD} --stack ${VIDEO_STACK}"
-        fi
-        if [ -n "${VIDEO_GST_DEBUG}" ]; then
-          CMD="${CMD} --gst-debug ${VIDEO_GST_DEBUG}"
-        fi
-        if [ -n "${VIDEO_CLIP_URL}" ]; then
-          CMD="${CMD} --clip-url ${VIDEO_CLIP_URL}"
-        fi
-
-        echo "[LAVA] Running: ${CMD}"
-        sh -c "${CMD}" || true
-
-        # Send result to LAVA (single command line to avoid YAML parsing issues)
-        ${REPO_PATH}/Runner/utils/send-to-lava.sh Video_Encode_Decode.res
+    - REPO_PATH="$PWD"
+    - cd Runner/suites/Multimedia/GSTreamer/Video/Video_Encode_Decode/
+    - export VIDEO_TEST_MODE VIDEO_CODECS VIDEO_RESOLUTIONS VIDEO_DURATION VIDEO_FRAMERATE
+    - export VIDEO_STACK VIDEO_GST_DEBUG VIDEO_CLIP_URL VIDEO_CLIP_PATH
+    - ./run.sh --mode "${VIDEO_TEST_MODE}" --codecs "${VIDEO_CODECS}" --resolutions "${VIDEO_RESOLUTIONS}" --duration "${VIDEO_DURATION}" --framerate "${VIDEO_FRAMERATE}" --stack "${VIDEO_STACK}" --gst-debug "${VIDEO_GST_DEBUG}" --clip-url "${VIDEO_CLIP_URL}" --clip-path "${VIDEO_CLIP_PATH}" || true
+    - $REPO_PATH/Runner/utils/send-to-lava.sh Video_Encode_Decode.res || true