This is an automated email from the ASF dual-hosted git repository.

pan3793 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git


The following commit(s) were added to refs/heads/main by this push:
     new 9c7ea13c3 [CELEBORN-2410] Add missing module coverage to labeler.yml
9c7ea13c3 is described below

commit 9c7ea13c3b08db32cb0895bde6c1ac27c6a7a7ec
Author: yew1eb <[email protected]>
AuthorDate: Thu Aug 20 22:57:50 2026 +0800

    [CELEBORN-2410] Add missing module coverage to labeler.yml
    
    ### What changes were proposed in this pull request?
    
    `labeler.yml` did not match changes under `lifecycle-manager/`, 
`multipart-uploader/`, and `rust/` — PRs touching only these paths received no 
module label.
    
    - `lifecycle-manager/**` → `module:service` (daemon wrapper around 
`client.LifecycleManager`, package 
`org.apache.celeborn.server.lifecyclemanager`)
    - `multipart-uploader/**` → `module:service` (S3/OSS impls of 
`server.common.service.mpu.MultipartUploadHandler`, defined in the `service` 
module)
    - `rust/**` → new `module:rust` (independent Cargo-built Rust client)
    
    ### Why are the changes needed?
    
    These three directories have real source but no labeler entry, so the 
labeler action silently skips them. Changes to them never get triaged into the 
correct module label.
    
    ### Does this PR resolve a correctness bug?
    
    - [ ] Yes
    
    ### Does this PR introduce _any_ user-facing change?
    
    - [ ] Yes
    
    ### How was this patch tested?
    
    YAML syntax validated. Existing module globs unchanged.
    
    Closes #3789 from yew1eb/CELEBORN-2410-labeler-modules.
    
    Authored-by: yew1eb <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
---
 .github/labeler.yml | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/.github/labeler.yml b/.github/labeler.yml
index 46d92c02f..466a91de5 100644
--- a/.github/labeler.yml
+++ b/.github/labeler.yml
@@ -110,7 +110,9 @@
 "module:service":
   - changed-files:
     - any-glob-to-any-file: [
-      'service/**/*'
+      'service/**/*',
+      'lifecycle-manager/**/*',
+      'multipart-uploader/**/*',
     ]
 
 "module:spi":
@@ -162,3 +164,9 @@
     - any-glob-to-any-file: [
       'client-tez/**/*',
     ]
+
+"module:rust":
+  - changed-files:
+    - any-glob-to-any-file: [
+      'rust/**/*',
+    ]

Reply via email to