Nickcw6 opened a new pull request, #7910:
URL: https://github.com/apache/incubator-devlake/pull/7910

   ### Summary
   Fixes bug whereby CircleCI deployments matching job name were not pulling 
through to `cicd_deployments`.
   
   This was due to the `job_converter` setting `pipeline_id` as the UUID of the 
CircleCI pipeline on `cicd_tasks`. The query to generate `cicd_deployments` 
attempts to join this column onto the `id` of `cicd_pipelines` - which is in 
the format `circleci:CircleciWorkflow:1:{workflow_id}`. Updating the value of 
`pipeline_id` on `cicd_task` resolved the issue.
   
   Also updated relevant tests & changed the name of the relevant IdGen 
function in the plugin -> `getWorkflowIdGen`.
   
   Retransforming the data after merge will fix the issue.
   
   ### Does this close any open issues?
   Closes #7889 
   
   ### Screenshots
   Before:
   
   2 deployments in `cicd_tasks`
   <img width="954" alt="Screenshot 2024-08-16 at 16 48 39" 
src="https://github.com/user-attachments/assets/b5e2bbfa-64ed-4800-b75f-116dfd61e702";>
   
   Nothing in `cicd_deployments`
   <img width="447" alt="Screenshot 2024-08-16 at 16 50 54" 
src="https://github.com/user-attachments/assets/423bdb9b-abce-4ca5-bf30-8c5f171bcf6c";>
   
   Nothing in `cicd_deployment_commits`
   <img width="615" alt="Screenshot 2024-08-16 at 16 52 00" 
src="https://github.com/user-attachments/assets/d8d5f428-4074-4c70-8eff-5285053f89f9";>
   
   After:
   <img width="965" alt="Screenshot 2024-08-16 at 16 54 16" 
src="https://github.com/user-attachments/assets/162828e7-b381-44d4-9d40-ffe4fc9deff0";>
   
   `cicd_deployments`:
   <img width="683" alt="Screenshot 2024-08-16 at 16 54 36" 
src="https://github.com/user-attachments/assets/8ed32726-b347-4d10-9f0d-2685aac755ec";>
   
   
   `cicd_deployment_commits`:
   <img width="784" alt="Screenshot 2024-08-16 at 16 54 55" 
src="https://github.com/user-attachments/assets/6121c67b-025f-47d2-8158-a7faa6f1f44b";>
   
   
   ### Other Information
   Any other information that is important to this PR.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to