Skip to content

Commit

Permalink
chore(ci): regression detection v2 followups (#21594)
Browse files Browse the repository at this point in the history
* chore(ci): regression v2 followups

* remove merge group condition

* actions/checkout@v4

* fix casing warning
  • Loading branch information
pront authored Oct 23, 2024
1 parent 8c4e276 commit 4617145
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 10 deletions.
20 changes: 11 additions & 9 deletions .github/workflows/regression_v2.yml
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ jobs:
outputs:
source_changed: ${{ steps.filter.outputs.SOURCE_CHANGED }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4

- name: Collect file changes
id: changes
Expand Down Expand Up @@ -152,7 +152,6 @@ jobs:
# wouldn't likely introduce regressions.
- name: Determine if should not run due to irrelevant file changes
id: filter
if: github.event_name == 'merge_group'
env:
ALL: ${{ steps.changes.outputs.all_changed_files }}
IGNORE: ${{ steps.changes.outputs.ignore_files }}
Expand All @@ -175,6 +174,9 @@ jobs:
runs-on: ubuntu-latest
needs: check-source-changed
if: ${{ needs.check-source-changed.outputs.source_changed }}
steps:
- name: Gate check passed
run: echo "Source code changes detected, proceeding with regression tests"

##
## BUILD
Expand All @@ -190,9 +192,9 @@ jobs:
steps:
- uses: colpal/actions-clean@v1

- uses: actions/checkout@v3
- uses: actions/checkout@v4

- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: ${{ needs.resolve-inputs.outputs.baseline-sha }}
path: baseline-vector
Expand Down Expand Up @@ -229,9 +231,9 @@ jobs:
steps:
- uses: colpal/actions-clean@v1

- uses: actions/checkout@v3
- uses: actions/checkout@v4

- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: ${{ needs.resolve-inputs.outputs.comparison-sha }}
path: comparison-vector
Expand Down Expand Up @@ -384,7 +386,7 @@ jobs:
-f context='Regression Detection Suite / submission' \
-f target_url=${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: ${{ needs.resolve-inputs.outputs.comparison-sha }}

Expand Down Expand Up @@ -499,7 +501,7 @@ jobs:
- should-run-gate
- resolve-inputs
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4

- name: Configure AWS Credentials
uses: aws-actions/[email protected]
Expand Down Expand Up @@ -590,7 +592,7 @@ jobs:
-f context='Regression Detection Suite / analyze-experiment' \
-f target_url=${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: ${{ needs.resolve-inputs.outputs.comparison-sha }}

Expand Down
2 changes: 1 addition & 1 deletion regression/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#
# VECTOR BUILDER
#
FROM ghcr.io/vectordotdev/vector/soak-builder@sha256:c51a7091de2caebaa690e17f37dbfed4d4059dcdf5114a5596e8ca9b5ef494f3 as builder
FROM ghcr.io/vectordotdev/vector/soak-builder@sha256:c51a7091de2caebaa690e17f37dbfed4d4059dcdf5114a5596e8ca9b5ef494f3 AS builder
WORKDIR /vector
COPY . .
RUN bash scripts/environment/install-protoc.sh
Expand Down

0 comments on commit 4617145

Please sign in to comment.