Skip to content

[CI] : add test scripts and enhance CI workflow#12

Merged
maulik-mk merged 3 commits intodevfrom
ci/improve-docker-ci-scripts
Mar 21, 2026
Merged

[CI] : add test scripts and enhance CI workflow#12
maulik-mk merged 3 commits intodevfrom
ci/improve-docker-ci-scripts

Conversation

@maulik-mk
Copy link
Copy Markdown
Owner

This pull request significantly refactors the CI workflow to make Docker image builds more robust and efficient, especially around versioning, label management, and conditional publishing. It also modularizes test asset downloading and test execution into separate scripts, improving maintainability and clarity.

CI Workflow Improvements:

  • Added steps to extract the VERSION from the Dockerfile and set a BUILD_DATE environment variable, enabling version-aware builds and metadata tagging.
  • Implemented logic to extract all Dockerfile labels, substitute build-time values, and use them in the Docker build process for better image metadata.
  • Added a check to skip Docker builds and pushes if the image version already exists in GHCR, preventing redundant work and accidental overwrites.
  • Restricted Docker image build and push steps to only run for new major/minor versions on the main branch, aligning with semantic versioning best practices.

Test Automation and Script Modularization:

  • Moved the video download logic into a dedicated script (.github/scripts/download_test_video.sh) and the test execution logic into another script (.github/scripts/run_tests.sh), simplifying the workflow YAML and making test steps reusable. [1] [2] [3]

Minor Workflow Cleanups:

  • Standardized branch list formatting in workflow triggers for consistency.

- Add scripts to download test video and run ffmpeg tests
- Refactor CI workflow for better Docker versioning, labeling, and image push logic
- Integrate new scripts into CI for improved test automation
@maulik-mk maulik-mk merged commit 3cd877c into dev Mar 21, 2026
1 check passed
@maulik-mk maulik-mk deleted the ci/improve-docker-ci-scripts branch April 5, 2026 13:19
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.

1 participant