Skip to content

Commit fa9c56d

Browse files
committed
refactor: use v2 cluster resources
1 parent 5210423 commit fa9c56d

2 files changed

Lines changed: 15 additions & 15 deletions

File tree

rust/operator-binary/src/controller.rs

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,9 @@ use const_format::concatcp;
77
use snafu::{ResultExt, Snafu};
88
use stackable_operator::{
99
cli::OperatorEnvironmentOptions,
10-
cluster_resources::{ClusterResourceApplyStrategy, ClusterResources},
10+
cluster_resources::ClusterResourceApplyStrategy,
1111
commons::rbac::build_rbac_resources,
1212
kube::{
13-
Resource,
1413
core::{DeserializeGuard, error_boundary},
1514
runtime::controller::Action,
1615
},
@@ -21,7 +20,10 @@ use stackable_operator::{
2120
compute_conditions, operations::ClusterOperationsConditionBuilder,
2221
statefulset::StatefulSetConditionBuilder,
2322
},
24-
v2::types::operator::{ControllerName, OperatorName, ProductName},
23+
v2::{
24+
cluster_resources::cluster_resources_new,
25+
types::operator::{ControllerName, OperatorName, ProductName},
26+
},
2527
};
2628
use strum::{EnumDiscriminants, IntoStaticStr};
2729

@@ -111,11 +113,6 @@ pub enum Error {
111113
source: stackable_operator::client::Error,
112114
},
113115

114-
#[snafu(display("failed to create cluster resources"))]
115-
CreateClusterResources {
116-
source: stackable_operator::cluster_resources::Error,
117-
},
118-
119116
#[snafu(display("failed to delete orphaned resources"))]
120117
DeleteOrphanedResources {
121118
source: stackable_operator::cluster_resources::Error,
@@ -209,15 +206,16 @@ pub async fn reconcile_druid(
209206
validate::validate(druid, &dereferenced_objects, &ctx.operator_environment)
210207
.context(ValidateClusterSnafu)?;
211208

212-
let mut cluster_resources = ClusterResources::new(
213-
APP_NAME,
214-
OPERATOR_NAME,
215-
DRUID_CONTROLLER_NAME,
216-
&druid.object_ref(&()),
209+
let mut cluster_resources = cluster_resources_new(
210+
&product_name(),
211+
&operator_name(),
212+
&controller_name(),
213+
&validated_cluster.name,
214+
&validated_cluster.namespace,
215+
&validated_cluster.uid,
217216
ClusterResourceApplyStrategy::from(&druid.spec.cluster_operation),
218217
&druid.spec.object_overrides,
219-
)
220-
.context(CreateClusterResourcesSnafu)?;
218+
);
221219

222220
let (rbac_sa, rbac_rolebinding) = build_rbac_resources(
223221
druid,

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,7 @@ pub struct ValidatedCluster {
103103
/// [`Resource`] and be passed directly to the metadata/owner-reference builders.
104104
metadata: ObjectMeta,
105105
pub name: ClusterName,
106+
pub namespace: NamespaceName,
106107
pub uid: Uid,
107108
pub image: ResolvedProductImage,
108109
pub cluster_config: ValidatedClusterConfig,
@@ -128,6 +129,7 @@ impl ValidatedCluster {
128129
Self {
129130
metadata,
130131
name,
132+
namespace,
131133
uid,
132134
image,
133135
cluster_config,

0 commit comments

Comments
 (0)