Skip to content

Add artifacts files to verify codeowners

Miranda Fluharty requested to merge add-artifacts-files-to-verify-codeowners into master

What does this MR do and why?

As part of the FCL for this incident, we want to build on the foundation laid by #393864 (closed) / !115481 (merged) to ensure that artifacts-related code is reviewed by a domain expert in order to maintain high reliability in this area.

At the same time, we don't want this to be too sensitive - I added all of the files that weren't covered that seemed related in the first commit, opened discussions with links to the files, then took some of them back out in the second commit and closed their discussions. I also left notes explaining how existing lines already covered some artifacts files.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Miranda Fluharty

Merge request reports