Skip to content

Commit c941688

Browse files
committed
refactor: switch to v2 Clusterresources
1 parent 7ac6820 commit c941688

2 files changed

Lines changed: 15 additions & 15 deletions

File tree

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,8 @@ pub struct ValidatedCluster {
6161
metadata: ObjectMeta,
6262
/// The logical (and Kubernetes object) name of the cluster.
6363
pub name: ClusterName,
64+
/// The namespace the cluster lives in.
65+
pub namespace: NamespaceName,
6466
/// The UID of the `HbaseCluster` object, used to build owner references.
6567
pub uid: Uid,
6668
pub image: ResolvedProductImage,
@@ -96,6 +98,7 @@ impl ValidatedCluster {
9698
..ObjectMeta::default()
9799
},
98100
name,
101+
namespace,
99102
uid,
100103
image,
101104
product_version,

rust/operator-binary/src/hbase_controller.rs

Lines changed: 12 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,9 @@ use std::sync::Arc;
1010
use snafu::{ResultExt, Snafu};
1111
use stackable_operator::{
1212
cli::OperatorEnvironmentOptions,
13-
cluster_resources::{ClusterResourceApplyStrategy, ClusterResources},
13+
cluster_resources::ClusterResourceApplyStrategy,
1414
commons::rbac::build_rbac_resources,
1515
kube::{
16-
Resource,
1716
core::{DeserializeGuard, error_boundary},
1817
runtime::controller::Action,
1918
},
@@ -24,19 +23,21 @@ use stackable_operator::{
2423
compute_conditions, operations::ClusterOperationsConditionBuilder,
2524
statefulset::StatefulSetConditionBuilder,
2625
},
26+
v2::cluster_resources::cluster_resources_new,
2727
};
2828
use strum::{EnumDiscriminants, IntoStaticStr};
2929

3030
use crate::{
3131
controller::{
32-
HBASE_CONTROLLER_NAME, RoleGroupName,
32+
RoleGroupName,
3333
build::resource::{
3434
config_map::build_rolegroup_config_map,
3535
discovery::build_discovery_config_map,
3636
pdb::build_pdb,
3737
service::{build_rolegroup_metrics_service, build_rolegroup_service},
3838
statefulset::build_rolegroup_statefulset,
3939
},
40+
controller_name, operator_name, product_name,
4041
},
4142
crd::{APP_NAME, HbaseClusterStatus, OPERATOR_NAME, v1alpha1},
4243
};
@@ -49,11 +50,6 @@ pub struct Ctx {
4950
#[derive(Snafu, Debug, EnumDiscriminants)]
5051
#[strum_discriminants(derive(IntoStaticStr))]
5152
pub enum Error {
52-
#[snafu(display("failed to create cluster resources"))]
53-
CreateClusterResources {
54-
source: stackable_operator::cluster_resources::Error,
55-
},
56-
5753
#[snafu(display("failed to delete orphaned resources"))]
5854
DeleteOrphanedResources {
5955
source: stackable_operator::cluster_resources::Error,
@@ -176,15 +172,16 @@ pub async fn reconcile_hbase(
176172
)
177173
.context(ValidateSnafu)?;
178174

179-
let mut cluster_resources = ClusterResources::new(
180-
APP_NAME,
181-
OPERATOR_NAME,
182-
HBASE_CONTROLLER_NAME,
183-
&hbase.object_ref(&()),
175+
let mut cluster_resources = cluster_resources_new(
176+
&product_name(),
177+
&operator_name(),
178+
&controller_name(),
179+
&validated_cluster.name,
180+
&validated_cluster.namespace,
181+
&validated_cluster.uid,
184182
ClusterResourceApplyStrategy::from(&hbase.spec.cluster_operation),
185183
&hbase.spec.object_overrides,
186-
)
187-
.context(CreateClusterResourcesSnafu)?;
184+
);
188185

189186
let (rbac_sa, rbac_rolebinding) = build_rbac_resources(
190187
hbase,

0 commit comments

Comments
 (0)