Skip to content

Commit 9c7ea13

Browse files
yew1ebpan3793
authored andcommitted
[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 <yew1eb@gmail.com> Signed-off-by: Cheng Pan <chengpan@apache.org>
1 parent 02dadde commit 9c7ea13

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

.github/labeler.yml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,9 @@
110110
"module:service":
111111
- changed-files:
112112
- any-glob-to-any-file: [
113-
'service/**/*'
113+
'service/**/*',
114+
'lifecycle-manager/**/*',
115+
'multipart-uploader/**/*',
114116
]
115117

116118
"module:spi":
@@ -162,3 +164,9 @@
162164
- any-glob-to-any-file: [
163165
'client-tez/**/*',
164166
]
167+
168+
"module:rust":
169+
- changed-files:
170+
- any-glob-to-any-file: [
171+
'rust/**/*',
172+
]

0 commit comments

Comments
 (0)