Skip to content

Commit dc2e15c

Browse files
committed
fix: remove obsolete resolved_product_image parameter from build_discovery_configmap (provided in ValidatedCluster)
1 parent 5209b06 commit dc2e15c

2 files changed

Lines changed: 3 additions & 12 deletions

File tree

rust/operator-binary/src/controller.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -551,7 +551,6 @@ pub async fn reconcile_hive(
551551
for discovery_cm in discovery::build_discovery_configmaps(
552552
&validated_cluster,
553553
HiveRole::MetaStore,
554-
&validated_cluster.image,
555554
None,
556555
listener,
557556
)

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

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
use snafu::{ResultExt, Snafu};
22
use stackable_operator::{
33
builder::{configmap::ConfigMapBuilder, meta::ObjectMetaBuilder},
4-
commons::product_image_selection::ResolvedProductImage,
54
crd::listener::v1alpha1::Listener,
65
k8s_openapi::api::core::v1::ConfigMap,
76
kube::runtime::reflector::ObjectRef,
@@ -46,17 +45,11 @@ fn cluster_object_ref(cluster: &ValidatedCluster) -> ObjectRef<v1alpha1::HiveClu
4645
pub async fn build_discovery_configmaps(
4746
cluster: &ValidatedCluster,
4847
hive_role: HiveRole,
49-
resolved_product_image: &ResolvedProductImage,
5048
chroot: Option<&str>,
5149
listener: Listener,
5250
) -> Result<Vec<ConfigMap>, Error> {
53-
let discovery_configmaps = vec![build_discovery_configmap(
54-
cluster,
55-
hive_role,
56-
resolved_product_image,
57-
chroot,
58-
listener,
59-
)?];
51+
let discovery_configmaps =
52+
vec![build_discovery_configmap(cluster, hive_role, chroot, listener)?];
6053

6154
Ok(discovery_configmaps)
6255
}
@@ -68,7 +61,6 @@ pub async fn build_discovery_configmaps(
6861
fn build_discovery_configmap(
6962
cluster: &ValidatedCluster,
7063
hive_role: HiveRole,
71-
resolved_product_image: &ResolvedProductImage,
7264
chroot: Option<&str>,
7365
listener: Listener,
7466
) -> Result<ConfigMap, Error> {
@@ -83,7 +75,7 @@ fn build_discovery_configmap(
8375
})?
8476
.with_recommended_labels(&build_recommended_labels(
8577
cluster,
86-
&resolved_product_image.app_version_label_value,
78+
&cluster.image.app_version_label_value,
8779
&hive_role.to_string(),
8880
"discovery",
8981
))

0 commit comments

Comments
 (0)