Skip to content

Commit bac8ff4

Browse files
authored
fix: Only take the pbject_overrides by reference in ClusterResources (#1132)
* fix: Only take the pbject_overrides by reference in ClusterResources * release 0.101.1
1 parent 5c3547b commit bac8ff4

File tree

4 files changed

+15
-7
lines changed

4 files changed

+15
-7
lines changed

Cargo.lock

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

crates/stackable-operator/CHANGELOG.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,14 @@ All notable changes to this project will be documented in this file.
44

55
## [Unreleased]
66

7+
## [0.101.1] - 2025-12-23
8+
9+
### Fixed
10+
11+
- BREAKING: Only take `object_overrides` by reference in `ClusterResources::new` ([#1132]).
12+
13+
[#1132]: https://github.com/stackabletech/operator-rs/pull/1132
14+
715
## [0.101.0] - 2025-12-23
816

917
### Added

crates/stackable-operator/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
[package]
22
name = "stackable-operator"
33
description = "Stackable Operator Framework"
4-
version = "0.101.0"
4+
version = "0.101.1"
55
authors.workspace = true
66
license.workspace = true
77
edition.workspace = true

crates/stackable-operator/src/cluster_resources.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -381,7 +381,7 @@ impl ClusterResource for Deployment {
381381
/// CONTROLLER_NAME,
382382
/// &app.object_ref(&()),
383383
/// ClusterResourceApplyStrategy::Default,
384-
/// app.spec.object_overrides.clone(),
384+
/// &app.spec.object_overrides,
385385
/// )
386386
/// .map_err(|source| Error::CreateClusterResources { source })?;
387387
///
@@ -428,7 +428,7 @@ impl ClusterResource for Deployment {
428428
/// }
429429
/// ```
430430
#[derive(Debug, PartialEq)]
431-
pub struct ClusterResources {
431+
pub struct ClusterResources<'a> {
432432
/// The namespace of the cluster
433433
namespace: String,
434434

@@ -458,10 +458,10 @@ pub struct ClusterResources {
458458
apply_strategy: ClusterResourceApplyStrategy,
459459

460460
/// Arbitrary Kubernetes object overrides specified by the user via the CRD.
461-
object_overrides: ObjectOverrides,
461+
object_overrides: &'a ObjectOverrides,
462462
}
463463

464-
impl ClusterResources {
464+
impl<'a> ClusterResources<'a> {
465465
/// Constructs new `ClusterResources`.
466466
///
467467
/// # Arguments
@@ -487,7 +487,7 @@ impl ClusterResources {
487487
controller_name: &str,
488488
cluster: &ObjectReference,
489489
apply_strategy: ClusterResourceApplyStrategy,
490-
object_overrides: ObjectOverrides,
490+
object_overrides: &'a ObjectOverrides,
491491
) -> Result<Self> {
492492
let namespace = cluster
493493
.namespace

0 commit comments

Comments
 (0)