ryankert01 opened a new issue, #4543: URL: https://github.com/apache/iggy/issues/4543
### Description [`.config/**` sits under the `rust-workspace` component](https://github.com/apache/iggy/blob/bb1ef105e54483aea94864c694457f42688cdfa7/.github/config/components.yml#L20-L26). It was put there when the [Doris nextest test-group was dropped](https://github.com/apache/iggy/commit/d693eb93d878a7759221fba05dcea20e52dd1249), so that a `nextest.toml` change would run the Rust tests at all. `rust-workspace` is the root of the dependency graph, though. Through `rust-sdk` it fans out to every foreign SDK, BDD and examples lane (`sdk-*`, `bdd-*`, `examples-*`), plus `rust-msrv`, `rust-miri` and `rust-publish`. None of those reads `nextest.toml` or `rail.toml`. That has a measurable cost. Since then, five Kafka gateway changes have edited `.config/nextest.toml` ([1](https://github.com/apache/iggy/commit/63362dc5478c63fe006341fadff5dd5d9acb68c2), [2](https://github.com/apache/iggy/commit/2ad0b19006bdfe9d72fb6d6420f83e22d027d0b3), [3](https://github.com/apache/iggy/commit/20d3237935eafeb0eff72f1402ca7a0b1c7d2a65), [4](https://github.com/apache/iggy/commit/8d0e28d9cebaa174b74709f8d256db510e067418), [5](https://github.com/apache/iggy/commit/8152a607410b686f37ca5224e18b4e2eabecbd60)). Apart from `.config/`, the first four touched only `core/gateways/kafka`, so the edit to `.config/` was the only reason the SDK, BDD and examples lanes ran. ### Affected area / component CI / build / tooling ### Proposed solution Move `.config/**` from `rust-workspace.paths` to the components whose jobs actually consume it: - [`rust`](https://github.com/apache/iggy/blob/bb1ef105e54483aea94864c694457f42688cdfa7/.github/config/components.yml#L186) (the `test-*` tasks run nextest and cargo-rail) - [`rust-storage-compat`](https://github.com/apache/iggy/blob/bb1ef105e54483aea94864c694457f42688cdfa7/.github/config/components.yml#L148) ([runs its own nextest invocation](https://github.com/apache/iggy/blob/bb1ef105e54483aea94864c694457f42688cdfa7/.github/workflows/_test.yml#L109)) Also fix the trailing comment, which currently claims the path gates every Rust test job. ### Alternatives considered - Leave it. This is safe, but every routine `nextest.toml` edit pays for the full SDK matrix. - Add `.config/**` to [`[change-detection].infrastructure` in `rail.toml`](https://github.com/apache/iggy/blob/bb1ef105e54483aea94864c694457f42688cdfa7/.config/rail.toml#L23). That has no effect: cargo-rail 0.23.0 already plans an ownerless top-level `.toml` change at workspace scope, and the problem is in `_detect.yml` gating, which happens before cargo-rail runs. ### Contribution - [x] I'm willing to submit a pull request to implement this feature -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
