Skip to content

Move lead and cycle time classes out from Group NS

Adam Hegyi requested to merge 327570-project-level-lead-and-cycle-time into master

What does this MR do?

This MR makes the cycle time and lead time figures available on the project-level. Note: the FE part is not available yet, so the change is not user-facing. This is an EE change because the cycle time and lead time needs license.

Code move: Move files from Gitlab::Analytics::CycleAnalytics::Summary::Group to Gitlab::Analytics::CycleAnalytics::Summary to have generic classes which work for both the group and project level features.

How to test it:

  1. Make sure you have a premium license.
  2. Seed VSA: SEED_VSA=true FILTER=cycle_analytics rake db:seed_fu
  3. Visit the generated project path (check the output)
  4. Copy the project path and replace the URL: http://localhost:3000/GROUP_PATH/PROJECT_PATH/-/analytics/value_stream_analytics/time_summary.json?created_after=2019-05-01
  5. The endpoint should load. It's likely that you won't see any data. (-)

Screenshots (strongly suggested)

Group-level view. Note: project-level view is not available yet:

image

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #327570 (closed)

Edited by Adam Hegyi

Merge request reports

Loading