Skip to content

Commit b6ec3b2

Browse files
committed
fix: use listener role group placeholder
1 parent e54361e commit b6ec3b2

2 files changed

Lines changed: 8 additions & 6 deletions

File tree

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
//! Builders that turn a `ValidatedCluster` into Kubernetes resource contents.
2-
//!
3-
//! `properties` renders the Trino `.properties` files; `resource` builds the individual
4-
//! Kubernetes resources (ConfigMap, Service, Listener, PDB, …); `graceful_shutdown`
5-
//! contributes graceful-shutdown `config.properties` entries and Pod lifecycle configuration;
6-
//! `ports` derives the client-facing port from the validated TLS configuration.
72
83
pub mod graceful_shutdown;
94
pub mod ports;
105
pub mod properties;
116
pub mod resource;
7+
8+
/// Placeholder role-group name used for the recommended labels of a role's group listener.
9+
///
10+
/// The group listener is owned by the role (not a single role-group), so there is no real
11+
/// role-group to attribute it to.
12+
pub(crate) const PLACEHOLDER_LISTENER_ROLE_GROUP: &str = "none";

rust/operator-binary/src/trino_controller.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,8 @@ pub async fn reconcile_trino(
354354
{
355355
let role_group_listener = build_group_listener(
356356
&validated_cluster,
357-
validated_cluster.recommended_labels(trino_role, "none"),
357+
validated_cluster
358+
.recommended_labels(trino_role, build::PLACEHOLDER_LISTENER_ROLE_GROUP),
358359
listener_class.to_string(),
359360
listener_group_name,
360361
);

0 commit comments

Comments
 (0)