File tree Expand file tree Collapse file tree 2 files changed +9
-2
lines changed
Expand file tree Collapse file tree 2 files changed +9
-2
lines changed Original file line number Diff line number Diff line change 44
55### Fixed
66
7+ - Don't panic on invalid authorization config. Previously, a missing OPA ConfigMap would crash the operator ([ #667 ] ).
78- getting_started: Add a 120 second timeout before trying to enable the DAG ([ #665 ] ).
89
910[ #665 ] : https://github.com/stackabletech/airflow-operator/pull/665
11+ [ #667 ] : https://github.com/stackabletech/airflow-operator/pull/667
1012
1113## [ 25.7.0] - 2025-07-23
1214
Original file line number Diff line number Diff line change @@ -346,6 +346,11 @@ pub enum Error {
346346
347347 #[ snafu( display( "failed to configure service" ) ) ]
348348 ServiceConfiguration { source : crate :: service:: Error } ,
349+
350+ #[ snafu( display( "invalid authorization config" ) ) ]
351+ InvalidAuthorizationConfig {
352+ source : stackable_operator:: commons:: opa:: Error ,
353+ } ,
349354}
350355
351356type Result < T , E = Error > = std:: result:: Result < T , E > ;
@@ -390,7 +395,7 @@ pub async fn reconcile_airflow(
390395 & airflow. spec . cluster_config . authorization ,
391396 )
392397 . await
393- . unwrap ( ) ;
398+ . context ( InvalidAuthorizationConfigSnafu ) ? ;
394399
395400 let mut roles = HashMap :: new ( ) ;
396401
@@ -918,7 +923,7 @@ fn build_server_rolegroup_statefulset(
918923 "kubectl.kubernetes.io/default-container" ,
919924 format ! ( "{}" , Container :: Airflow ) ,
920925 ) )
921- . unwrap ( ) ,
926+ . expect ( "static annotation is always valid" ) ,
922927 )
923928 . build ( ) ;
924929
You can’t perform that action at this time.
0 commit comments