Skip to content

Commit d06543c

Browse files
committed
cleaned up comments
1 parent a595224 commit d06543c

2 files changed

Lines changed: 2 additions & 6 deletions

File tree

rust/operator-binary/src/airflow_controller.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -494,8 +494,6 @@ pub async fn reconcile_airflow(
494494
)
495495
.await?;
496496

497-
// This secret is created even if spooling is not configured.
498-
// Trino currently requires the secret to be exactly 256 bits long.
499497
create_random_secret(
500498
airflow.shared_jwt_secret_name().as_ref(),
501499
ENV_JWT_SECRET,

rust/operator-binary/src/env_vars.rs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,10 +102,9 @@ pub fn build_airflow_statefulset_envs(
102102
ENV_INTERNAL_SECRET,
103103
),
104104
);
105+
// Replaces AIRFLOW__WEBSERVER__SECRET_KEY >= 3.0.2.
105106
env.insert(
106107
"AIRFLOW__API__SECRET_KEY".into(),
107-
// The secret key is used to run the webserver flask app and also
108-
// used to authorize requests to Celery workers when logs are retrieved.
109108
env_var_from_secret(
110109
"AIRFLOW__API__SECRET_KEY",
111110
&internal_secret_name,
@@ -483,8 +482,7 @@ fn add_version_specific_env_vars(
483482
// This should be random, but must also be consistent across
484483
// api-services and replicas/roles for a given
485484
// cluster, but should also be cluster-specific.
486-
// See issue <https://github.com/stackabletech/airflow-operator/issues/639>:
487-
// it is accessed from a secret to avoid cluster restarts
485+
// It is accessed from a secret to avoid cluster restarts
488486
// being triggered by an operator restart.
489487
let jwt_secret_name = airflow.shared_jwt_secret_name();
490488
env.insert(

0 commit comments

Comments
 (0)