Merge pull request #4715 from DataDog/tonycthsu/refactor-update-system-tests

Refactor system test update job

Author: TonyCTHsu

.github/scripts/update_reference.rb

@@ -0,0 +1,51 @@
+#!/bin/bash
+
+# This script updates the reference in a YAML file.
+
+# Check if required environment variables are set
+if [ -z "$TARGET" ]; then
+    echo "Error: TARGET environment variable is not set"
+    exit 1
+fi
+
+if [ -z "$REF" ]; then
+    echo "Error: REF environment variable is not set"
+    exit 1
+fi
+
+if [ -z "$PATTERN" ]; then
+    echo "Error: PATTERN environment variable is not set"
+    exit 1
+fi
+
+echo "Target: $TARGET"
+echo "Ref: $REF"
+
+# Remove leading and trailing forward slashes from pattern
+CLEAN_PATTERN=$(echo "$PATTERN" | sed 's/^\///;s/\/$//')
+echo "Pattern: $CLEAN_PATTERN"
+
+# Create a temporary file
+TEMP_FILE=$(mktemp)
+
+# Set trap to cleanup on script exit
+trap 'rm -f "$TEMP_FILE"' EXIT
+
+# Read the file and perform the substitution
+if [ -f "$TARGET" ]; then
+    # Perform the substitution and save to temporary file
+    # We use perl here because sed's regex support varies across platforms
+    perl -pe "s/$CLEAN_PATTERN/\${1}$REF\${3}/g" "$TARGET" > "$TEMP_FILE"
+
+    # Compare files to check if any changes were made
+    if cmp -s "$TARGET" "$TEMP_FILE"; then
+        echo "No references found in $TARGET"
+    else
+        # Copy the temp file back to the target
+        cp "$TEMP_FILE" "$TARGET"
+        echo "✓ Updated references in $TARGET"
+    fi
+else
+    echo "Error: Target file $TARGET does not exist"
+    exit 1
+fi

.github/scripts/update_reference.sh

@@ -40,13 +40,13 @@ jobs:
           persist-credentials: false
           ref: ${{ github.event.inputs.ref || '' }}
 
-      - run: .github/scripts/update_reference.rb
+      - run: .github/scripts/update_reference.sh
         env:
           TARGET: ".github/workflows/system-tests.yml"
           PATTERN: '(\s*SYSTEM_TESTS_REF:\s+)(\S+)(\s+# Automated:.*)'
           REF: ${{ steps.system-test-ref.outputs.commit }}
 
-      - run: .github/scripts/update_reference.rb
+      - run: .github/scripts/update_reference.sh
         env:
           TARGET: ".github/workflows/parametric-tests.yml"
           PATTERN: '(\s*SYSTEM_TESTS_REF:\s+)(\S+)(\s+# Automated:.*)'
@@ -70,12 +70,13 @@ jobs:
           branch: auto-generate/update-system-tests
           title: '[🤖] Update System Tests'
           base: master
+          sign-commits: true
           labels: dev/internal
-          commit-message: "[🤖] Update System Tests: https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}"
+          commit-message: "[🤖] Update System Tests: ${{github.server_url}}/${{ github.repository }}/actions/runs/${{ github.run_id }}"
           delete-branch: true
           add-paths: |
             .github/
           body: |
-            _This is an auto-generated PR from [here](https://github.com/DataDog/dd-trace-rb/blob/master/.github/workflows/update-system-tests.yml)_
+            _This is an auto-generated PR from [here](${{github.server_url}}/${{ github.repository }}/blob/master/.github/workflows/update-system-tests.yml)_
             The PR updates the system tests (Updated to commit: ${{ steps.system-test-ref.outputs.commit }})
             Please review the changes and merge when ready