WIP: support Ruby 2.4

For td-agent v3.1.1 and later.
(Embedded Fluentd version must be v1)

Exclude macOS because the latest macOS runner does not support
old Ruby versions.

Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>

Author: daipom

.github/workflows/main.yml

@@ -15,9 +15,13 @@ jobs:
         os: [ubuntu-latest, macos-latest, windows-latest]
         include:
           - os: ubuntu-latest
-            ruby: 2.7.1
+            ruby: '2.7.1'
+          - os: ubuntu-latest
+            ruby: '2.4.2'
+          - os: windows-latest
+            ruby: '2.7.1'
           - os: windows-latest
-            ruby: 2.7.1
+            ruby: '2.4.2'
     runs-on: ${{ matrix.os }}
     name: Test with Ruby ${{ matrix.ruby }} on ${{ matrix.os }}
     steps:
@@ -50,14 +54,18 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ['3.4', '3.3', '3.2', '2.7.1']
+        ruby: ['3.4', '3.3', '3.2', '2.7.1', '2.4.2']
         # macOS is excluded because of the difficulty in assuming a macOS shell behavior.
         os: [ubuntu-latest, windows-latest]
         include:
           - os: ubuntu-latest
             shell: bash
           - os: windows-latest
             shell: cmd
+          - ruby: '2.7.1'
+            fluentd: '1.11.1'
+          - ruby: '2.4.2'
+            fluentd: '1.0.2'
     runs-on: ${{ matrix.os }}
     name: Gem test with Ruby ${{ matrix.ruby }} on ${{ matrix.os }}
     steps:
@@ -69,6 +77,9 @@ jobs:
         with:
           name: gem
           path: pkg
+      - name: Install specific version Fluentd
+        if: ${{ matrix.fluentd != '' }}
+        run: gem install fluentd --version ${{ matrix.fluentd }}
       - name: Install gem
         run: gem install pkg/*
       - name: Test command (bash)

lib/fluent/tail_checker/collection_ratio_checker.rb

@@ -46,7 +46,7 @@ def check
           ratio = collection_ratio(entry.pos, size)
           if ratio < @collection_ratio_threshold
             unacceptable_collection_ratio_found = true
-            unacceptable_collection_ratio_path_and_ratio_list.append([entry.path, ratio])
+            unacceptable_collection_ratio_path_and_ratio_list.push([entry.path, ratio])
           end
         end
 

lib/fluent/tail_checker/pos.rb

@@ -50,7 +50,7 @@ def self.load_entries(path)
       end
 
       def watching_entries
-        entries.filter do |entry|
+        entries.select do |entry|
           not entry.unwatched?
         end
       end