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

Issue 639038 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"setup_properties" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Aug 18 2016

Issue description

"setup_properties" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 49 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyGwsSBUZsYWtlIhBzZXR1cF9wcm9wZXJ0aWVzDA.



This flaky test/step was previously tracked in  issue 637042 .
 
Labels: -Sheriff-Chromium Infra-Troopers
This is looking green in recent builds, but it looks like the setup_properties has a history of occasionally failing on this bot (20 failures in last 200 builds).  This failure also looks similar to the one on official builders in issue 638988.  Adding troopers to take a look.

Sample failure log:

Traceback (most recent call last):
  File "../../../scripts/slave/annotated_run.py", line 340, in <module>
    sys.exit(shell_main(sys.argv))
  File "../../../scripts/slave/annotated_run.py", line 335, in shell_main
    return main(argv[1:])
  File "../../../scripts/slave/annotated_run.py", line 315, in main
    tdir, opts.build_properties, use_factory_properties_from_disk)
  File "../../../scripts/slave/annotated_run.py", line 101, in get_recipe_properties
    workdir, mastername, buildername)
  File "../../../scripts/slave/annotated_run.py", line 165, in get_factory_properties_from_disk
    dump_cmd, env.Build, proc.returncode, out, err))
LookupError: Failed to get the master config; running ['/usr/bin/python', '/b/build/scripts/tools/dump_master_cfg.py', '/b/build/masters/master.tryserver.chromium.mac', '/b/build/slave/ios-device/.recipe_runtime/tmpMgiuV8/dump_master_cfg.json'] in '/b/build' returned exit code 1
stdout: 
stderr: error while parsing /b/build/masters/master.tryserver.chromium.mac/master.cfg: 
Traceback (most recent call last):
  File "/b/build/scripts/common/master_cfg_utils.py", line 105, in LoadConfig
    localdict = ExecuteConfig(canonical_config)
  File "/b/build/scripts/common/master_cfg_utils.py", line 80, in ExecuteConfig
    exec f in localDict
  File "/b/build/masters/master.tryserver.chromium.mac/master.cfg", line 16, in <module>
    from master import master_utils
  File "/b/build/scripts/master/master_utils.py", line 24, in <module>
    from buildbot.status.web.baseweb import WebStatus
  File "/b/build/third_party/buildbot_8_4p1/buildbot/status/web/baseweb.py", line 46, in <module>
    from infra_libs.ts_mon.common import http_metrics
ImportError: No module named infra_libs.ts_mon.common

Traceback (most recent call last):
  File "/b/build/scripts/tools/dump_master_cfg.py", line 55, in <module>
    sys.exit(main(sys.argv[1:]))
  File "/b/build/scripts/tools/dump_master_cfg.py", line 47, in main
    json.dump(result['BuildmasterConfig'],
TypeError: 'NoneType' object has no attribute '__getitem__'


Comment 2 by estaab@chromium.org, Aug 18 2016

Owner: sergeybe...@chromium.org
Status: Assigned (was: Untriaged)
Hey Sergey, is the ts_mon error a problem here or is it a red herring for something else?
Cc: dsansome@chromium.org
This was likely due to dsansome@'s recent attempts to land infra_libs & ts_mon into build.git. AFAIK, the change was reverted, and I don't yet know if it re-landed again. 
+dsansome for more info.
Status: Fixed (was: Assigned)
Yeah this was me, it's fixed with https://codereview.chromium.org/2256183003/
Issue 638988 has been merged into this issue.

Sign in to add a comment