Update wpt-exporter to use the new creds file and use the blink-w3c-autoroller gerrit account |
|||
Issue descriptionThis 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/
,
Jul 19 2017
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.
,
Jul 19 2017
That's true :-/
,
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
,
Jul 19 2017
(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/
,
Jul 19 2017
/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?
,
Jul 19 2017
Sorry I had a typo in the puppet module :( s/indra/infra/
,
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
,
Jul 19 2017
Yaaaaaaaassssss it's working with the new account! https://chromium-review.googlesource.com/c/577961/#message-a3de150b027211e7b6341ac4a3cdc09a73fb3c70
,
Jul 19 2017
Excellent :-D |
|||
►
Sign in to add a comment |
|||
Comment 1 by jeffcarp@chromium.org
, Jul 19 2017