Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Ensure integration status is set #12257

Merged
merged 3 commits into from
Mar 14, 2024
Merged

ci: Ensure integration status is set #12257

merged 3 commits into from
Mar 14, 2024

Conversation

olix0r
Copy link
Member

@olix0r olix0r commented Mar 14, 2024

We have been using a path filter for PRs to avoid running integration tests needlessly. However, this prevents us from having a status check that can be satisfied (i.e. for the purposes of having required checks).

This commit updates the integration workflow to match paths within the workflow, and it adds a final integrations-ok job that succeeds as long as no integrations failed.

@olix0r olix0r marked this pull request as ready for review March 14, 2024 02:12
@olix0r olix0r requested a review from a team as a code owner March 14, 2024 02:12
We have been using a path filter for PRs to avoid running integration tests
needlessly. However, this prevents us from having a status check that can be
satisfied (i.e. for the purposes of having required checks).

This commit updates the integration workflow to match paths within the workflow,
and it adds a final *integrations-ok* job that succeeds as long as no
integrations failed.
@olix0r olix0r enabled auto-merge (squash) March 14, 2024 02:53
@olix0r olix0r disabled auto-merge March 14, 2024 02:53
@olix0r olix0r merged commit 7c0b6be into main Mar 14, 2024
29 checks passed
@olix0r olix0r deleted the ver/integrations-ok branch March 14, 2024 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants