Add CI build trace sections extractor
What does this MR do?
This MR extracts timestamped section marker from build trace log.
As a first iteration, sections are just identified and recorded on DB.
Are there points in the code the reviewer needs to double check?
Why was this MR needed?
With https://gitlab.com/gitlab-org/gitlab-runner/merge_requests/689 gitlab-runner
will start sending timestamped sections marker
Screenshots (if relevant)
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary -
Documentation created/updated -
API support added -
Tests added for this feature/bug - Review
-
Has been reviewed by UX -
Has been reviewed by Frontend -
Has been reviewed by Backend -
Has been reviewed by Database
-
-
Conform by the merge request performance guides -
Conform by the style guides -
Squashed related commits together
What are the relevant issue numbers?
See https://gitlab.com/gitlab-org/gitlab-runner/issues/2505 Closes #37970 (closed)
Merge request reports
Activity
- Resolved by Kamil Trzcińśki
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Kamil Trzcińśki
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
@grzesiek Could you take a look at this and propose how we can implement the backend code?
- Resolved by Kamil Trzcińśki
- Resolved by Kamil Trzcińśki
- Resolved by Kamil Trzcińśki
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Grzegorz Bizon
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
Nice merge request @nolith! I left just a few notes!
assigned to @nolith
mentioned in merge request !14393 (merged)
- Resolved by Kamil Trzcińśki
- Resolved by Kamil Trzcińśki
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Kamil Trzcińśki
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
- Resolved by Kamil Trzcińśki
added 356 commits
- 6fdc7dec...f1b8d79f - 350 commits from branch
master
- 65adc654 - Add CI build trace sections extractor
- 198afe12 - DB related review
- 5c560662 - Fix broken test
- 3843208e - Code review corrections
- 98bbd305 - Don't catch errors in
Build#parse_trace_sections!
- 5b4fbdf7 - DB migrations code review
Toggle commit list- 6fdc7dec...f1b8d79f - 350 commits from branch
- Resolved by Alessio Caiazza
assigned to @ayufan
- Resolved by Alessio Caiazza
- Resolved by Alessio Caiazza
@grzesiek Can you take a look?
assigned to @nolith
- Resolved by Grzegorz Bizon
added CI/CD label
added 1 commit
-
c0cfc9eb - Extract
Ci::Build#parse_trace_sections!
into a service
-
c0cfc9eb - Extract
assigned to @ayufan
@yorickpeterse Can you review DB?
@grzesiek Can you do one more Backend review?
If everything will be fine I will do final review and Approve :)
- Resolved by Kamil Trzcińśki
@ayufan Let's merge it!
added 407 commits
-
c0cfc9eb...d4f3963e - 406 commits from branch
master
- 5ee20b63 - Merge branch 'master' into '37970-ci-sections-tracking'
-
c0cfc9eb...d4f3963e - 406 commits from branch
enabled an automatic merge when the pipeline for 5ee20b63 succeeds
marked the checklist item Changelog entry added, if necessary as completed
marked the checklist item Documentation created/updated as completed
marked the checklist item Conform by the merge request performance guides as completed
marked the checklist item Conform by the style guides as completed
marked the checklist item Squashed related commits together as completed
mentioned in commit 8eec69ef