Skip to content

Commit e7c9462

Browse files
committed
sql工单执行进度迁移至api
1 parent a99f0f1 commit e7c9462

File tree

6 files changed

+9
-33
lines changed

6 files changed

+9
-33
lines changed

sql/sql_workflow.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -139,21 +139,6 @@ def cancel(request):
139139
return HttpResponseRedirect(reverse("sql:detail", args=(workflow_id,)))
140140

141141

142-
def get_workflow_status(request):
143-
"""
144-
获取某个工单的当前状态
145-
"""
146-
workflow_id = request.POST["workflow_id"]
147-
if workflow_id == "" or workflow_id is None:
148-
context = {"status": -1, "msg": "workflow_id参数为空.", "data": ""}
149-
return HttpResponse(json.dumps(context), content_type="application/json")
150-
151-
workflow_id = int(workflow_id)
152-
workflow_detail = get_object_or_404(SqlWorkflow, pk=workflow_id)
153-
result = {"status": workflow_detail.status, "msg": "", "data": ""}
154-
return JsonResponse(result)
155-
156-
157142
def osc_control(request):
158143
"""用于mysql控制osc执行"""
159144
workflow_id = request.POST.get("workflow_id")

sql/templates/detail.html

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -527,7 +527,7 @@ <h4 class="modal-title text-danger">修改可执行时间</h4>
527527
}
528528

529529
// 工单操作
530-
function sqlWorkflowAction(http_method ,action, data, btnAttrId) {
530+
function sqlWorkflowAction(http_method, action, data, btnAttrId) {
531531
$.ajax({
532532
type: http_method,
533533
url: "/api/v2/workflow/sql/{{ workflow_detail.id }}/" + action + "/",
@@ -942,13 +942,11 @@ <h4 class="modal-title text-danger">修改可执行时间</h4>
942942
if (workflow_id > 0) {
943943
// console.log('get workflow status');
944944
$.ajax({
945-
type: "post",
945+
type: "get",
946946
async: false,
947-
url: "/getWorkflowStatus/",
947+
contentType: "application/json",
948+
url: "/api/v2/workflow/sql/" + workflow_id + "/progress/",
948949
dataType: "json",
949-
data: {
950-
workflow_id: workflow_id
951-
},
952950
complete: function () {
953951
if (wfStatus !== -1 && wfStatus !== "workflow_executing") {
954952
window.location.reload(true);

sql/tests.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -791,14 +791,6 @@ def tearDown(self):
791791
self.resource_group1.delete()
792792
SysConfig().purge()
793793

794-
def testWorkflowStatus(self):
795-
"""测试获取工单状态"""
796-
c = Client(header={})
797-
c.force_login(self.u1)
798-
r = c.post("/getWorkflowStatus/", {"workflow_id": self.wf1.id})
799-
r_json = r.json()
800-
self.assertEqual(r_json["status"], "workflow_finish")
801-
802794
@patch("sql.utils.workflow_audit.Audit.logs")
803795
@patch("sql.utils.workflow_audit.Audit.detail_by_workflow_id")
804796
@patch("sql.utils.workflow_audit.Audit.review_info")
@@ -873,7 +865,6 @@ def testWorkflowPassedView(self, mock_operate, _):
873865
self.wf2.refresh_from_db()
874866
self.assertEqual(self.wf2.status, "workflow_review_pass")
875867

876-
877868
@patch("sql.sql_workflow.Audit.add_log")
878869
@patch("sql.notify.auto_notify")
879870
@patch("sql.utils.workflow_audit.AuditV2.operate")

sql/urls.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,6 @@
7575
path("audit_sqlquery/", views.audit_sqlquery),
7676
path("audit_sqlworkflow/", views.audit_sqlworkflow),
7777
path("authenticate/", auth.authenticate_entry),
78-
path("getWorkflowStatus/", sql_workflow.get_workflow_status),
7978
path("del_sqlcronjob/", tasks.del_schedule),
8079
path("inception/osc_control/", sql_workflow.osc_control),
8180
path("sql_analyze/generate/", sql_analyze.generate),

sql_api/permissions/sql_workflow.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ def has_permission(self, request, view):
3232
[
3333
request.user.has_perm("sql.menu_sqlworkflow"),
3434
request.user.has_perm("sql.audit_user"),
35-
request.user.has_perm("sql.sql_submit")
35+
request.user.has_perm("sql.sql_submit"),
3636
]
3737
)
3838

sql_api/urls.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,10 @@
5353
path("v1/instance/tunnel/", api_instance.TunnelList.as_view()),
5454
path("v1/instance/rds/", api_instance.AliyunRdsList.as_view()),
5555
path("v1/workflow/", api_workflow.WorkflowList.as_view()),
56-
path("v1/workflow/sqlcheck/", api_sql_workflow.SqlWorkflowView.as_view({'post': 'check'})),
56+
path(
57+
"v1/workflow/sqlcheck/",
58+
api_sql_workflow.SqlWorkflowView.as_view({"post": "check"}),
59+
),
5760
path("v1/workflow/audit/", api_workflow.AuditWorkflow.as_view()),
5861
path("v1/workflow/auditlist/", api_workflow.WorkflowAuditList.as_view()),
5962
path("v1/workflow/execute/", api_workflow.ExecuteWorkflow.as_view()),

0 commit comments

Comments
 (0)