[go: up one dir, main page]
Skip to content

Draft: POC - Store and read expose_as data from p_ci_job_artifacts for MR UI widget feature

Leaminn Marequested to merge
poc-ci-builds-artifacts-expose-as into master

What does this MR do and why?

Context

options => artifacts:expose_as is needed long term for the MR widgets exposing artifacts and it's used when has_exposed_artfacts: true

This should be considered intrinsic data because it's used in MR widgets to display artifacts.

In !195693 (merged), we stopped using has_exposed_artifacts.

This MR

This is the POC for adding the columns exposed_as and exposed_paths to the p_ci_job_artifacts table. The columns are unindexed for this iteration, so we still use the same batching technique from !195693 (merged) to find jobs with exposed artifacts.

We only read from these columns for the MR UI widget feature, i.e only after the job and artifacts have persisted. During job processing, we still obtain the values from options.

References

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #549079 (closed)

Edited by Leaminn Ma

Merge request reports