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

Issue 710652 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 0
Type: Bug



Sign in to add a comment

Cronet Builders giving errors for missing --bucket parameter

Project Member Reported by donnd@chromium.org, Apr 11 2017

Issue description

hzl@, can you take a look?

We're just now seeing multiple errors of a missing "bucket" param, and it looks like your recent change removed a default value, so maybe the scripts need to be updated too?

Builder link: https://uberchromegw.corp.google.com/i/chromium.android/builders/Android%20Cronet%20Marshmallow%2064bit%20Builder

Bot command:
python -u /b/build/slave/Android_Cronet_Marshmallow_64bit_Builder/build/src/build/android/pylib/results/presentation/test_results_presentation.py --json-file /tmp/tmpr_5oaD.json --test-name cronet_sample_test_apk --builder-name 'Android Cronet Marshmallow 64bit Builder' --build-number 8279

Error:
usage: test_results_presentation.py [-h] --json-file JSON_FILE
                                    [--cs-base-url CS_BASE_URL] --bucket
                                    BUCKET --builder-name BUILDER_NAME
                                    --build-number BUILD_NUMBER --test-name
                                    TEST_NAME [--server-url SERVER_URL]
                                    [--content-type {text/html,application/octet-stream}]
test_results_presentation.py: error: argument --bucket is required
step returned non-zero exit code: 2

Suspect CL: https://codereview.chromium.org/2799153008

 

Comment 1 by donnd@chromium.org, Apr 11 2017

Components: Infra
Labels: -Pri-3 OS-Android Pri-0
Status: Assigned (was: Untriaged)

Comment 2 by donnd@chromium.org, Apr 11 2017

Cc: bauerb@chromium.org

Comment 3 by donnd@chromium.org, Apr 11 2017

Looks like the suspect change is being reverted (https://codereview.chromium.org/2814903002/)
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 12 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/85f4a60bca6cbc5f6f1e15d270d5cb69efcdba0b

commit 85f4a60bca6cbc5f6f1e15d270d5cb69efcdba0b
Author: Zhiling Huang <hzl@google.com>
Date: Wed Apr 12 17:19:33 2017

Comment 5 by hzl@chromium.org, Apr 13 2017

Status: Fixed (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 14 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/cd09a4afb81a09ef326bc5617d0066728236bf27

commit cd09a4afb81a09ef326bc5617d0066728236bf27
Author: Zhiling Huang <hzl@google.com>
Date: Fri Apr 14 15:46:27 2017

Project Member

Comment 7 by bugdroid1@chromium.org, Apr 14 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/cd09a4afb81a09ef326bc5617d0066728236bf27

commit cd09a4afb81a09ef326bc5617d0066728236bf27
Author: Zhiling Huang <hzl@google.com>
Date: Fri Apr 14 15:46:27 2017

Project Member

Comment 8 by bugdroid1@chromium.org, Apr 14 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/cd09a4afb81a09ef326bc5617d0066728236bf27

commit cd09a4afb81a09ef326bc5617d0066728236bf27
Author: Zhiling Huang <hzl@google.com>
Date: Fri Apr 14 15:46:27 2017

Sign in to add a comment