Skip to content

Commit

Permalink
chore(ci): minor fixes to workflows post merge queue enabling (vector…
Browse files Browse the repository at this point in the history
…dotdev#17462)

- fix files changed detection logic for aws integration tests
- fix job run logic in integration tests workflow
- restrict spell check workflow to only run on pull requests (not a push
to any branch)
  • Loading branch information
neuronull committed May 23, 2023
1 parent c425006 commit 9f6f6ec
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 12 deletions.
17 changes: 12 additions & 5 deletions .github/workflows/changes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -211,13 +211,20 @@ jobs:
- "src/sinks/appsignal/**"
- "src/sinks/util/**"
aws:
- "src/aws_**"
- "src/internal_events/aws_**"
- "src/sources/aws_**"
- "src/aws/**"
- "src/internal_events/aws*"
- "src/sources/aws_ecs_metrics/**"
- "src/sources/aws_kinesis_firehose/**"
- "src/sources/aws_s3/**"
- "src/sources/aws_sqs/**"
- "src/sources/util/**"
- "src/sinks/aws_**"
- "src/sinks/aws_cloudwatch_logs/**"
- "src/sinks/aws_cloudwatch_metrics/**"
- "src/sinks/aws_kinesis/**"
- "src/sinks/aws_s3/**"
- "src/sinks/aws_sqs/**"
- "src/sinks/util/**"
- "src/transforms/aws_**"
- "src/transforms/aws*"
axiom:
- "src/sinks/axiom.rs"
- "src/sinks/util/**"
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ env:
jobs:

changes:
if: github.event_name == 'pull_request'
uses: ./.github/workflows/changes.yml
with:
base_ref: ${{ github.event.pull_request.base.ref }}
Expand All @@ -43,7 +42,6 @@ jobs:

# Calls the Integration Test workflow for each integration that was detected to have files changed that impact it.
integration-matrix:
if: always()
uses: ./.github/workflows/integration-test.yml
with:
if: ${{ matrix.run.if }}
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/regression.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,8 @@ jobs:
all_changed:
- added|deleted|modified: "**"
ignore:
- "./.github/**"
- "./.gitignore"
- "distribution/**"
- "rust-doc/**"
- "docs/**"
Expand Down
5 changes: 0 additions & 5 deletions .github/workflows/spelling.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,11 +57,6 @@ name: Check Spelling
# ... otherwise adjust the `with:` as you wish

on:
push:
branches:
- "**"
tags-ignore:
- "**"
pull_request_target:
branches:
- "**"
Expand Down

0 comments on commit 9f6f6ec

Please sign in to comment.