New issue
Advanced search Search tips

Issue 659334 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

run_puppet error for drone role

Project Member Reported by dshi@chromium.org, Oct 25 2016

Issue description

run_puppet in lab failed with following error in drone:

Error: Parameter path failed on File[undef]: File paths must be fully qualified, not 'undef' at /root/chromeos-admin/puppet/modules/lab/manifests/chrome_infra_service_accounts_dir.pp:11
Wrapped exception:
File paths must be fully qualified, not 'undef'


 

Comment 1 by dshi@chromium.org, Oct 25 2016

Owner: akes...@chromium.org
Status: Assigned (was: Untriaged)
It's added in following cl:
https://chrome-internal-review.googlesource.com/#/c/297558

Comment 2 by dshi@chromium.org, Oct 25 2016

Cc: ayatane@chromium.org
$chrome_infra_service_accounts_dir is not assigned anywhere in the puppet code, did I miss anything?
Sorry, should have caught that in review.  Puppet tests should be ready soon, which will start catching these.

Comment 4 by dshi@chromium.org, Oct 25 2016

one more related issue:
Error: Could not find dependency File[undef] for File[/creds/service_accounts/service_account_mon_pubsub.json] at /root/chromeos-admin/puppet/modules/lab/manifests/ts_mon.pp:14

in ts_mon.pp

  file { "${dir}/service_account_mon_pubsub.json":
    require => [ File[$chrome_infra_service_accounts_dir], Exec["create ${user}"] ],
    ensure  => present,

chrome_infra_service_accounts_dir needs to be dir too?

Comment 5 by dshi@chromium.org, Oct 25 2016

same issue in modules/lab/manifests/datastore_writer_prod.pp
done in that module too

Comment 7 by dshi@chromium.org, Oct 26 2016

Are you referring to:
https://chrome-internal-review.googlesource.com/#/c/299937/3/puppet/modules/lab/manifests/datastore_writer_prod.pp

It still has:
 File[$chrome_infra_service_accounts_dir]
Status: Fixed (was: Assigned)

Comment 9 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 10 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 11 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 12 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 14 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment