[chore](build) Reduce redundant Actions on branch-4.1 - #68025
Open
hello-stephen wants to merge 3 commits into
Open
hello-stephen wants to merge 3 commits into
hello-stephen wants to merge 3 commits into
Conversation
Limit release-branch Code Checks push runs to shell-script changes, remove the duplicate large-file push run and obsolete two-approval workflow, and path-filter the thirdparty PR workflow. Preserve existing PR, comment, and manual triggers.
Contributor
Author
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
Member
|
@hello-stephen also: consider using ubuntu-slim for short running actions like doris/.github/workflows/approve-label-trigger.yml Lines 25 to 28 in 28577df it would come with less overhead since it runs in a container. |
…triggers Remove the regression-only FE config from both the PR entry path filter and the existing focused test change filter. Thirdparty-related changes still run the script test, including its ADBC configuration assertion.
Restore the original Code Checks triggers. The shell filename filter could miss extensionless shell scripts, so this PR retains only the confirmed release-branch Action reductions.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Background
Release-branch GitHub Actions are frequently queued, including redundant post-merge runs and a thirdparty change detector on unrelated PRs.
Changes
Scope note
Code Checks is unchanged; its push trigger optimization is deferred for separate review.
External Regression's regression-test/pipeline/external/conf/fe.conf is excluded from both the PR entry filter and the old focused test filter. A change to that file alone will no longer trigger Thirdparty Script Test or its ADBC JNI assertion. The assertion still runs when a thirdparty-related change triggers the script test; this PR does not move it to another workflow.
Validation
No TeamCity pipeline or branch governance configuration is changed.