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

Issue 723863 link

Starred by 1 user
Status: Archived
Owner:
haddowk@chromium.org
Closed: Jun 2017
Cc:
█ ntang@chromium.org
█ krk@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Feature



Sign in to add a comment

If the pubsub credentials file exists in the partner bucket, use it and enable cloud notifications

Project Member Reported by haddowk@chromium.org, May 17 2017 


Currently partners have to go through several awkward steps to enable cloud notifications which are critical to the partner console working.

If the account manager has set up a access token for the partner and placed it into the partners bucket, use that credential file and enable cloud notification so the partner does not have to do any additional steps in the UI.
Project Member

Comment 1 by bugdroid1@chromium.org, May 23 2017

Labels: merge-merged-release-R59-9460.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/7049e9e5282d783fc459e0e4bef0d3b8350d542b

commit 7049e9e5282d783fc459e0e4bef0d3b8350d542b
Author: Keith Haddow <haddowk@chromium.org>
Date: Tue May 23 15:51:00 2017

[moblab] Use credentials file from the bucket if available.

If the partner bucket has a credentials file - use that file and
enable the cloud notifications.

Fix issue with the command to restart after the wizard, in some
cases restart was not a good option as the service had not started
stop and start solves that issue.

Cleanup, only validate the boto key one time, since it is now a
more expensive operation.

TEST=unit tests, manually tested running GTS on local moblab
BUG= chromium:723863 

Change-Id: I88e6d6901227fa667299a7c973d4f9276aceaf96
Reviewed-on: https://chromium-review.googlesource.com/510645
Reviewed-by: Michael Tang <ntang@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/7049e9e5282d783fc459e0e4bef0d3b8350d542b/frontend/afe/moblab_rpc_interface_unittest.py
[modify] https://crrev.com/7049e9e5282d783fc459e0e4bef0d3b8350d542b/frontend/afe/moblab_rpc_interface.py

Comment 2 by haddowk@chromium.org, Jun 5 2017

Status: Fixed (was: Started)

Comment 3 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)
►

Sign in to add a comment