Skip to content

Let update-tests-metadata pull data from clickhouse

Lin Jen-Shin requested to merge update-tests-metadata-for-clickhouse into master

What does this MR do and why?

Let update-tests-metadata pull data from clickhouse.

!137267 (merged) was attempting to update https://gitlab-org.gitlab.io/gitlab/knapsack/report-master.json so it can have spec/lib/gitlab/database/click_house_client_spec.rb data available. It turns out that it's not enough because update-tests-metadata didn't pull the report from the jobs.

dependencies does not care when the jobs are missing, so we don't need to worry about not running clickhouse jobs for update-tests-metadata.

See gitlab-org/quality/engineering-productivity/team#323 (comment 1654025755) for the background.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

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

Merge request reports