Skip to content

Commit a59cb29

Browse files
authored
ci: skip "Install dependencies" conditionally (#812)
Before: Always execute "Install dependencies" even though build step could be skipped. After: Skip "Install dependencies" step if build step is omitted. It will reduces workflow execution times. Signed-off-by: Kentaro Hayashi <[email protected]>
1 parent 184d9f3 commit a59cb29

File tree

4 files changed

+30
-24
lines changed

4 files changed

+30
-24
lines changed

.github/workflows/apt-arm.yml

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -37,20 +37,22 @@ jobs:
3737
runs-on: ubuntu-24.04-arm
3838
steps:
3939
- uses: actions/checkout@v4
40-
- name: Install dependencies
41-
run: |
42-
sudo apt update
43-
sudo apt -V install ruby ruby-bundler ruby-serverspec
44-
sudo gem install bundler:2.2.9 --no-document
45-
sudo gem uninstall fileutils
4640
- name: cache deb
4741
uses: actions/cache@v4
4842
id: cache-deb
4943
with:
5044
path: |
5145
fluent-package/apt/repositories
5246
key: ${{ runner.os }}-cache-${{ matrix.rake-job }}-arm64-${{ hashFiles('**/config.rb', '**/Rakefile', '**/Gemfile*', 'fluent-package/templates/**', 'fluent-package/debian/**', 'fluent-package/apt/**/Dockerfile') }}
47+
- name: Install dependencies
48+
if: ${{ ! steps.cache-deb.outputs.cache-hit }}
49+
run: |
50+
sudo apt update
51+
sudo apt -V install ruby ruby-bundler ruby-serverspec
52+
sudo gem install bundler:2.2.9 --no-document
53+
sudo gem uninstall fileutils
5354
- name: Build deb with Docker
55+
if: ${{ ! steps.cache-deb.outputs.cache-hit }}
5456
run: |
5557
rake apt:build APT_TARGETS=${{ matrix.rake-job }}-arm64 ${{ matrix.rake-options }}
5658
- name: Upload fluent-package deb

.github/workflows/apt.yml

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,6 @@ jobs:
2929
runs-on: ubuntu-latest
3030
steps:
3131
- uses: actions/checkout@v4
32-
- name: Install dependencies
33-
run: |
34-
sudo apt update
35-
sudo apt -V install ruby ruby-bundler ruby-serverspec
36-
sudo gem install bundler:2.2.9 --no-document
37-
sudo gem uninstall fileutils
3832
- name: cache deb
3933
uses: actions/cache@v4
4034
id: cache-deb
@@ -45,6 +39,13 @@ jobs:
4539
fluent-lts-apt-source/apt/repositories
4640
v6-test/fluent-package/apt/repositories
4741
key: ${{ runner.os }}-cache-${{ matrix.rake-job }}-${{ hashFiles('**/config.rb', '**/Rakefile', '**/Gemfile*', 'fluent-package/templates/**', 'fluent-package/debian/**', 'fluent-package/apt/**/Dockerfile') }}
42+
- name: Install dependencies
43+
if: ${{ ! steps.cache-deb.outputs.cache-hit }}
44+
run: |
45+
sudo apt update
46+
sudo apt -V install ruby ruby-bundler ruby-serverspec
47+
sudo gem install bundler:2.2.9 --no-document
48+
sudo gem uninstall fileutils
4849
- name: Build deb with Docker
4950
if: ${{ ! steps.cache-deb.outputs.cache-hit }}
5051
run: |

.github/workflows/yum-arm.yml

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,20 +35,22 @@ jobs:
3535
runs-on: ubuntu-24.04-arm
3636
steps:
3737
- uses: actions/checkout@v4
38-
- name: Install dependencies
39-
run: |
40-
sudo apt update
41-
sudo apt -V install ruby ruby-bundler ruby-serverspec
42-
sudo gem install bundler:2.2.9 --no-document
43-
sudo gem uninstall fileutils
4438
- name: cache rpm
4539
uses: actions/cache@v4
4640
id: cache-rpm
4741
with:
4842
path: |
4943
fluent-package/yum/repositories
5044
key: ${{ runner.os }}-cache-${{ matrix.rake-job }}-aarch64-${{ hashFiles('**/config.rb', '**/Rakefile', '**/Gemfile*', '**/*.spec.in', 'fluent-package/templates/**', 'fluent-package/yum/**/Dockerfile') }}
45+
- name: Install dependencies
46+
if: ${{ ! steps.cache-rpm.outputs.cache-hit }}
47+
run: |
48+
sudo apt update
49+
sudo apt -V install ruby ruby-bundler ruby-serverspec
50+
sudo gem install bundler:2.2.9 --no-document
51+
sudo gem uninstall fileutils
5152
- name: Build rpm with Docker
53+
if: ${{ ! steps.cache-rpm.outputs.cache-hit }}
5254
run: |
5355
rake yum:build YUM_TARGETS=${{ matrix.rake-job }}-aarch64
5456
- name: Upload fluent-package rpm

.github/workflows/yum.yml

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,6 @@ jobs:
2929
runs-on: ubuntu-latest
3030
steps:
3131
- uses: actions/checkout@v4
32-
- name: Install dependencies
33-
run: |
34-
sudo apt update
35-
sudo apt -V install ruby ruby-bundler ruby-serverspec
36-
sudo gem install bundler:2.2.9 --no-document
37-
sudo gem uninstall fileutils
3832
- name: cache rpm
3933
uses: actions/cache@v4
4034
id: cache-rpm
@@ -43,6 +37,13 @@ jobs:
4337
fluent-package/yum/repositories
4438
v6-test/fluent-package/yum/repositories
4539
key: ${{ runner.os }}-cache-${{ matrix.rake-job }}-${{ hashFiles('**/config.rb', '**/Rakefile', '**/Gemfile*', '**/*.spec.in', 'fluent-package/templates/**', 'fluent-package/yum/**/Dockerfile') }}
40+
- name: Install dependencies
41+
if: ${{ ! steps.cache-rpm.outputs.cache-hit }}
42+
run: |
43+
sudo apt update
44+
sudo apt -V install ruby ruby-bundler ruby-serverspec
45+
sudo gem install bundler:2.2.9 --no-document
46+
sudo gem uninstall fileutils
4647
- name: Build rpm with Docker
4748
if: ${{ ! steps.cache-rpm.outputs.cache-hit }}
4849
run: |

0 commit comments

Comments
 (0)