Skip to content

Commit b3cd85b

Browse files
committed
fix merge conflicts
2 parents f306694 + c7fc7d0 commit b3cd85b

6 files changed

Lines changed: 17 additions & 9 deletions

File tree

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,13 @@ All notable changes to this project will be documented in this file.
1111
### Changed
1212

1313
- Document Helm deployed RBAC permissions and remove unnecessary permissions ([#916]).
14+
- BREAKING: `configOverrides` now only accepts `bootstrap.conf`, `nifi.properties` and `security.properties`.
15+
Previously, arbitrary keys were silently accepted but ignored ([#921]).
16+
- Bump `stackable-operator` to 0.110.1 and `kube` to 3.1.0 ([#921]).
1417

1518
[#903]: https://github.com/stackabletech/nifi-operator/pull/903
1619
[#916]: https://github.com/stackabletech/nifi-operator/pull/916
20+
[#921]: https://github.com/stackabletech/nifi-operator/pull/921
1721

1822
## [26.3.0] - 2026-03-16
1923

Cargo.lock

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

Cargo.nix

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

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

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -196,13 +196,16 @@ pub mod versioned {
196196
#[derive(Clone, Debug, Default, Deserialize, JsonSchema, PartialEq, Serialize)]
197197
#[serde(rename_all = "camelCase")]
198198
pub struct NifiConfigOverrides {
199-
#[serde(rename = "bootstrap.conf")]
199+
#[serde(rename = "bootstrap.conf", skip_serializing_if = "Option::is_none")]
200200
pub bootstrap_conf: Option<KeyValueConfigOverrides>,
201201

202-
#[serde(rename = "nifi.properties")]
202+
#[serde(rename = "nifi.properties", skip_serializing_if = "Option::is_none")]
203203
pub nifi_properties: Option<KeyValueConfigOverrides>,
204204

205-
#[serde(rename = "security.properties")]
205+
#[serde(
206+
rename = "security.properties",
207+
skip_serializing_if = "Option::is_none"
208+
)]
206209
pub security_properties: Option<KeyValueConfigOverrides>,
207210
}
208211
}

rust/operator-binary/src/security/authorization.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -243,6 +243,7 @@ impl ResolvedNifiAuthorizationConfig {
243243
.ephemeral(
244244
SecretOperatorVolumeSourceBuilder::new(
245245
secret_class,
246+
// NiFi needs the public CA certificate to verify the OPA server's cert.
246247
SecretClassVolumeProvisionParts::Public,
247248
)
248249
.build()

rust/operator-binary/src/security/tls.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,10 @@ pub(crate) fn build_tls_volume(
3030
requested_secret_lifetime: &Duration,
3131
listener_scope: Option<&str>,
3232
) -> Result<Volume> {
33-
// NiFi serves its own TLS endpoints, so the Pod needs both the public
34-
// certificate and the private key.
3533
let mut secret_volume_source_builder = SecretOperatorVolumeSourceBuilder::new(
3634
nifi.server_tls_secret_class(),
35+
// NiFi serves its own TLS endpoints, so the Pod needs both the public
36+
// certificate and the private key.
3737
SecretClassVolumeProvisionParts::PublicPrivate,
3838
);
3939

0 commit comments

Comments
 (0)