New issue
Advanced search Search tips

Issue 746601 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Update wpt-exporter to use the new creds file and use the blink-w3c-autoroller gerrit account

Project Member Reported by qyears...@chromium.org, Jul 19 2017

Issue description

This is the current task after bug 738610; currently, wpt-exporter is currently failing due to missing credentials: https://luci-milo.appspot.com/buildbot/chromium.infra.cron/wpt-exporter/
 
Cc: jeffcarp@chromium.org
 Issue 742532  has been merged into this issue.
Creating a CL for this - it would have been better to do this in tandem with the creds change so the exporter isn't down in the interim.
That's true :-/
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/8d303199641a5f8dd3d5177d3c354207b79ab831

commit 8d303199641a5f8dd3d5177d3c354207b79ab831
Author: Jeff Carpenter <jeffcarp@chromium.org>
Date: Wed Jul 19 22:06:17 2017

Move WPT Export credentials location

The credentials file has been moved to Puppet in a new location:
/creds/json/wpt-export.json

Bug:  746601 
Change-Id: I193bc85eba0a38fa764cd9c24b2175d00291e346
Reviewed-on: https://chromium-review.googlesource.com/578481
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Jeff Carpenter <jeffcarp@chromium.org>

[modify] https://crrev.com/8d303199641a5f8dd3d5177d3c354207b79ab831/recipes/recipes/wpt_export.py
[modify] https://crrev.com/8d303199641a5f8dd3d5177d3c354207b79ab831/recipes/recipes/wpt_export.expected/wpt-export.json

(I could have foreseen this though, also it's not a big deal to run it manually a couple times)

Just sent the CL to the CQ.
https://chromium-review.googlesource.com/c/578481/
Cc: friedman@chromium.org
/creds/json is owned by root, so chrome-bot can't access the file. What's the best way to grant chrome-bot access to the file? Can this be done in Puppet?
Sorry I had a typo in the puppet module :(  s/indra/infra/
Project Member

Comment 8 by bugdroid1@chromium.org, Jul 19 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/puppet/+/e10ae4942ef6f6bd60cb0b10b4c50ed8a307e707

commit e10ae4942ef6f6bd60cb0b10b4c50ed8a307e707
Author: Elliott Friedman <friedman@google.com>
Date: Wed Jul 19 22:48:19 2017

Status: Fixed (was: Assigned)
Yaaaaaaaassssss it's working with the new account!
https://chromium-review.googlesource.com/c/577961/#message-a3de150b027211e7b6341ac4a3cdc09a73fb3c70
Excellent :-D

Sign in to add a comment