Add top-level namespace to Usage Overview panel
What does this MR do and why?
Adds a new element with the top-level namespace's full name, type, visibility level icon and avatar to the Usage Overview panel on the Value Stream Dashboard.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
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
- Please make sure you have a GitLab Ultimate license.
Feature.enable(:group_analytics_dashboard_dynamic_vsd)
- Create a new (or choose an existing) top-level group and invoke the seed script:
GROUP_ID=<group-id> FILTER=vsd_overview_counts SEED_VSD_COUNTS=1 bundle exec rake db:seed_fu
- Navigate to the group above and, from the left sidebar, click the
Analyze
>Analytics dashboards
link - The group dashboard list should appear, select
Value Stream Dashboard
- Verify that the top-level namespace and related elements appear in the Usage overview panel.
Related to #439699 (closed)
Merge request reports
Activity
changed milestone to %16.11
assigned to @rcrespo3
1 Warning This merge request contains lines with testid selectors. Please ensure e2e:package-and-test
job is run.testid
selectorsThe following changed lines in this MR contain
testid
selectors:ee/app/assets/javascripts/analytics/analytics_dashboards/components/visualizations/usage_overview.vue
+ data-testid="usage-overview-namespace"
If the
e2e:package-and-test
job in theqa
stage has run automatically, please ensure the tests are passing. If the job has not run, please start themanual:e2e-test-pipeline-generate
job in theprepare
stage and ensure the tests infollow-up:e2e:package-and-test-ee
pipeline are passing.For the list of known failures please refer to the latest pipeline triage issue.
If your changes are under a feature flag, please check our Testing with feature flags documentation for instructions.
Reviewer roulette
Category Reviewer Maintainer frontend @thutterer
(UTC+2, 6 hours ahead of author)
@aturinske
(UTC-7, 3 hours behind author)
UX @lvanc
(UTC-10, 6 hours behind author)
Maintainer review is optional for UX Please check reviewer's status!
Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
Dangeradded 1 commit
- a04fb28d - Add top-level namespace to Usage Overview panel
Bundle size analysis [beta]
This compares changes in bundle size for entry points between the commits 7cd8b637 and 138dfafb
Special assetsEntrypoint / Name Size before Size after Diff Diff in percent average 4.32 MB 4.32 MB - 0.0 % mainChunk 3.31 MB 3.31 MB - 0.0 %
Note: We do not have exact data for 7cd8b637. So we have used data from: ea2a00b1.
The intended commit has no webpack pipeline, so we chose the last commit with one before it.Please look at the full report for more details
Read more about how this report works.
Generated by
Danger- Resolved by Rudy Crespo
Hey @ekigbo, when you get the chance, could you please provide some early feedback here? Looking to see if this implementation fits well within the existing architecture
Generated bygitlab_quality-test_tooling
.
Slow tests detected in this merge request. These slow tests might be related to this merge request's changes.Click to expand
Job File Name Duration Expected duration #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L101
Analytics Dashboard Visualizations with legacy value streams dashboard behaves like renders metrics comparison table renders the metrics comparison visualization 97.12 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L101
Analytics Dashboard Visualizations with legacy value streams dashboard behaves like renders metrics comparison table renders the available metrics 71.66 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L105
Analytics Dashboard Visualizations with legacy value streams dashboard behaves like renders contributor count renders the contributor count metric 64.73 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L106
Analytics Dashboard Visualizations with legacy value streams dashboard behaves like does not render usage overview metrics does not render the usage overview panel 62.85 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L107
Analytics Dashboard Visualizations with legacy value streams dashboard behaves like renders dora performers score renders the dora performers score visualization 80.74 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L35
Analytics Dashboard Visualizations for dora_chart visualization when ClickHouse is enabled for analytics behaves like renders metrics comparison table renders the metrics comparison visualization 80.69 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L35
Analytics Dashboard Visualizations for dora_chart visualization when ClickHouse is enabled for analytics behaves like renders metrics comparison table renders the available metrics 59.83 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L39
Analytics Dashboard Visualizations for dora_chart visualization when ClickHouse is enabled for analytics behaves like renders contributor count renders the contributor count metric 52.64 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L49
Analytics Dashboard Visualizations for dora_chart visualization when ClickHouse is disabled for analytics behaves like renders metrics comparison table renders the metrics comparison visualization 55.35 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L49
Analytics Dashboard Visualizations for dora_chart visualization when ClickHouse is disabled for analytics behaves like renders metrics comparison table renders the available metrics 65.4 s < 50.13 s #6543659781 ee/spec/features/groups/analytics/visualizations_spec.rb#L53
Analytics Dashboard Visualizations for dora_chart visualization when ClickHouse is disabled for analytics behaves like does not render contributor count does not render the contributor count metric 62.44 s < 50.13 s #6553292129 spec/features/projects/graph_spec.rb#L67
Project Graph charts graph ref switcher switches ref to branch 65.51 s < 50.13 s #6586729777 ee/spec/features/groups/analytics/visualizations_spec.rb#L101
Analytics Dashboard Visualizations with legacy value streams dashboard behaves like renders metrics comparison table renders the metrics comparison visualization 55.62 s < 50.13 s #6596632075 ee/spec/features/groups/analytics/visualizations_spec.rb#L101
Analytics Dashboard Visualizations with legacy value streams dashboard behaves like renders metrics comparison table renders the metrics comparison visualization 58.83 s < 50.13 s - A deleted user
added rspec:slow test detected label
added 140 commits
-
a04fb28d...45031aad - 139 commits from branch
master
- c746abe2 - Add top-level namespace to Usage Overview panel
-
a04fb28d...45031aad - 139 commits from branch
added workflowin review label and removed workflowin dev label
- Resolved by Rudy Crespo
requested review from @lvanc
- Resolved by Alexander Turinske
Hey @lcallahan, hope all is well! When you get the chance, could you please do the initial frontend review? Thanks!
requested review from @lcallahan
added 90 commits
-
c746abe2...3f4884a4 - 88 commits from branch
master
- 262654d9 - Add top-level namespace to Usage Overview panel
- a9b3e66d - Fix tooltip's positioning
-
c746abe2...3f4884a4 - 88 commits from branch
added pipeline:mr-approved label
- Resolved by Ezekiel Kigbo
@lvanc
, thanks for approving this merge request.This is the first time the merge request has been approved. To ensure we don't only run predictive pipelines, and we don't break
master
, a new pipeline will be started shortly.Please wait for the pipeline to start before resolving this discussion and set auto-merge for the new pipeline. See merging a merge request for more details.
E2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for 138dfafbexpand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Package | 24 | 0 | 6 | 0 | 30 | ✅ | | Govern | 66 | 0 | 0 | 0 | 66 | ✅ | | Create | 77 | 0 | 9 | 0 | 86 | ✅ | | Data Stores | 31 | 0 | 0 | 0 | 31 | ✅ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | | Release | 5 | 0 | 0 | 0 | 5 | ✅ | | Verify | 35 | 0 | 1 | 0 | 36 | ✅ | | Plan | 51 | 0 | 2 | 0 | 53 | ✅ | | Monitor | 7 | 0 | 0 | 0 | 7 | ✅ | | Manage | 0 | 0 | 1 | 0 | 1 | ➖ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 298 | 0 | 19 | 0 | 317 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
e2e-package-and-test:
test report for 138dfafbexpand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Plan | 249 | 0 | 19 | 0 | 268 | ✅ | | Create | 152 | 0 | 21 | 2 | 173 | ✅ | | Monitor | 8 | 0 | 0 | 0 | 8 | ✅ | | Govern | 6 | 0 | 0 | 0 | 6 | ✅ | | Data Stores | 4 | 0 | 0 | 0 | 4 | ✅ | | Package | 0 | 0 | 2 | 0 | 2 | ➖ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 419 | 0 | 42 | 2 | 461 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
requested review from @jmontal and removed review request for @lcallahan
removed review request for @lvanc
mentioned in issue #454898 (closed)
added 544 commits
-
a9b3e66d...e0d33e30 - 542 commits from branch
master
- 02fa476b - Add top-level namespace to Usage Overview panel
- 138dfafb - Fix tooltip's positioning
-
a9b3e66d...e0d33e30 - 542 commits from branch
requested review from @aturinske and removed review request for @jmontal
enabled an automatic merge when the pipeline for 2a53b118 succeeds
mentioned in incident gitlab-org/quality/engineering-productivity/master-broken-incidents#5844 (closed)
added workflowstaging-canary label and removed workflowin review label
added workflowcanary label and removed workflowstaging-canary label
added workflowstaging label and removed workflowcanary label
added workflowproduction label and removed workflowstaging label
added workflowpost-deploy-db-production label and removed workflowproduction label
added releasedcandidate label
mentioned in issue #455939 (closed)
mentioned in merge request kubitus-project/kubitus-installer!2945 (merged)
added releasedpublished label and removed releasedcandidate label
added pipelinetier-3 label