Skip to content

Commit 29dfcee

Browse files
committed
rename resource name functions
1 parent 335c3b3 commit 29dfcee

5 files changed

Lines changed: 16 additions & 13 deletions

File tree

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ pub fn build_rolegroup_config_map(
4444
let metadata = cluster
4545
.object_meta(
4646
cluster
47-
.resource_names(role_group_name)
47+
.role_group_resource_names(role_group_name)
4848
.role_group_config_map()
4949
.to_string(),
5050
role_group_name,

rust/operator-binary/src/controller/build/resource/daemonset.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -375,7 +375,7 @@ pub fn build_server_rolegroup_daemonset(
375375
VolumeBuilder::new(CONFIG_VOLUME_NAME.as_ref())
376376
.with_config_map(
377377
cluster
378-
.resource_names(role_group_name)
378+
.role_group_resource_names(role_group_name)
379379
.role_group_config_map()
380380
.to_string(),
381381
)
@@ -405,7 +405,7 @@ pub fn build_server_rolegroup_daemonset(
405405
.context(AddVolumeSnafu)?
406406
.service_account_name(
407407
cluster
408-
.rbac_resource_names()
408+
.cluster_resource_names()
409409
.service_account_name()
410410
.to_string(),
411411
)
@@ -423,13 +423,13 @@ pub fn build_server_rolegroup_daemonset(
423423
.with_service_scope(cluster.server_role_service_name())
424424
.with_service_scope(
425425
cluster
426-
.resource_names(role_group_name)
426+
.role_group_resource_names(role_group_name)
427427
.headless_service_name()
428428
.to_string(),
429429
)
430430
.with_service_scope(
431431
cluster
432-
.resource_names(role_group_name)
432+
.role_group_resource_names(role_group_name)
433433
.metrics_service_name()
434434
.to_string(),
435435
)
@@ -572,7 +572,7 @@ pub fn build_server_rolegroup_daemonset(
572572
&container_name(&Container::Vector),
573573
resolved_product_image,
574574
vector_log_config,
575-
&cluster.resource_names(role_group_name),
575+
&cluster.role_group_resource_names(role_group_name),
576576
&CONFIG_VOLUME_NAME,
577577
&LOG_VOLUME_NAME,
578578
EnvVarSet::new(),
@@ -587,7 +587,7 @@ pub fn build_server_rolegroup_daemonset(
587587
let metadata = cluster
588588
.object_meta(
589589
cluster
590-
.resource_names(role_group_name)
590+
.role_group_resource_names(role_group_name)
591591
.daemon_set_name()
592592
.to_string(),
593593
role_group_name,

rust/operator-binary/src/controller/build/resource/rbac.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ stackable_operator::constant!(NONE_ROLE_GROUP_NAME: RoleGroupName = "none");
2020
pub fn build_service_account(cluster: &ValidatedCluster) -> ServiceAccount {
2121
rbac::build_service_account(
2222
cluster,
23-
&cluster.rbac_resource_names(),
23+
&cluster.cluster_resource_names(),
2424
rbac_labels(cluster),
2525
)
2626
}
@@ -30,7 +30,7 @@ pub fn build_service_account(cluster: &ValidatedCluster) -> ServiceAccount {
3030
pub fn build_role_binding(cluster: &ValidatedCluster) -> RoleBinding {
3131
rbac::build_role_binding(
3232
cluster,
33-
&cluster.rbac_resource_names(),
33+
&cluster.cluster_resource_names(),
3434
rbac_labels(cluster),
3535
)
3636
}

rust/operator-binary/src/controller/build/resource/service.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ pub(crate) fn build_rolegroup_headless_service(
5959
let metadata = cluster
6060
.object_meta(
6161
cluster
62-
.resource_names(role_group_name)
62+
.role_group_resource_names(role_group_name)
6363
.headless_service_name()
6464
.to_string(),
6565
role_group_name,
@@ -117,7 +117,7 @@ pub(crate) fn build_rolegroup_metrics_service(
117117
let metadata = cluster
118118
.object_meta(
119119
cluster
120-
.resource_names(role_group_name)
120+
.role_group_resource_names(role_group_name)
121121
.metrics_service_name()
122122
.to_string(),
123123
role_group_name,

rust/operator-binary/src/controller/mod.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,15 +112,18 @@ impl ValidatedCluster {
112112

113113
/// Type-safe names for the per-cluster RBAC resources: the ServiceAccount shared by all
114114
/// Pods, its (namespaced) RoleBinding, and the operator-deployed ClusterRole it binds.
115-
pub fn rbac_resource_names(&self) -> role_utils::ResourceNames {
115+
pub fn cluster_resource_names(&self) -> role_utils::ResourceNames {
116116
role_utils::ResourceNames {
117117
cluster_name: self.name.clone(),
118118
product_name: product_name(),
119119
}
120120
}
121121

122122
/// Type-safe names for the resources of a given role group.
123-
pub(crate) fn resource_names(&self, role_group_name: &RoleGroupName) -> ResourceNames {
123+
pub(crate) fn role_group_resource_names(
124+
&self,
125+
role_group_name: &RoleGroupName,
126+
) -> ResourceNames {
124127
ResourceNames {
125128
cluster_name: self.name.clone(),
126129
role_name: Self::role_name(),

0 commit comments

Comments
 (0)