Skip to content

Commit bf406c9

Browse files
committed
bump to op-rs 0.80.0
1 parent 220cda7 commit bf406c9

File tree

6 files changed

+111
-58
lines changed

6 files changed

+111
-58
lines changed

Cargo.lock

Lines changed: 77 additions & 41 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ serde = { version = "1.0", features = ["derive"] }
2121
serde_json = "1.0"
2222
serde_yaml = "0.9"
2323
snafu = "0.8"
24-
stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.79.0" }
24+
stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.80.0" }
2525
product-config = { git = "https://github.com/stackabletech/product-config.git", tag = "0.7.0" }
2626
strum = { version = "0.26", features = ["derive"] }
2727
tokio = { version = "1.40", features = ["full"] }

rust/operator-binary/src/discovery.rs

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ use stackable_operator::{
88
commons::product_image_selection::ResolvedProductImage,
99
k8s_openapi::api::core::v1::ConfigMap,
1010
kube::runtime::reflector::ObjectRef,
11+
utils::cluster_info::KubernetesClusterInfo,
1112
};
1213

1314
use crate::{
@@ -43,12 +44,15 @@ pub enum Error {
4344
/// Creates a discovery config map containing the `hbase-site.xml` for clients.
4445
pub fn build_discovery_configmap(
4546
hbase: &HbaseCluster,
47+
cluster_info: &KubernetesClusterInfo,
4648
zookeeper_connection_information: &ZookeeperConnectionInformation,
4749
resolved_product_image: &ResolvedProductImage,
4850
) -> Result<ConfigMap> {
4951
let mut hbase_site = zookeeper_connection_information.as_hbase_settings();
50-
hbase_site
51-
.extend(kerberos_discovery_config_properties(hbase).context(AddKerberosDiscoverySnafu)?);
52+
hbase_site.extend(
53+
kerberos_discovery_config_properties(hbase, cluster_info)
54+
.context(AddKerberosDiscoverySnafu)?,
55+
);
5256

5357
ConfigMapBuilder::new()
5458
.metadata(

rust/operator-binary/src/hbase_controller.rs

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ use stackable_operator::{
6060
statefulset::StatefulSetConditionBuilder,
6161
},
6262
time::Duration,
63-
utils::COMMON_BASH_TRAP_FUNCTIONS,
63+
utils::{cluster_info::KubernetesClusterInfo, COMMON_BASH_TRAP_FUNCTIONS},
6464
};
6565
use strum::{EnumDiscriminants, IntoStaticStr, ParseError};
6666

@@ -374,6 +374,7 @@ pub async fn reconcile_hbase(hbase: Arc<HbaseCluster>, ctx: Arc<Ctx>) -> Result<
374374
// discovery config map
375375
let discovery_cm = build_discovery_configmap(
376376
&hbase,
377+
&client.kubernetes_cluster_info,
377378
&zookeeper_connection_information,
378379
&resolved_product_image,
379380
)
@@ -421,6 +422,7 @@ pub async fn reconcile_hbase(hbase: Arc<HbaseCluster>, ctx: Arc<Ctx>) -> Result<
421422
build_rolegroup_service(&hbase, &hbase_role, &rolegroup, &resolved_product_image)?;
422423
let rg_configmap = build_rolegroup_config_map(
423424
&hbase,
425+
&client.kubernetes_cluster_info,
424426
&rolegroup,
425427
rolegroup_config,
426428
&zookeeper_connection_information,
@@ -549,6 +551,7 @@ pub fn build_region_server_role_service(
549551
#[allow(clippy::too_many_arguments)]
550552
fn build_rolegroup_config_map(
551553
hbase: &HbaseCluster,
554+
cluster_info: &KubernetesClusterInfo,
552555
rolegroup: &RoleGroupRef<HbaseCluster>,
553556
rolegroup_config: &HashMap<PropertyNameKind, BTreeMap<String, String>>,
554557
zookeeper_connection_information: &ZookeeperConnectionInformation,
@@ -572,8 +575,10 @@ fn build_rolegroup_config_map(
572575
PropertyNameKind::File(file_name) if file_name == HBASE_SITE_XML => {
573576
let mut hbase_site_config = BTreeMap::new();
574577
hbase_site_config.extend(zookeeper_connection_information.as_hbase_settings());
575-
hbase_site_config
576-
.extend(kerberos_config_properties(hbase).context(AddKerberosConfigSnafu)?);
578+
hbase_site_config.extend(
579+
kerberos_config_properties(hbase, cluster_info)
580+
.context(AddKerberosConfigSnafu)?,
581+
);
577582
hbase_site_config
578583
.extend(hbase_opa_config.map_or(vec![], |config| config.hbase_site_config()));
579584

0 commit comments

Comments
 (0)