Apply suggestions from code review

Co-authored-by: Sebastian Bernauer <[email protected]>

Author: razvan

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

@@ -462,8 +462,6 @@ mod tests {
                     reference: test-s3-connection
                   maxErrorRetries: 5
                   uploadPartSize: 10Mi
-                  iamRole: arn:aws:iam::123456789012:role/TrinoRole
-                  externalId: external-id-123
         "#});
 
         let fte_config =

rust/operator-binary/src/controller.rs

@@ -1550,6 +1550,8 @@ fn shared_spooling_secret_name(trino: &v1alpha1::TrinoCluster) -> String {
     format!("{}-spooling-secret", trino.name_any())
 }
 
+// TODO: Maybe switch to something non-openssl.
+// See https://github.com/stackabletech/airflow-operator/pull/686#discussion_r2348354468 (which is currently under discussion)
 fn get_random_base64(byte_size: usize) -> String {
     let mut buf: Vec<u8> = vec![0; byte_size];
     openssl::rand::rand_bytes(&mut buf).unwrap();