Skip to content

Commit 2eaebde

Browse files
authored
Fix broken o2ims operator Flask setup (#1034)
Flask app fails to run. Fix needed. Lots of other cleanup required. Last change was not even tested. Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
1 parent a29054b commit 2eaebde

6 files changed

Lines changed: 23 additions & 22 deletions

File tree

operators/o2ims-operator/api/app.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
1-
from flask import Flask, request, jsonify
2-
from kubernetes import client, config
3-
from kubernetes.client.rest import ApiException
4-
import os
51
import logging
62
from datetime import datetime
3+
4+
from flask import Flask, request, jsonify
5+
from kubernetes import client, config
6+
77
from controllers.utils import validate_cluster_creation_request
8+
89
app = Flask(__name__)
910

1011
@app.route('/O2ims_infrastructureProvisioning/v1/provisioningRequests ', methods=['POST'])

operators/o2ims-operator/controllers/manager.py

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,18 +14,15 @@
1414
# limitations under the License.
1515
##########################################################################
1616

17-
from utils import LOG_LEVEL, CLUSTER_PROVISIONER, CREATION_TIMEOUT
18-
from provisioning_request_controller import *
19-
from provisioning_request_validation_controller import *
20-
from datetime import datetime
2117
import logging
18+
import threading
19+
2220
import kopf
23-
import os
2421

25-
from flask import Flask
26-
import threading
2722
from api import app # Import the Flask app
28-
23+
from controllers.utils import LOG_LEVEL, CLUSTER_PROVISIONER, CREATION_TIMEOUT
24+
from provisioning_request_controller import *
25+
from provisioning_request_validation_controller import *
2926

3027

3128
# Start Flask in a separate thread

operators/o2ims-operator/controllers/provisioning_request_controller.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,12 @@
1414
# limitations under the License.
1515
##########################################################################
1616

17-
from utils import *
1817
import time
1918
import uuid
19+
from datetime import datetime
20+
21+
from controllers.utils import check_o2ims_provisioning_request, UPSTREAM_PKG_REPO, create_package_variant, \
22+
get_package_variant, get_capi_cluster, TIME_FORMAT
2023

2124

2225
def check_creation_request_status(

operators/o2ims-operator/controllers/utils.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,7 @@
1515
##########################################################################
1616

1717
import os
18-
from datetime import datetime
19-
from dateutil.tz import tzutc
18+
2019
import requests
2120

2221
requests.packages.urllib3.disable_warnings()

operators/o2ims-operator/requirements.txt

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
1-
## requirements.txt
1+
Flask==3.1.2
2+
kopf==1.40.1
3+
kubernetes==35.0.0
4+
python_kubernetes==0.2
5+
Requests==2.32.5
6+
responses==0.25.8
27
jinja2==3.1.6
3-
kopf==1.36.0
4-
requests==2.32.4
5-
python-dateutil==2.8.2
6-
flask==2.3.3
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
responses
2-
pytest
1+
pytest==9.0.2
2+
responses

0 commit comments

Comments
 (0)