chore: fix some deprecation warnings

bcgov · Jul 17, 2024 · 15dd19d · 15dd19d
1 parent e53affb
commit 15dd19d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/helm/cas-airflow/values.yaml b/helm/cas-airflow/values.yaml
@@ -137,7 +137,7 @@ airflow:
 
   config:
     core:
-      dag_concurrency: 5
+      max_active_tasks_per_dag: 5
       max_active_runs_per_dag: 1
       min_serialized_dag_update_interval: 10
       min_serialized_dag_fetch_interval: 5

diff --git a/helm/cas-airflow/webserver_config.py b/helm/cas-airflow/webserver_config.py
@@ -1,4 +1,3 @@
-from airflow.www.security import AirflowSecurityManager
 from airflow.auth.managers.fab.security_manager.override import FabAirflowSecurityManagerOverride
 import logging
 from typing import Dict, Any, List, Union
@@ -8,7 +7,7 @@
 log = logging.getLogger(__name__)
 log.setLevel(os.getenv("AIRFLOW__LOGGING__FAB_LOGGING_LEVEL", "INFO"))
 
-## Define the security manager class to be used 
+## Define the security manager class to be used
 
 FAB_ADMIN_ROLE = "Admin"
 FAB_VIEWER_ROLE = "Viewer"
@@ -42,7 +41,7 @@ def map_roles(team_list: List[int]) -> List[str]:
 
 # Workaround for Airflow 2.8.1
 # See https://github.com/apache/airflow/issues/36432
-class GithubTeamAuthorizer(AirflowSecurityManager,FabAirflowSecurityManagerOverride):
+class GithubTeamAuthorizer(FabAirflowSecurityManagerOverride):
     # If you ever want to support other providers, see how it is done here:
     # https://github.com/dpgaspar/Flask-AppBuilder/blob/master/flask_appbuilder/security/manager.py#L550
     def get_oauth_user_info(