Skip to content
Snippets Groups Projects

Re-add Product Analytics feature flag and update docs

Merged Robert Hunt requested to merge readd-product-analytics-feature-flag into master
All threads resolved!

What does this MR do and why?

Since Product Analytics (PA) is being deprioritised, we're moving PA back into closed beta behind a default-off feature flag.

Adding the feature flag will make it clear to self-managed instances that we don't currently support it. We can manually enable it for specific projects on .com as needed. There are currently no self-managed or dedicated users of PA.

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.

Scenario Flag disabled Flag enabled
Listing - PA not set up Google_Chrome_Beta_2024-09-26_12.51.37 image
Listing - PA set up Google_Chrome_Beta_2024-09-26_12.51.41 (custom dashboards are still there) image
Dashboard editor image (!167296 (comment 2131010244)) image
Dashboards page - Non-PA dashboard image image
Dashboards page - built in image image
Dashboards page - Custom image(PA visualizations will error with link to docs) image

How to set up and validate locally

  1. You need a GitLab Ultimate license.
  2. Navigate to Project > Analytics > Analytics dashboards
  3. Verify that you can see the VSA dashboard, but can't set up PA.
  4. Enable the new feature flag: echo "Feature.enable(:product_analytics_features)" | gdk rails c
  5. Follow the guide to enable product analytics on your GDK.
  6. Navigate back to Project > Analytics > Analytics dashboards.
  7. Click on the alert CTA on the dashboard listing to enable custom dashboards.
  8. Verify that you can set up PA, create PA dashboards, and navigate around.

