Skip to content

Commit 04d0ef3

Browse files
razvanmaltesander
andauthored
Apply suggestions from code review
Co-authored-by: Malte Sander <contact@maltesander.com>
1 parent 47abc7e commit 04d0ef3

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,7 @@ impl v1alpha1::KafkaCluster {
254254
})
255255
}
256256

257-
/// List pod descriptors for a given role and all it's groups.
257+
/// List pod descriptors for a given role and all its rolegroups.
258258
/// If no role is provided, pod descriptors for all roles (and all groups) are listed.
259259
/// We try to predict the pods here rather than looking at the current cluster state in order to
260260
/// avoid instance churn.

rust/operator-binary/src/resource/configmap.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ pub enum Error {
7474
name: String,
7575
},
7676

77-
#[snafu(display("failed to build jaas configuration file for {}", rolegroup))]
77+
#[snafu(display("failed to build jaas configuration file for {rolegroup}"))]
7878
BuildJaasConfig { rolegroup: String },
7979
}
8080

@@ -206,7 +206,7 @@ pub fn build_rolegroup_config_map(
206206
})
207207
}
208208

209-
// Generate the content of both server.properties and controller.properties files.
209+
// Generate the content of both broker.properties and controller.properties files.
210210
fn server_properties_file(
211211
kraft_mode: bool,
212212
role: &str,

0 commit comments

Comments
 (0)