chore(config): rename saluki-config to saluki-config-tools - #1894
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This comment has been minimized.
This comment has been minimized.
Binary Size Analysis (Agent Data Plane)Baseline: da59d73 · Comparison: a8c0369 · diff ✅ Binary size difference within thresholdChanges by Module
Detailed Symbol Changes |
Regression Detector (Agent Data Plane)Run ID: Optimization Goals: ✅ No significant changes detectedFine details of change detection per experiment (5)Experiments configured
Bounds Checks: ✅ Passed (5)
ExplanationA change is flagged as a regression when |Δ mean %| > 5.00% in the regressing direction for its optimization goal AND SMP marks the experiment as a regression ( |
b4e40b9 to
5a9e285
Compare
5a9e285 to
101a99e
Compare
There was a problem hiding this comment.
Pull request overview
This PR renames the workspace configuration utilities crate from saluki-config to saluki-config-tools to avoid future naming collisions as typed configuration crates are introduced.
Changes:
- Renames the crate and updates workspace membership/dependencies (
Cargo.tomlfiles) and Rust import paths across binaries and libraries. - Moves/introduces
saluki-config-toolscrate source files (configuration loader, dynamic config utilities, and Serde helpers) under the new crate name. - Updates dependency metadata files (
Cargo.lock,LICENSE-3rdparty.csv) to reflect the new resolution/state.
Reviewed changes
Copilot reviewed 83 out of 94 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| LICENSE-3rdparty.csv | Updates third-party license inventory entries. |
| Cargo.toml | Workspace member/dependency rename to saluki-config-tools. |
| Cargo.lock | Lockfile updated to reflect crate rename and (currently) broader dependency resolution changes. |
| lib/saluki-config-tools/Cargo.toml | Crate renamed to saluki-config-tools. |
| lib/saluki-config-tools/src/lib.rs | ConfigurationLoader / GenericConfiguration implementation under new crate name. |
| lib/saluki-config-tools/src/provider.rs | File provider helpers for YAML/JSON with key-alias support. |
| lib/saluki-config-tools/src/space_separated.rs | Serde helper for space-separated-or-seq string lists. |
| lib/saluki-config-tools/src/duration_string.rs | DurationString and Go-style duration parsing utilities. |
| lib/saluki-config-tools/src/dynamic/mod.rs | Dynamic config module wiring. |
| lib/saluki-config-tools/src/dynamic/event.rs | Dynamic config event/update types. |
| lib/saluki-config-tools/src/dynamic/diff.rs | Dynamic config diffing logic. |
| lib/saluki-config-tools/src/dynamic/watcher.rs | Key-specific dynamic config watcher. |
| lib/saluki-env/Cargo.toml | Dependency rename to saluki-config-tools. |
| lib/saluki-env/src/workload/on_demand_pid.rs | Import path updated to saluki_config_tools. |
| lib/saluki-env/src/workload/helpers/containerd/mod.rs | Import path updated (incl. tests). |
| lib/saluki-env/src/workload/helpers/cgroups.rs | Import path updated. |
| lib/saluki-env/src/workload/collectors/containerd.rs | Import path updated. |
| lib/saluki-env/src/workload/collectors/cgroups.rs | Import path updated. |
| lib/saluki-env/src/host/providers/fixed.rs | Import path updated. |
| lib/saluki-env/src/features/detector.rs | Import path updated. |
| lib/saluki-env/src/features/containerd.rs | Import path updated. |
| lib/saluki-components/Cargo.toml | Dependency rename + feature flag rename for antithesis. |
| lib/saluki-components/src/transforms/aggregate/mod.rs | Import path updated. |
| lib/saluki-components/src/transforms/apm_stats/mod.rs | Import path updated. |
| lib/saluki-components/src/transforms/autoscaling_failover_gateway/mod.rs | Import path updated (tests). |
| lib/saluki-components/src/transforms/dogstatsd_mapper/mod.rs | Import path updated. |
| lib/saluki-components/src/transforms/mrf_gateway/mod.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/transforms/trace_obfuscation/mod.rs | Import path updated. |
| lib/saluki-components/src/transforms/trace_sampler/mod.rs | Import path updated. |
| lib/saluki-components/src/sources/otlp/mod.rs | Import path updated. |
| lib/saluki-components/src/sources/dogstatsd/mod.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/sources/dogstatsd/replay/capture_api.rs | Import path updated for duration parsing helper. |
| lib/saluki-components/src/sources/checks_ipc/mod.rs | Import path updated. |
| lib/saluki-components/src/relays/otlp/mod.rs | Import path updated. |
| lib/saluki-components/src/forwarders/datadog/mod.rs | Import path updated (incl. dynamic tests). |
| lib/saluki-components/src/forwarders/cluster_agent/mod.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/forwarders/otlp/mod.rs | Import path updated. |
| lib/saluki-components/src/encoders/datadog/events/mod.rs | Import path updated. |
| lib/saluki-components/src/encoders/datadog/logs/mod.rs | Import path updated. |
| lib/saluki-components/src/encoders/datadog/metrics/mod.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/encoders/datadog/service_checks/mod.rs | Import path updated. |
| lib/saluki-components/src/encoders/datadog/stats/mod.rs | Import path updated. |
| lib/saluki-components/src/encoders/datadog/traces/mod.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/decoders/otlp/mod.rs | Import path updated. |
| lib/saluki-components/src/destinations/dsd_debug_log/mod.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/config/mod.rs | Rustdoc links updated to new crate path. |
| lib/saluki-components/src/config/autoscaling_failover.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/config/cluster_agent.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/config/mrf.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/common/otlp/config.rs | Import path updated. |
| lib/saluki-components/src/common/datadog/apm.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/common/datadog/config.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/common/datadog/endpoints.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/common/datadog/io.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/common/datadog/obfuscation.rs | Import path updated. |
| lib/saluki-components/src/common/datadog/proxy.rs | Import path updated. |
| lib/saluki-components/src/common/datadog/retry.rs | Import path updated (incl. tests). |
| lib/saluki-components/src/common/datadog/validation.rs | Import path updated (incl. tests + type refs). |
| lib/saluki-app/Cargo.toml | Dependency rename to saluki-config-tools. |
| lib/saluki-app/src/accounting.rs | Import path updated. |
| lib/saluki-app/src/bootstrap.rs | Import path updated. |
| lib/saluki-app/src/config.rs | Import path updated. |
| lib/datadog-agent/config-testing/Cargo.toml | Dependency rename to saluki-config-tools. |
| lib/datadog-agent/config-testing/src/smoke_test.rs | Import path updated + upsert path updated. |
| lib/datadog-agent/commons/Cargo.toml | Dependency rename to saluki-config-tools. |
| lib/datadog-agent/commons/src/ipc/config.rs | Import path updated (incl. tests). |
| lib/datadog-agent/commons/src/ipc/client/mod.rs | Import path updated. |
| bin/correctness/panoramic/Cargo.toml | Dependency rename to saluki-config-tools. |
| bin/correctness/panoramic/src/config.rs | Import path updated. |
| bin/correctness/panoramic/src/correctness/config.rs | Import path updated. |
| bin/agent-data-plane/Cargo.toml | Dependency rename to saluki-config-tools. |
| bin/agent-data-plane/src/main.rs | Import path updated. |
| bin/agent-data-plane/src/config.rs | Import path updated (incl. tests). |
| bin/agent-data-plane/src/internal/mod.rs | Import path updated. |
| bin/agent-data-plane/src/internal/control_plane.rs | Import path updated. |
| bin/agent-data-plane/src/internal/logging.rs | Import path updated + log target string updated (incl. tests). |
| bin/agent-data-plane/src/internal/remote_agent.rs | Import path updated (dynamic + upsert). |
| bin/agent-data-plane/src/internal/env/mod.rs | Import path updated. |
| bin/agent-data-plane/src/internal/env/autodiscovery.rs | Import path updated. |
| bin/agent-data-plane/src/internal/env/host.rs | Import path updated. |
| bin/agent-data-plane/src/internal/env/workload/mod.rs | Import path updated. |
| bin/agent-data-plane/src/internal/env/workload/collectors/workloadmeta.rs | Import path updated. |
| bin/agent-data-plane/src/internal/env/workload/collectors/tagger.rs | Import path updated. |
| bin/agent-data-plane/src/components/tag_filterlist/mod.rs | Import path updated (incl. tests). |
| bin/agent-data-plane/src/components/ottl_filter_processor/mod.rs | Import path updated (incl. tests). |
| bin/agent-data-plane/src/components/ottl_transform_processor/mod.rs | Import path updated (incl. tests). |
| bin/agent-data-plane/src/components/host_tags/mod.rs | Import path updated. |
| bin/agent-data-plane/src/components/dogstatsd_prefix_filter/mod.rs | Import path updated (incl. tests). |
| bin/agent-data-plane/src/components/dogstatsd_post_aggregate_filter/mod.rs | Import path updated (incl. tests). |
| bin/agent-data-plane/src/cli/config.rs | Import path updated. |
| bin/agent-data-plane/src/cli/debug/mod.rs | Import path updated. |
| bin/agent-data-plane/src/cli/dogstatsd.rs | Import path updated (incl. tests). |
| bin/agent-data-plane/src/cli/run.rs | Import path updated. |
| bin/agent-data-plane/src/cli/utils.rs | Import path updated. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
tobz
left a comment
There was a problem hiding this comment.
This one I'm a little bit sketchy on.
It's not clear to me why the typed configuration that we'll generate, either for the Agent side or Saluki side, would need to live in saluki-config and thus necessitate this crate renaming... 🤔
2aa1e33 to
0b4737b
Compare
@tobz the new crate names are |
Rename the forwarder DatadogConfiguration to DatadogForwarderConfiguration, then generate a public nested DatadogConfiguration in datadog-agent-config for the support: full / support: partial overlay keys. The overlay selects the keys (schema pruning); typify generates the nested struct tree from the pruned JSON Schema. Numerics mirror the schema (f64); refinement is deferred to the translator. Mostly unused until the translator PR.
The name saluki-config conflicts with the typed config structs used to configure saluki components and ADP. The crate is a utility crate for config mechanics; -tools disambiguates it from the config translation system being introduced.
saluki-components referenced the old saluki-config name in its antithesis feature list.
0b4737b to
c9aa69b
Compare
|
Closing. I still want to rename it 😅 , but a design change in the works may make it less defensible. |

Summary
saluki-configis a utilities crate for configuration mechanics (providers, loaders,dynamic watchers). A follow-on PR series introduces typed configuration structs, at which
point the name
saluki-configwould conflate with those new crates. This rename preemptsthat ambiguity.
Change Type
How did you test this PR?
This is a mechanical rename with no logic changes.
make fmt && make check-allpassed.References