Skip to content

Commit 8706a3e

Browse files
committed
fixed podref name for workers
1 parent 5e7b41b commit 8706a3e

2 files changed

Lines changed: 4 additions & 18 deletions

File tree

rust/operator-binary/src/airflow_controller.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -589,7 +589,7 @@ pub async fn reconcile_airflow(
589589
}
590590
}
591591

592-
tracing::info!(
592+
tracing::debug!(
593593
"Listener references prepared for the ConfigMap {:#?}",
594594
listener_refs
595595
);

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

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -440,7 +440,7 @@ impl v1alpha1::AirflowCluster {
440440
.unwrap_or_default();
441441
listener_class_role.merge(&listener_class_default);
442442
listener_class_rolegroup.merge(&listener_class_role);
443-
tracing::info!("Merged listener-class: {:?}", listener_class_rolegroup);
443+
tracing::debug!("Merged listener-class: {:?}", listener_class_rolegroup);
444444
Ok(listener_class_rolegroup)
445445
}
446446

@@ -535,10 +535,7 @@ impl v1alpha1::AirflowCluster {
535535
})
536536
.map(|(rolegroup_name, role_group)| {
537537
(
538-
self.rolegroup_ref(
539-
AirflowRole::Scheduler.to_string(),
540-
rolegroup_name,
541-
),
538+
self.rolegroup_ref(AirflowRole::Worker.to_string(), rolegroup_name),
542539
role_group.replicas.unwrap_or_default(),
543540
)
544541
})
@@ -556,22 +553,11 @@ impl v1alpha1::AirflowCluster {
556553
rolegroup_name: &&String,
557554
) -> bool {
558555
if let Ok(Some(listener_class)) = self.merged_listener_class(role, rolegroup_name) {
559-
tracing::info!(
560-
"Merged listener-class for role is discoverable?: {}/{}/{}",
561-
role,
562-
listener_class,
563-
listener_class.discoverable()
564-
);
565556
listener_class.discoverable()
566557
} else {
567558
// merged_listener_class returns an error is one of the roles was not found:
568559
// all roles are mandatory for airflow to work, but a missing role will by
569560
// definition not have a listener class
570-
tracing::info!(
571-
"Merged listener-class for role is NOT discoverable?: {}/{}",
572-
role,
573-
rolegroup_name
574-
);
575561
false
576562
}
577563
}
@@ -605,7 +591,7 @@ impl v1alpha1::AirflowCluster {
605591
) -> Result<Vec<PodRef>, Error> {
606592
let pod_refs = self.pod_refs(role)?;
607593

608-
tracing::info!("pod_refs for role {role}: {:#?}", pod_refs);
594+
tracing::debug!("Pod references for role {role}: {:#?}", pod_refs);
609595
get_persisted_listener_podrefs(client, pod_refs, LISTENER_VOLUME_NAME)
610596
.await
611597
.context(ListenerPodRefSnafu)

0 commit comments

Comments
 (0)