Skip to content

Commit 63744a2

Browse files
adwk67claude
andcommitted
refactor: align error variant naming with airflow and hbase operators
Rename FailedToResolveResourceConfig to FailedToResolveConfig and fix OPA error display string to match the convention used across all three dereference/validate extraction PRs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 6da580c commit 63744a2

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ pub enum Error {
2828
source: stackable_operator::database_connections::Error,
2929
},
3030

31-
#[snafu(display("invalid OpaConfig"))]
31+
#[snafu(display("invalid OPA configuration"))]
3232
InvalidOpaConfig {
3333
source: stackable_operator::commons::opa::Error,
3434
},

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ pub enum Error {
3232
source: stackable_operator::product_config_utils::Error,
3333
},
3434

35-
#[snafu(display("failed to resolve and merge resource config for role and role group"))]
36-
FailedToResolveResourceConfig { source: crate::crd::Error },
35+
#[snafu(display("failed to resolve and merge config for role and role group"))]
36+
FailedToResolveConfig { source: crate::crd::Error },
3737
}
3838

3939
/// Per-role configuration extracted during validation.
@@ -123,7 +123,7 @@ pub fn validate_cluster(
123123

124124
let merged_config = hive
125125
.merged_config(&hive_role, &rolegroup)
126-
.context(FailedToResolveResourceConfigSnafu)?;
126+
.context(FailedToResolveConfigSnafu)?;
127127

128128
group_configs.insert(
129129
rolegroup_name.clone(),

0 commit comments

Comments
 (0)