Skip to content

Commit 46bfa1d

Browse files
committed
fix: remove product-config references in comments
1 parent dc220f9 commit 46bfa1d

2 files changed

Lines changed: 4 additions & 11 deletions

File tree

rust/operator-binary/src/controller/build/config_map.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
//! The per-file configs (runtime.properties / security.properties / jvm.config) are rendered here
44
//! from the merged [`DruidRoleGroupConfig`] (config plus the merged config overrides); the
55
//! recommended cluster-level runtime properties and the erased roles needed for `jvm.config` are
6-
//! carried on `ValidatedCluster`. Product-config is no longer involved.
6+
//! carried on `ValidatedCluster`.
77
//!
88
//! Metadata, owner reference and recommended labels are derived entirely from `ValidatedCluster`
99
//! (which carries the validated name/namespace/uid and implements `Resource`).

rust/operator-binary/src/controller/build/properties/runtime_properties.rs

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,7 @@
1-
//! Builder for the static, role-specific `runtime.properties` defaults that were
2-
//! previously injected by product-config's `recommendedValues`.
1+
//! Builder for the static, role-specific `runtime.properties`.
32
//!
4-
//! Only includes a recommended value for a role where the property was
5-
//! `required: true` for that role (verified against
6-
//! `tests/templates/kuttl/smoke/53-assert.yaml.j2`). Dynamic, cluster-derived
7-
//! values (ZooKeeper, extensions, metadata DB, deep storage, TLS, auth, ports,
8-
//! resource-derived sizes) are added by the controller, not here.
3+
//! Dynamic, cluster-derived values (ZooKeeper, extensions, metadata DB, deep storage,
4+
//! TLS, auth, ports, resource-derived sizes) are added by the controller, not here.
95
106
use std::collections::BTreeMap;
117

@@ -25,9 +21,6 @@ const PROMETHEUS_PORT: &str = "druid.emitter.prometheus.port";
2521

2622
/// The recommended cluster-level `runtime.properties` derived from the cluster config (deep
2723
/// storage, OPA authorization and metrics). These are independent of role and role group.
28-
///
29-
/// `opa_authorization_enabled` mirrors `authorization.opa` being configured (equivalently, the
30-
/// OPA connection string having been resolved during dereferencing).
3124
pub fn cluster_runtime_properties(
3225
deep_storage: &DeepStorageSpec,
3326
opa_authorization_enabled: bool,

0 commit comments

Comments
 (0)