Skip to content

Commit ec25adb

Browse files
committed
CH-109 fix flask-sql, tweak dependencies
1 parent 9b59a1d commit ec25adb

7 files changed

Lines changed: 18 additions & 10 deletions

File tree

applications/common/server/common/__main__.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,15 @@
44
from cloudharness import log
55
from flask_cors import CORS
66
from common.repository.db import open_db
7+
from common.controllers.sentry_controller import global_dsn
8+
9+
710

811
def init_fn(app):
912
log.info("initializing database from app")
1013
cors = CORS(app, resources={r"/api/*": {"origins": "*"}})
11-
open_db(app)
14+
if not global_dsn:
15+
open_db(app)
1216

1317
app = init_flask(init_app_fn=init_fn)
1418

applications/common/server/common/controllers/sentry_controller.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,14 @@
55
from cloudharness.utils.env import get_sentry_service_cluster_address
66
from common.repository.sentry import get_token, get_dsn, SentryProjectNotFound
77

8+
89
try:
910
global_dsn = os.environ.get("SENTRY_DSN", "")
1011
if len(global_dsn) < 1:
1112
global_dsn = None
1213
except:
1314
global_dsn = None
1415

15-
1616
def getdsn(appname): # noqa: E501
1717
"""
1818
Gets the Sentry DSN for a given application or returns the global dsn when set

applications/common/server/common/repository/db.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
from flask_sqlalchemy import SQLAlchemy
2+
from cloudharness import log
23

34
db = None
45

@@ -12,6 +13,9 @@ def get_db():
1213

1314
def open_db(app):
1415
global db
15-
if not db:
16-
db = SQLAlchemy(app)
16+
try:
17+
if not db:
18+
db = SQLAlchemy(app)
19+
except Exception as e:
20+
log.exception("Sentry database cannot be initialized")
1721
return db

applications/common/server/requirements.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,6 @@ connexion[swagger-ui]==2.14.2
22
swagger-ui-bundle >= 0.0.2
33
python_dateutil >= 2.6.0
44
setuptools >= 21.0.0
5-
Flask >= 2.1.0
6-
flask_sqlalchemy==2.4.4
7-
sqlalchemy
5+
Flask<3.0.0
6+
flask_sqlalchemy==3.0.2
7+
sqlalchemy<2.0.0

applications/samples/backend/requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@ connexion[swagger-ui]==2.14.2
22
swagger-ui-bundle >= 0.0.2
33
python_dateutil >= 2.6.0
44
setuptools >= 21.0.0
5-
Flask >= 2.1.0
5+
Flask<3.0.0

applications/volumemanager/server/requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@ connexion[swagger-ui]==2.14.2
22
swagger-ui-bundle >= 0.0.2
33
python_dateutil >= 2.6.0
44
setuptools >= 21.0.0
5-
Flask >= 2.1.0
5+
Flask<3.0.0

infrastructure/common-images/cloudharness-flask/requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
connexion[swagger-ui]==2.14.2
2-
Flask>=2.1.0
2+
Flask<3.0.0
33
Flask-Cors>=4.0.0
44
gunicorn>=21.2.0
55
itsdangerous>=2.1.2

0 commit comments

Comments
 (0)