Skip to content

Commit 937faeb

Browse files
committed
Cleanup outdated comments and code
1 parent d20ea74 commit 937faeb

2 files changed

Lines changed: 2 additions & 12 deletions

File tree

rust/operator-binary/src/airflow_controller.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1156,7 +1156,7 @@ fn build_server_rolegroup_statefulset(
11561156
match_labels: Some(statefulset_match_labels.into()),
11571157
..LabelSelector::default()
11581158
},
1159-
service_name: stateful_set_service_name(airflow_role, rolegroup_ref),
1159+
service_name: stateful_set_service_name(rolegroup_ref),
11601160
template: pod_template,
11611161
volume_claim_templates: pvcs,
11621162
..StatefulSetSpec::default()

rust/operator-binary/src/service.rs

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,7 @@ use stackable_operator::{
88
role_utils::RoleGroupRef,
99
};
1010

11-
use crate::crd::{
12-
AirflowRole, HTTP_PORT, HTTP_PORT_NAME, METRICS_PORT, METRICS_PORT_NAME, v1alpha1,
13-
};
11+
use crate::crd::{HTTP_PORT, HTTP_PORT_NAME, METRICS_PORT, METRICS_PORT_NAME, v1alpha1};
1412

1513
pub const METRICS_SERVICE_SUFFIX: &str = "metrics";
1614
pub const HEADLESS_SERVICE_SUFFIX: &str = "headless";
@@ -111,16 +109,8 @@ pub fn build_rolegroup_metrics_service(
111109
}
112110

113111
pub fn stateful_set_service_name(
114-
_airflow_role: &AirflowRole,
115112
rolegroup_ref: &RoleGroupRef<v1alpha1::AirflowCluster>,
116113
) -> Option<String> {
117-
//match airflow_role {
118-
//AirflowRole::Webserver => Some(rolegroup_headless_service_name(
119-
//&rolegroup_ref.object_name(),
120-
//)),
121-
122-
//AirflowRole::Scheduler | AirflowRole::Worker => None,
123-
//}
124114
Some(rolegroup_headless_service_name(
125115
&rolegroup_ref.object_name(),
126116
))

0 commit comments

Comments
 (0)