Related to Bring Product Analytics back to a closed beta s... (#495013 - closed)

Edited by Robert Hunt

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 1 Message
    :book: This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge.

    Documentation review

    The following files require a review from a technical writer:

    The review does not need to block merging this merge request. See the:

    Pipeline Changes

    This merge request contains changes to the pipeline configuration for the GitLab project.

    Please consider the effect of the changes in this merge request on the following:

    Please consider communicating these changes to the broader team following the communication guideline for pipeline changes

    Reviewer roulette

    Category Reviewer Maintainer
    backend @jfypk profile link current availability (UTC+0, 1 hour behind author) @mwoolf profile link current availability (UTC+1, same timezone as author)
    test for spec/features/* @jfypk profile link current availability (UTC+0, 1 hour behind author) Maintainer review is optional for test for spec/features/*
    ~"Pipeline configuration" Reviewer review is optional for ~"Pipeline configuration" @acunskis profile link current availability (UTC+3, 2 hours ahead of author)

    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 :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

  • Robert Hunt changed the description

    changed the description

  • Author Maintainer

    author note (non-blocking): If the flag is disabled, the visualization designer will currently show an empty list of visualization options. This is a known issue, we need to add an empty state, although this will be remediated as we add additional data sources to reduce the likelihood this will happen in the first place.

  • Robert Hunt changed the description

    changed the description

  • Robert Hunt added 71 commits

    added 71 commits

    Compare with previous version

  • Robert Hunt changed the description

    changed the description

  • Robert Hunt
  • Robert Hunt
  • Robert Hunt marked this merge request as ready

    marked this merge request as ready

  • Robert Hunt requested review from @lciutacu

    requested review from @lciutacu

  • Robert Hunt requested review from @mwoolf

    requested review from @mwoolf

  • Author Maintainer

    @zmartins would you mind doing the test review for this MR? :pray:

  • Robert Hunt requested review from @zmartins

    requested review from @zmartins

  • Robert Hunt changed the description

    changed the description

  • Robert Hunt changed the description

    changed the description

  • Robert Hunt added 36 commits

    added 36 commits

    Compare with previous version

  • Lorena Ciutacu
  • Lorena Ciutacu
  • Lorena Ciutacu
  • Lorena Ciutacu
  • Max Woolf approved this merge request

    approved this merge request

  • Max Woolf requested review from @halilcoban

    requested review from @halilcoban

  • Before you set this MR to auto-merge

    This merge request will progress on pipeline tiers until it reaches the last tier: pipelinetier-3. We will trigger a new pipeline for each transition to a higher tier.

    Before you set this MR to auto-merge, please check the following:

    • You are the last maintainer of this merge request
    • The latest pipeline for this merge request is pipelinetier-3 (You can find which tier it is in the pipeline name)
    • This pipeline is recent enough (created in the last 8 hours)

    If all the criteria above apply, please set auto-merge for this merge request.

    See pipeline tiers and merging a merge request for more details.

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for a6cd03a8

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Plan        | 75     | 0      | 1       | 1     | 76    | ✅     |
    | Package     | 24     | 0      | 11      | 0     | 35    | ✅     |
    | Verify      | 45     | 0      | 2       | 0     | 47    | ✅     |
    | Create      | 128    | 0      | 18      | 0     | 146   | ✅     |
    | Fulfillment | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Monitor     | 16     | 0      | 0       | 0     | 16    | ✅     |
    | Govern      | 72     | 0      | 0       | 0     | 72    | ✅     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Secure      | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
    | Data Stores | 33     | 0      | 1       | 0     | 34    | ✅     |
    | Manage      | 1      | 0      | 1       | 0     | 2     | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 407    | 0      | 34      | 1     | 441   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-test-on-omnibus: :white_check_mark: test report for be90bfcc

    expand test summary
    +--------------------------------------------------------------+
    |                        suites summary                        |
    +---------+--------+--------+---------+-------+-------+--------+
    |         | passed | failed | skipped | flaky | total | result |
    +---------+--------+--------+---------+-------+-------+--------+
    | Monitor | 16     | 0      | 24      | 0     | 40    | ✅     |
    +---------+--------+--------+---------+-------+-------+--------+
    | Total   | 16     | 0      | 24      | 0     | 40    | ✅     |
    +---------+--------+--------+---------+-------+-------+--------+
  • Halil Coban approved this merge request

    approved this merge request

  • Max Woolf requested review from @ichernikov and removed review request for @zmartins

    requested review from @ichernikov and removed review request for @zmartins

  • Max Woolf added 1 commit

    added 1 commit

    Compare with previous version

  • Max Woolf reset approvals from @mwoolf and @halilcoban by pushing to the branch

    reset approvals from @mwoolf and @halilcoban by pushing to the branch

  • Max Woolf added 1 commit

    added 1 commit

    Compare with previous version

  • Ievgen Chernikov approved this merge request

    approved this merge request

  • Max Woolf requested review from @lciutacu

    requested review from @lciutacu

  • Lorena Ciutacu approved this merge request

    approved this merge request

  • Halil Coban approved this merge request

    approved this merge request

  • Halil Coban resolved all threads

    resolved all threads

  • Halil Coban enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Halil Coban cancelled automatic add to merge train

    cancelled automatic add to merge train

  • Halil Coban requested review from @splattael

    requested review from @splattael

  • Greg Alfaro approved this merge request

    approved this merge request

  • Halil Coban requested review from @sliaquat and removed review request for @splattael

    requested review from @sliaquat and removed review request for @splattael

  • Sanad Liaquat
  • Sanad Liaquat approved this merge request

    approved this merge request

  • Max Woolf resolved all threads

    resolved all threads

  • Max Woolf enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • New pipeline created an auto-merge set since we have all approvals.

  • merged

  • Max Woolf mentioned in commit 9f829ba5

    mentioned in commit 9f829ba5

  • added workflowstaging label and removed workflowcanary label

  • Ievgen Chernikov mentioned in merge request !167918 (merged)

    mentioned in merge request !167918 (merged)

  • Robert Hunt mentioned in merge request !171023 (merged)

    mentioned in merge request !171023 (merged)

  • Please register or sign in to reply
    Loading