New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 686979 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Feature



Sign in to add a comment

Add feedback button to Swarming/Isolate/Auth/MP instances

Project Member Reported by mar...@chromium.org, Jan 31 2017

Issue description

1) Add a new "feedback_url" entry to
https://github.com/luci/luci-py/blob/master/appengine/swarming/proto/config.proto
https://github.com/luci/luci-py/blob/master/appengine/machine_provider/proto/config.proto
https://github.com/luci/luci-py/blob/master/appengine/auth_service/proto/config.proto

2) When the entry is set, add the <script> tag listed here to the UI:
https://chromium.googlesource.com/infra/infra/+/master/crdx/feedback

Do this for Swarming, Isolate, Auth, MP and whatever else.


For isolate, it is more work because it doesn't have luci-config integration yet so this is a prerequisite.
 

Comment 1 by kjlubick@google.com, Jan 31 2017

Would this eventually replace the feedback button already on the swarming UI?
https://screenshot.googleplex.com/4oArUAUx070

Comment 2 by mar...@chromium.org, Jan 31 2017

yeah the idea is to make it more generic and consistent. Did you get reports at all? If not, probably not worth.

Comment 3 by kjlubick@google.com, Jan 31 2017

I got several reports via that button.

Comment 4 by s...@google.com, Feb 16 2017

I copied the script to the machine-provider-dev but it doesn't seem to be working?

https://2644-4ac1c37-tainted-smut-dot-machine-provider-dev.appspot.com/

View source verifies the <script> is present.
Project Member

Comment 5 by sheriffbot@chromium.org, Feb 21 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 6 by mar...@chromium.org, Feb 21 2018

Labels: -Type-Bug -Hotlist-Recharge-Cold Type-Feature
Status: Available (was: Untriaged)

Sign in to add a comment