Project

General

Profile

Actions

Bug #44851

closed

mgr/dashboard: report bugs/improvements/feedback

Added by Ernesto Puerta about 4 years ago. Updated 22 days ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
General
Target version:
% Done:

100%

Source:
Community (dev)
Tags:
backport_processed
Backport:
pacific
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Add UI support in dashboard for users to report bugs, improvements and general feedback:
  1. Cluster admin may get their Ceph Tracker API key and configure it in the dashboard (CLI, UI -> Cluster -> Mgr Modules -> Dashboard).
    • A Ceph Tracker account is needed (see open points).
  2. Help menu could include a "Report/Send feedback..." entry.
    • If no API key has been configured, a modal should point users to the Ceph Tracker account creation and explain the API key (or points to the Ceph Docs section describing this procedure).
  3. "Report/Send feedback..." modal should expose the minimal set of fields required to fill up a Ceph-Dashboard Tracker Issue (e.g.: summary, description, severity).
    • Everything else should be auto-filled: tracker type (issue -> bug, suggestion -> feature, ...), affected Ceph version, category (mapping current Dashboard page to existing Tracker categories).
    • Recent pop-up error notifications could/should? be attached to the report.
    • Optionally a screenshot can be taken or attachments can be added.
Open points:
  • Ceph Tracker accounts require (for the time being) to be individually approved to avoid past spamming episodes. By forcing Ceph-dashboard users to wait for their accounts to be manually approved could be tiresome. Is there any effective Captcha to allow automated creation?
  • Contextual "report issue/feedback"? (on mouse-over? right-click?)
  • Add Web Analytics support (would it require explicit approval?). Apart from the Google Analytics , there exist different 'Open Source' analytic frameworks (e.g.: http://www.openwebanalytics.com/, https://matomo.org/, ...).

Subtasks 1 (0 open1 closed)

Subtask #51153: mgr/dashboard: report issues demoClosed

Actions

Related issues 1 (0 open1 closed)

Copied to Dashboard - Backport #53920: pacific: mgr/dashboard: report bugs/improvements/feedbackRejectedShreya SharmaActions
Actions #1

Updated by Ernesto Puerta about 3 years ago

  • Project changed from mgr to Dashboard
  • Category changed from 132 to General
Actions #2

Updated by Ernesto Puerta almost 3 years ago

Fields display (5):
  • Bug or Feature (dropdown)
  • Project -> Component (Ernesto to provide a shortlist of components to display)
  • Severity: Critical, major, minor. (dropdown)
  • Subject (free-form text)
  • Description (free-form text)
  • Affected version (auto-guess from Ceph? Retrieve from Redmine API. Discard old versions: v0.*) (drop-down with an auto-guessed pre-selection)
  • Source [Internal]: "Dashboard Report" (ask DavidG)
First time a user wants to report an issue ("Welcome to Ceph", wizard-like):
  • Clicks on the report issue
  • If not API Key is registered, "please enter your Ceph Tracker API Key" (with a link to https://tracker.ceph.com/my/api_key).
  • "If not registered, please sign up in Ceph Tracker <url>"
  • API key will be validated before moving on
Key becomes invalid/obsolete/etc:
  • Detect API Key validity on Form loading
  • If not valid, redirect to the Wizard (informing "Registered API Key is no longer valid")
User wants to change API key for whatever reason (new username, etc):
  • Provide CLI command (e.g.: ceph dashboard set-ceph-tracker-api-key <new_key>)
Where to store the API Key:

Python Redmine: https://www.redmine.org/projects/redmine/wiki/Rest_api_with_python

Actions #3

Updated by Ernesto Puerta over 2 years ago

Shreya, this is the feedback collected from your demo:

  • Josh: release shared public key to allow any user anonymously report issues.
    • Check with David about spamkillers.
  • Sage: extend the description field and, after creating the issue, display a link to the trackers.
  • Yaarit: Source to indentify users (CLI, Dashboard).
  • Sage: fill up 'Affected versions'.
  • Prepopulate description field with: 'expected behavior', 'actual behavior', 'steps to reproduce'
  • Nizam: "dashboard feedback create" vs. "create issue"
  • Ernesto: create new standalone mgr module (for users not running the Dashboard)?
Actions #4

Updated by Ernesto Puerta over 2 years ago

  • Status changed from New to In Progress
  • Assignee set to Shreya Sharma
Actions #5

Updated by Pere Díaz Bou over 2 years ago

Hey one more suggestion :P. What about when creating the issue from the cli you could have the option to open a template file, with your default editor, so you don't have to go the tracker url to complete the description. Basically, simulate something like the command `git commit` that opens `vim`, `nano` or whatever.

Actions #6

Updated by Ernesto Puerta over 2 years ago

Top:
#. Docs
#. Blog post

#. Sage: extend the description field and, after creating the issue, display a link to the trackers.
#. Prepopulate description field with: 'expected behavior', 'actual behavior', 'steps to reproduce'
#. Nizam: "dashboard feedback create" vs. "create issue"
#. Ernesto: create new standalone mgr module (for users not running the Dashboard)?

Nice-to-have:
#. Pere: open a CLI editor for filling up the description (maybe with a template too).
#. Yaarit: Source to indentify users (CLI, Dashboard).
#. Sage: fill up 'Affected versions'.

Actions #7

Updated by Alfonso Martínez over 2 years ago

  • Pull request ID set to 42090
Actions #8

Updated by Alfonso Martínez over 2 years ago

  • Status changed from In Progress to Fix Under Review
Actions #9

Updated by Yaarit Hatuka over 2 years ago

If crash info is included - please fill the crash signature V1 field

Actions #10

Updated by Ernesto Puerta about 2 years ago

  • Status changed from Fix Under Review to Pending Backport
  • Backport set to pacific

We might tentatively try a backport to Pacific

Actions #11

Updated by Backport Bot about 2 years ago

  • Copied to Backport #53920: pacific: mgr/dashboard: report bugs/improvements/feedback added
Actions #12

Updated by Backport Bot over 1 year ago

  • Tags set to backport_processed
Actions #13

Updated by Konstantin Shalygin 22 days ago

  • Tracker changed from Feature to Bug
  • Status changed from Pending Backport to Resolved
  • Regression set to No
  • Severity set to 3 - minor
Actions

Also available in: Atom PDF