Skip to content

Public CI Video Decode and Encode Shell Scripts. #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 8 commits into from

Conversation

abbajaj806
Copy link
Contributor

Implemented shell scripts for public CI for Decode and Encode Scenarios.

  • Added Video encode test script
  • Added Video decode test script
  • Added readme document for video validation
  • Implemented Function for ActiveIP
  • Implemented Function for wget pull resource
  • Implemented Functions for untar of resource

@abbajaj806
Copy link
Contributor Author

Git rebase done. Please check and let us know if anything to be done.

@abbajaj806
Copy link
Contributor Author

Squashed changes to one Commit

Copy link
Contributor

@smuppand smuppand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

executable permissions reverted. Kindly provide 100755 to the shell scripts

smuppand
smuppand previously approved these changes May 21, 2025
Copy link
Contributor

@smuppand smuppand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LTGM.

@abbajaj806 abbajaj806 force-pushed the Video branch 2 times, most recently from 8661e57 to 427c33b Compare May 23, 2025 12:17
@abbajaj806 abbajaj806 requested a review from mwasilew May 23, 2025 12:20
Copy link
Contributor

@mwasilew mwasilew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the check for file existence before downloading was dropped entirely. Could you bring it back?

# Import test suite definitions
/var/Runner/init_env
TESTNAME="video_decode"
# shellcheck source=/Runner/init_env
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be

source=../../../../init_env


#import test functions library
. $TOOLS/functestlib.sh
# shellcheck source=/Runner/utils/functestlib.sh
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be

souce=../../../../utils/functestlib.sh

vnarapar and others added 7 commits May 23, 2025 18:34
Integrated LAVA framework with new YAML plans
Added send-to-lava.sh script with SPDX license header
Removed redundant variables in YAML files

Removed Audio_Premerge yaml file

Copied send-to-lava.sh from linaro to support result parsing with LAVA framework.
Original source: https://github.com/Linaro/test-definitions/blob/master/automated/utils/send-to-lava.sh
License: GPL-2.0-only, MIT

This change is added for LAVA framework support to all the scripts based
on testplan.

Added the license file with 644 permissionsin the same folder where send-to-lava.sh is
present as per review comment

Removed chmod permissions in testplansand gave permissions to repository

Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
Replaced 'source' with '.' in all scripts for consistency
Cleaned up bash syntax for better readability and maintainability

Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
Modified the function find_test_case_by_name and
run_specific_test_by_name in run-test.sh to find the testcase to execute
easily without depending on find command

Removed unwanted lines from run-test.sh as it is dead code

Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
Fixed all the indentation issues in all shell scripts for consistancy

Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
Made few changes for the script to work for sh. The previous change
works for bash. Hence modified to work with sh

Added back local variables as per review comment

Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
Signed-off-by: Milosz Wasilewski <mwasilew@oss.qualcomm.com>
Removed logging related functions at testcase level since it is duplicate
code and already present in functestlib.sh

Added functestlib.sh for IPA and RMNET as logging related functions
log_pass/log_fail are defined in functestlib.sh

Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
Implemented shell scripts for public CI for Decode and Encode Scenarios.

Added Video encode test script
Added Video decode test script
Added readme document for video validation
Implemented Function for ActiveIP
Implemented Function for wget pull resource
Implemented Function for untar of resource
Implemented Function to check if file is already present or not

Signed-off-by: Abhishek Bajaj <abbajaj@qti.qualcomm.com>

Public CI Video Decode and Encode Shell Scripts.
Implemented shell scripts for public CI for Decode and Encode Scenarios.

Added Video encode test script
Added Video decode test script
Added readme document for video validation
Implemented Function for ActiveIP
Implemented Function for wget pull resource
Implemented Function for untar of resource
Implemented Function to check if file is already present or not

Signed-off-by: Abhishek Bajaj <abbajaj@qti.qualcomm.com>
Implemented shell scripts for public CI for Decode and Encode Scenarios.

Added Video encode test script
Added Video decode test script
Added readme document for video validation
Implemented Function for ActiveIP
Implemented Function for wget pull resource
Implemented Function for untar of resource
Implemented Function to check if file is already present or not

Signed-off-by: Abhishek Bajaj <abbajaj@qti.qualcomm.com>
@abbajaj806 abbajaj806 closed this May 23, 2025
@abbajaj806 abbajaj806 deleted the Video branch May 23, 2025 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants