This is an automated email from the ASF dual-hosted git repository.
hello-stephen pushed a commit to branch branch-4.2
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-4.2 by this push:
new 4865cb73eb0 [chore](build) Reduce redundant Actions on branch-4.2
(#68026)
4865cb73eb0 is described below
commit 4865cb73eb015ca95fc0acc5d0205f55ab77b590
Author: Dongyang Li <[email protected]>
AuthorDate: Tue Sep 15 21:26:53 2026 +0800
[chore](build) Reduce redundant Actions on branch-4.2 (#68026)
## Background
Release-branch GitHub Actions are frequently queued, including redundant
post-merge runs and a thirdparty change detector on unrelated PRs.
## Changes
- Remove the duplicate Check Large File push trigger; keep
pull_request_target.
- Remove the legacy Need_2_Approval workflow. The Code Owner approval
policy is tracked separately in #68014; this PR does not change it.
- Start Build Third Party Libraries on PRs only when its relevant inputs
change. Keep workflow_dispatch and issue_comment triggers.
## 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
- actionlint passed for all edited workflow files (excluding
pre-existing inline shellcheck and old-runner warnings).
- git diff --check passed.
- The currently configured branch-4.2 governance does not require these
Actions as checks; this PR does not change that policy.
No TeamCity pipeline or branch governance configuration is changed.
---
.github/workflows/build-thirdparty.yml | 5 +++-
.github/workflows/lfs-warning.yml | 3 +--
.github/workflows/pr-approve-status.yml | 43 ---------------------------------
3 files changed, 5 insertions(+), 46 deletions(-)
diff --git a/.github/workflows/build-thirdparty.yml
b/.github/workflows/build-thirdparty.yml
index 90b8f33640c..42f7abe95ac 100644
--- a/.github/workflows/build-thirdparty.yml
+++ b/.github/workflows/build-thirdparty.yml
@@ -19,6 +19,10 @@ name: Build Third Party Libraries
on:
pull_request:
+ paths:
+ - 'thirdparty/**'
+ - 'env.sh'
+ - '.github/workflows/build-thirdparty.yml'
workflow_dispatch:
issue_comment:
types: [ created ]
@@ -56,7 +60,6 @@ jobs:
- 'thirdparty/**'
- 'env.sh'
focused_test_changes:
- - 'regression-test/pipeline/external/conf/fe.conf'
- 'thirdparty/test/adbc-jni-config-test.sh'
- '.github/workflows/build-thirdparty.yml'
diff --git a/.github/workflows/lfs-warning.yml
b/.github/workflows/lfs-warning.yml
index 4db79ceded9..2bc7d1a591a 100644
--- a/.github/workflows/lfs-warning.yml
+++ b/.github/workflows/lfs-warning.yml
@@ -18,7 +18,7 @@
---
name: 'Check Large File'
-on: [push, pull_request_target]
+on: pull_request_target
jobs:
large-file-checker:
@@ -44,4 +44,3 @@ jobs:
with:
token: ${{ secrets.GITHUB_TOKEN }}
filesizelimit: 1MB
-
diff --git a/.github/workflows/pr-approve-status.yml
b/.github/workflows/pr-approve-status.yml
deleted file mode 100644
index fe0a0161607..00000000000
--- a/.github/workflows/pr-approve-status.yml
+++ /dev/null
@@ -1,43 +0,0 @@
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements. See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership. The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License. You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied. See the License for the
-# specific language governing permissions and limitations
-# under the License.
----
-name: Need_2_Approval
-
-on:
- pull_request_review:
- types: [ submitted ]
-
-jobs:
- Need_2_Approval:
- runs-on: ubuntu-latest
- timeout-minutes: 5
- steps:
- - uses: actions/checkout@v3
- - name: Install Python dependencies
- uses: actions/setup-python@v5
- with:
- python-version: '3.10' # Adjust if needed
- - name: Install match library
- run: |
- pip install --upgrade pip
- pip install match
- pip install requests
- - name: Run Python script
- run: |
- python tools/maintainers/check_review.py ${{
github.event.pull_request.number }} ${{secrets.GITHUB_TOKEN}}
-env:
- GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
