Skip to content

Show warning when running VSA seed on broken repo

Adam Hegyi requested to merge 373523-show-warning-when-seeding-vsa-with-broken-project into master

What does this MR do and why?

This MR addresses the rarely occurring VSA seeding error mentioned in: #373523 (closed)

Problem: When we seed a project without git repository the seed script errors out. Creating an MR requires a functioning repository.

Fixes:

  • When a project doesn't have a repository, show a warning to the user.
  • When the VSA_SEED_PROJECT_ID parameter is not set, create a project with repository. (this how we ended up "broken" project records without repository)
  • Additionally: fix possible nil values for issue and merge_request metrics records.

How to test it

  1. Create a project via FactoryBot: FactoryBot.create(:project).id
  2. Use the id to seed VSA: VSA_SEED_PROJECT_ID=1 SEED_CYCLE_ANALYTICS=true SEED_VSA=true FILTER=cycle_analytics rake db:seed_fu
  3. You should see a warning message.

MR acceptance checklist

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

Related to #373523 (closed)

Edited by Adam Hegyi

Merge request reports