Skip to content
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

FIX [CI] Fix test docker CI #1535

Merged
merged 5 commits into from
Mar 18, 2024
Merged
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
121 changes: 61 additions & 60 deletions .github/workflows/test-docker-build.yml
Original file line number Diff line number Diff line change
@@ -1,61 +1,62 @@
# name: Test Docker images (on PR)
name: Test Docker images (on PR)

# on:
# pull_request:
# paths:
# # Run only when DockerFile files are modified
# - "docker/**"

# jobs:
# get_changed_files:
# name: "Build all modified docker images"
# runs-on: ubuntu-latest
# outputs:
# matrix: ${{ steps.set-matrix.outputs.matrix }}
# steps:
# - name: Check out code
# uses: actions/checkout@v3
# - name: Get changed files
# id: changed-files
# uses: tj-actions/changed-files@3f54ebb830831fc121d3263c1857cfbdc310cdb9 #v42
# with:
# files: docker/**
# json: "true"
# - name: Run step if only the files listed above change
# if: steps.changed-files.outputs.any_changed == 'true'
# id: set-matrix
# run: |
# for file in ${{ steps.changed-files.outputs.all_changed_files}}; do
# echo "$file was changed"
# done
# echo "matrix=${{ steps.changed-files.outputs.all_changed_files}}" >> $GITHUB_OUTPUT
# build_modified_files:
# needs: get_changed_files
# name: Build Docker images on modified files
# runs-on: ubuntu-latest
# if: ${{ needs.get_changed_files.outputs.matrix }} != ''
# strategy:
# fail-fast: false
# matrix:
# docker-file: ${{ fromJson(needs.get_changed_files.outputs.matrix) }}
# steps:
# - name: Cleanup disk
# run: |
# sudo ls -l /usr/local/lib/
# sudo ls -l /usr/share/
# sudo du -sh /usr/local/lib/
# sudo du -sh /usr/share/
# sudo rm -rf /usr/local/lib/android
# sudo rm -rf /usr/share/dotnet
# sudo du -sh /usr/local/lib/
# sudo du -sh /usr/share/
# - name: Set up Docker Buildx
# uses: docker/setup-buildx-action@v1
# - name: Check out code
# uses: actions/checkout@v3
# - name: Build Docker image
# uses: docker/build-push-action@v4
# with:
# file: ${{ matrix.docker-file }}
# context: .
# push: False
on:
pull_request:
paths:
# Run only when DockerFile files are modified
- "docker/**"
jobs:
get_changed_files:
name: "Build all modified docker images"
runs-on: ubuntu-latest
outputs:
matrix: ${{ steps.set-matrix.outputs.matrix }}
steps:
- name: Check out code
uses: actions/checkout@v3
- name: Get changed files
id: changed-files
uses: tj-actions/changed-files@1c8e6069583811afb28f97afeaf8e7da80c6be5c #v42
with:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

points out to the latest commit

files: docker/**
json: "true"
- name: Run step if only the files listed above change
if: steps.changed-files.outputs.any_changed == 'true'
id: set-matrix
env:
ALL_CHANGED_FILES: ${{ steps.changed-files.outputs.all_changed_files }}
run: |
for file in "$ALL_CHANGED_FILES"; do
echo "$file was changed"
done
echo "matrix=${{ steps.changed-files.outputs.all_changed_files}}" >> $GITHUB_OUTPUT
younesbelkada marked this conversation as resolved.
Show resolved Hide resolved
build_modified_files:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is the main change

needs: get_changed_files
name: Build Docker images on modified files
runs-on: ubuntu-latest
if: ${{ needs.get_changed_files.outputs.matrix }} != ''
strategy:
fail-fast: false
matrix:
docker-file: ${{ fromJson(needs.get_changed_files.outputs.matrix) }}
steps:
- name: Cleanup disk
run: |
sudo ls -l /usr/local/lib/
sudo ls -l /usr/share/
sudo du -sh /usr/local/lib/
sudo du -sh /usr/share/
sudo rm -rf /usr/local/lib/android
sudo rm -rf /usr/share/dotnet
sudo du -sh /usr/local/lib/
sudo du -sh /usr/share/
- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v1
- name: Check out code
uses: actions/checkout@v3
- name: Build Docker image
uses: docker/build-push-action@v4
with:
file: ${{ matrix.docker-file }}
context: .
push: False
Loading