Skip to content

Commit 2bcfa15

Browse files
committed
fix: remove error & comment cleanup
1 parent c593ce9 commit 2bcfa15

5 files changed

Lines changed: 5 additions & 10 deletions

File tree

rust/operator-binary/src/config/jvm.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ pub enum Error {
3030
MergeJvmArgumentOverrides { source: role_utils::Error },
3131
}
3232

33-
/// Create the NiFi bootstrap.conf
33+
/// Build the merged JVM argument overrides used in the NiFi bootstrap.conf
3434
pub fn build_merged_jvm_config(
3535
merged_config: &NifiConfig,
3636
role: &NifiRoleType,

rust/operator-binary/src/controller.rs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -217,11 +217,6 @@ pub enum Error {
217217
source: stackable_operator::commons::rbac::Error,
218218
},
219219

220-
#[snafu(display("Invalid NiFi Authorization Configuration"))]
221-
InvalidNifiAuthorizationConfig {
222-
source: crate::security::authorization::Error,
223-
},
224-
225220
#[snafu(display("failed to create PodDisruptionBudget"))]
226221
FailedToCreatePdb {
227222
source: crate::operations::pdb::Error,

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,7 @@ impl HasStatusCondition for v1alpha1::NifiCluster {
211211
}
212212

213213
impl v1alpha1::NifiCluster {
214-
/// Metadata about a metastore rolegroup
214+
/// Metadata about a Node rolegroup
215215
pub fn node_rolegroup_ref(&self, group_name: impl Into<String>) -> RoleGroupRef<Self> {
216216
RoleGroupRef {
217217
cluster: ObjectRef::from_obj(self),

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ pub fn build_maybe_reporting_task(
149149
}
150150
}
151151

152-
/// Return the name of the reporting task.
152+
/// Return the name of the reporting task Service.
153153
pub fn build_reporting_task_service_name(nifi_cluster_name: &str) -> String {
154154
format!("{nifi_cluster_name}-{REPORTING_TASK_CONTAINER_NAME}")
155155
}

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -206,8 +206,8 @@ impl NifiAuthenticationConfig {
206206
commands
207207
}
208208

209-
/// Returns
210-
/// - A list of extra commands for the init container
209+
/// Adds the volumes and volume mounts required by the configured authentication
210+
/// method to the pod and the given container builders.
211211
pub fn add_volumes_and_mounts(
212212
&self,
213213
pod_builder: &mut PodBuilder,

0 commit comments

Comments
 (0)