Follow-up from "Add InternalEvent tracking to finished pipelines"
The following discussion from !178392 (merged) should be addressed:
-
@mfanGitLab started a discussion: (+3 comments)
Looks like the current convention is to group similar outcomes together.
This one is a bit difference, so maybe we should do another
after_transition any => ::Ci::Pipeline.completed_statuses
clause for it?I am curious if we could combine everything though, so I'll create an issue for myself to see and do that in a future MR
I want to split this into two distinct sections
- Worker section where the transition enqueues a worker
- Sync section where it does internal events and lightweight tracking
But I can do that in a follow-up