Skip to content
Snippets Groups Projects

Add self-monitoring deprecation notice for self-monitoring project

Merged Rajendra Kadam requested to merge 353145-self-monitor-deprecation-notice into master
All threads resolved!

What does this MR do and why?

It adds a deprecation notice to GitLab self-monitoring project.

Addresses #353145 (closed)

Screenshots or screen recordings

Screenshot_2023-03-07_at_5.27.18_PM

How to set up and validate locally

  1. Go to On the Admin > Settings > Metrics and profiling page (where self-monitoring is enabled)
  2. Select self-monitoring project and check the alert on the project details page.
  3. The alert should be visible in both cases, empty project and project with files.

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 #353145 (closed)

Edited by Rajendra Kadam

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
  • Tristan Read removed review request for @tristan.read

    removed review request for @tristan.read

  • Rajendra Kadam added 1 commit

    added 1 commit

    • fdcd1d61 - Change deprecation milestone

    Compare with previous version

  • requested review from @tristan.read

  • Rajendra Kadam added 1 commit

    added 1 commit

    • 73285806 - Update translations for strings

    Compare with previous version

  • Tristan Read resolved all threads

    resolved all threads

    • Resolved by Tristan Read

      Thanks @rkadam3, the changes look good to me, I'll approve. Because it's a user-facing change I believe it technically needs a product design reviewer to look at it.

      @nickleonard I spun the roulette and it chose you - would you mind helping with this?

      Feel free to ping back to me for merge, or another maintainer.

  • Tristan Read approved this merge request

    approved this merge request

  • Tristan Read requested review from @nickleonard

    requested review from @nickleonard

  • added UX label

  • Please wait for Reviewer Roulette to suggest a designer for UX review, and then assign them as Reviewer. This helps evenly distribute reviews across UX.

    This message was generated automatically. You're welcome to improve it.

  • Rajendra Kadam changed the description

    changed the description

  • Rajendra Kadam added 3 commits

    added 3 commits

    • f5aca9c7 - Add warning to self-monitoring form
    • 91e62685 - Make note sticky
    • 16166a7a - Revert "Add warning to self-monitoring form"

    Compare with previous version

  • Rajendra Kadam removed review request for @tristan.read

    removed review request for @tristan.read

  • closed

  • Rajendra Kadam requested review from @tristan.read

    requested review from @tristan.read

  • reopened

  • Rajendra Kadam changed the description

    changed the description

  • Rajendra Kadam added 1 commit

    added 1 commit

    • d74e7d2b - Make alert page level and container width

    Compare with previous version

  • Nick Leonard approved this merge request

    approved this merge request

  • Nick Leonard removed review request for @nickleonard

    removed review request for @nickleonard

  • Tristan Read approved this merge request

    approved this merge request

  • Tristan Read resolved all threads

    resolved all threads

  • Tristan Read enabled an automatic merge when the pipeline for 4a40d158 succeeds

    enabled an automatic merge when the pipeline for 4a40d158 succeeds

  • merged

  • Tristan Read mentioned in commit 236de23c

    mentioned in commit 236de23c

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading