New issue
Advanced search Search tips

Issue 797036 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 794630
issue 796614



Sign in to add a comment

puppet delivers monarch credentials in the wrong place

Reported by jrbarnette@chromium.org, Dec 21 2017

Issue description

In chromeos-admin, in puppet/modules/profiles/manifests/base.pp,
you see the following assignment:
    $service_accounts_dir = "${creds_dir}/service_accounts_dir"

In chromite, in third_party/infra_libs/httplib2_utils.py, you
see this assignment:
    SERVICE_ACCOUNTS_CREDS_ROOT = '/creds/service_accounts'

IOW, puppet is delivering to the wrong directory.

This is blocking new shards from reporting metrics...

 
Blocking: 796614
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 21 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-admin/+/892acc2aa1c19b03002aa5d188f986a1dd2f7942

commit 892acc2aa1c19b03002aa5d188f986a1dd2f7942
Author: Richard Barnette <jrbarnette@google.com>
Date: Thu Dec 21 21:44:22 2017

Blocking: 794630
Components: Infra>Client>ChromeOS
Owner: jrbarnette@chromium.org
Status: Fixed (was: Untriaged)
server118 has the change, and is able to deliver metrics
Labels: -Pri-0 Pri-3
Owner: pprabhu@chromium.org
Status: Assigned (was: Fixed)
Going to let pprabhu@ use this to wrap up the change by
deleting the obsolete/wrong stuff via puppet.

Project Member

Comment 6 by bugdroid1@chromium.org, Dec 22 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-admin/+/2af1d33522a4211e89c54e7eb057d936e11f0d56

commit 2af1d33522a4211e89c54e7eb057d936e11f0d56
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Fri Dec 22 01:39:54 2017

Status: Fixed (was: Assigned)

Sign in to add a comment