Skip to content

Commit

Permalink
Trigger windows e2e tests when windows-related files change (open-tel…
Browse files Browse the repository at this point in the history
…emetry#36857)

#### Description
We've noticed that many of the CI failures on main come from Windows e2e
tests. We want to be more proactive in catching those failures, so this
PR changes the workflow triggering Windows e2e tests to ensure they run
on Pull Requests that change Windows-related files, even if the label
`Run Windows` is absent.

It does so by looking at changed files. If the word `windows` is present
in the file name, it triggers the workflow.

#### Link to tracking issue
Related to
open-telemetry#36788

---------

Signed-off-by: Arthur Silva Sens <[email protected]>
  • Loading branch information
ArthurSens authored and sbylica-splunk committed Dec 17, 2024
1 parent b8575f0 commit c25889f
Show file tree
Hide file tree
Showing 2 changed files with 48 additions and 1 deletion.
15 changes: 14 additions & 1 deletion .github/workflows/e2e-tests-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,22 @@ env:
SEGMENT_DOWNLOAD_TIMEOUT_MINS: 2

jobs:
windows-file-changed:
runs-on: ubuntu-latest
if: ${{ github.event_name == 'pull_request' }}
steps:
- name: Checkout
uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Did windows files changed
run: echo "changed=$(./.github/workflows/scripts/is_changed_file_windows.sh )" >> "$GITHUB_OUTPUT"
- run: echo $(./.github/workflows/scripts/is_changed_file_windows.sh ${{ github.event.pull_request.base.sha }} ${{ github.sha }} )

collector-build:
runs-on: windows-latest
if: ${{ github.actor != 'dependabot[bot]' && (contains(github.event.pull_request.labels.*.name, 'Run Windows') || github.event_name == 'push' || github.event_name == 'merge_group') }}
needs: [windows-file-changed]
if: ${{ github.actor != 'dependabot[bot]' && ((contains(github.event.pull_request.labels.*.name, 'Run Windows') || github.event_name == 'push' || github.event_name == 'merge_group') || needs.windows-file-changed.outputs.changed == 'true') }}
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down
34 changes: 34 additions & 0 deletions .github/workflows/scripts/is_changed_file_windows.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
#!/usr/bin/env bash

# Copyright The OpenTelemetry Authors
# SPDX-License-Identifier: Apache-2.0

#
# verifies if any changed file is related to windows architecture.
# this script is used to determine if e2e tests should be run on windows.
#
# It's intended to be used in a GitHub Actions workflow:
# bash .github/workflows/scripts/is_changed_file_windows.sh ${{ github.event.pull_request.base.sha }} ${{ github.sha }}


# Get changed files
base_sha=$1
head_sha=$2
changed_files=$(git diff --name-only $base_sha $head_sha)

# Find windows related files
windows_files=$(find * -regex ".*windows.*.go")


# Loop over changed files and check if they exist in windows_files
found_windows_file=false
for file in $changed_files; do
for windows_file in $windows_files; do
if [[ $file == "$windows_file" ]]; then
found_windows_file=true
break
fi
done
done

echo "$found_windows_file"

0 comments on commit c25889f

Please sign in to comment.