Skip to content

Commit e0584d6

Browse files
magichairjibsen-vh
andauthored
fix(circleci): CircleCI Workflow nil check for CreatedDate (#8310)
* fix(circleci): Add CircleCI workflow nil check for CreatedDate Adds a nil check during `convertWorkflows` task when CircleCI workflows API returns a weird malformed pipeline data with no created_date. Previously, the entire task fails and panics when trying to get convert a nil in ToTime util. Fixes Issue #8309 * Update to fix compile problems * Tweak the _tool csv to allow for the NULL value entries --------- Co-authored-by: John Ibsen <john@videa.ai>
1 parent 8f654df commit e0584d6

File tree

3 files changed

+20
-0
lines changed

3 files changed

+20
-0
lines changed

backend/plugins/circleci/e2e/raw_tables/_raw_circleci_api_workflows.csv

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,3 +98,14 @@ id,params,data,url,input,created_at
9898
""created_at"" : ""2023-03-25T18:06:13Z"",
9999
""stopped_at"" : ""2023-03-25T18:06:28Z""
100100
}",https://circleci.com/api/v2/pipeline/afe0cabe-e7ee-4eb7-bf13-bb6170d139f0/workflow,"{""id"": ""afe0cabe-e7ee-4eb7-bf13-bb6170d139f0"", ""vcs"": {""tag"": """", ""branch"": """", ""commit"": {""body"": """", ""subject"": """"}, ""revision"": """", ""review_id"": """", ""review_url"": """", ""provider_name"": """", ""origin_repository_url"": """", ""target_repository_url"": """"}, ""state"": """", ""errors"": null, ""number"": 0, ""trigger"": {""type"": """", ""actor"": {""login"": """", ""avatar_url"": """"}, ""received_at"": """"}, ""createdAt"": ""0001-01-01T00:00:00Z"", ""updatedAt"": ""0001-01-01T00:00:00Z"", ""created_at"": null, ""updated_at"": null, ""ConnectionId"": 0, ""_raw_data_id"": 0, ""project_slug"": """", ""_raw_data_table"": """", ""_raw_data_params"": """", ""_raw_data_remark"": """", ""trigger_parameters"": null}",2023-03-28 15:39:58.245
101+
28,"{""ConnectionId"":1,""ProjectSlug"":""github/coldgust/coldgust.github.io""}","{
102+
""pipeline_id"" : ""f5a2a6e8-4430-4935-8bec-77ca2b26c61d"",
103+
""id"" : ""b3b77371-e27a-4e93-a554-c728f757cd24"",
104+
""name"" : ""workflow"",
105+
""project_slug"" : ""gh/coldgust/coldgust.github.io"",
106+
""status"" : ""failed"",
107+
""started_by"" : """",
108+
""pipeline_number"" : 16,
109+
""created_at"" : """",
110+
""stopped_at"" : """"
111+
}",https://circleci.com/api/v2/pipeline/f5a2a6e8-4430-4935-8bec-77ca2b26c61d/workflow,"{""id"": ""f5a2a6e8-4430-4935-8bec-77ca2b26c61d"", ""vcs"": {""tag"": """", ""branch"": """", ""commit"": {""body"": """", ""subject"": """"}, ""revision"": """", ""review_id"": """", ""review_url"": """", ""provider_name"": """", ""origin_repository_url"": """", ""target_repository_url"": """"}, ""state"": """", ""errors"": null, ""number"": 0, ""trigger"": {""type"": """", ""actor"": {""login"": """", ""avatar_url"": """"}, ""received_at"": """"}, ""createdAt"": """", ""updatedAt"": ""0001-01-01T00:00:00Z"", ""created_at"": null, ""updated_at"": null, ""ConnectionId"": 0, ""_raw_data_id"": 0, ""project_slug"": """", ""_raw_data_table"": """", ""_raw_data_params"": """", ""_raw_data_remark"": """", ""trigger_parameters"": null}",2023-03-28 15:45:58.245

backend/plugins/circleci/e2e/snapshot_tables/_tool_circleci_workflows.csv

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ connection_id,id,project_slug,pipeline_id,canceled_by,name,errored_by,tag,status
44
1,89054eb2-8e85-4f5c-9a93-66d753a0e970,github/coldgust/coldgust.github.io,625ca634-68fe-4515-91f0-7ba8af51dc99,,say-hello-workflow,,,success,1c762fc2-b0fb-4fe2-97d2-5e54ddd1eba7,3,2023-03-25T17:39:23.000+00:00,2023-03-25T17:39:28.000+00:00,5
55
1,8971a56b-5547-4824-94dd-07bb467524c5,github/coldgust/coldgust.github.io,87aad008-1ad5-486a-8174-fdeed846561a,,say-hello-workflow,,,success,1c762fc2-b0fb-4fe2-97d2-5e54ddd1eba7,2,2023-03-25T17:12:18.000+00:00,2023-03-25T17:12:23.000+00:00,5
66
1,8fe60291-68f7-40e2-acec-d99bff4da713,github/coldgust/coldgust.github.io,afef32b3-5ffe-48d2-8d9e-46dcedd82554,,say-hello-workflow,,,success,1c762fc2-b0fb-4fe2-97d2-5e54ddd1eba7,1,2023-03-25T17:12:18.000+00:00,2023-03-25T17:12:22.000+00:00,4
7+
1,b3b77371-e27a-4e93-a554-c728f757cd24,github/coldgust/coldgust.github.io,f5a2a6e8-4430-4935-8bec-77ca2b26c61d,,workflow,,,failed,,16,,,0
78
1,b9ab7bbe-2f30-4c59-b4e2-eb2005bffb14,github/coldgust/coldgust.github.io,70f3eb15-3b94-4f80-b65e-f23f4b74c33a,,workflow,,,failed,1c762fc2-b0fb-4fe2-97d2-5e54ddd1eba7,6,2023-03-25T17:54:09.000+00:00,2023-03-25T17:54:23.000+00:00,14
89
1,c7df82a6-0d2b-4e19-a36a-3f3aa9fd3943,github/coldgust/coldgust.github.io,2c45280f-7fb3-4025-b703-a547c4a94916,,workflow,,,failed,1c762fc2-b0fb-4fe2-97d2-5e54ddd1eba7,4,2023-03-25T17:50:20.000+00:00,2023-03-25T17:50:25.000+00:00,5
910
1,fc76deef-bcdd-4856-8e96-a8e2d1c5a85f,github/coldgust/coldgust.github.io,afe0cabe-e7ee-4eb7-bf13-bb6170d139f0,,workflow,,,failed,1c762fc2-b0fb-4fe2-97d2-5e54ddd1eba7,8,2023-03-25T18:06:13.000+00:00,2023-03-25T18:06:28.000+00:00,15

backend/plugins/circleci/tasks/workflow_converter.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,9 @@ var ConvertWorkflowsMeta = plugin.SubTaskMeta{
4040

4141
func ConvertWorkflows(taskCtx plugin.SubTaskContext) errors.Error {
4242
rawDataSubTaskArgs, data := CreateRawDataSubTaskArgs(taskCtx, RAW_WORKFLOW_TABLE)
43+
logger := taskCtx.GetLogger()
44+
logger.Info("convert workflows")
45+
4346
db := taskCtx.GetDal()
4447
clauses := []dal.Clause{
4548
dal.From(&models.CircleciWorkflow{}),
@@ -57,6 +60,11 @@ func ConvertWorkflows(taskCtx plugin.SubTaskContext) errors.Error {
5760
Input: cursor,
5861
Convert: func(inputRow interface{}) ([]interface{}, errors.Error) {
5962
userTool := inputRow.(*models.CircleciWorkflow)
63+
// Skip if CreatedDate is null or empty string - still enters into the `_tool_circleci_workflows` table with null values
64+
if userTool.CreatedDate.ToNullableTime() == nil {
65+
logger.Info("CreatedDate is null or empty string in the CircleCI API response for %s", userTool.PipelineId)
66+
return []interface{}{}, nil
67+
}
6068
createdAt := userTool.CreatedDate.ToTime()
6169
pipeline := &devops.CICDPipeline{
6270
DomainEntity: domainlayer.DomainEntity{

0 commit comments

Comments
 (0)