Ubuntu Test Cases

Merge lp://qastaging/~doanac/ubuntu-test-cases/statsd into lp://qastaging/ubuntu-test-cases/touch

  1. statsd
  2. Merge into touch
Proposed by Andy Doan
Status: Merged
Merged at revision: 102
Proposed branch: lp://qastaging/~doanac/ubuntu-test-cases/statsd
Merge into: lp://qastaging/ubuntu-test-cases/touch
Diff against target: 182 lines (+94/-36)
5 files modified
jenkins/setup_jenkins.py (+1/-0)
jenkins/staging.py (+1/-0)
jenkins/templates/touch-smoke.xml.jinja2 (+1/-0)
scripts/run-smoke (+73/-36)
scripts/statsd-cli (+18/-0)
To merge this branch: bzr merge lp://qastaging/~doanac/ubuntu-test-cases/statsd
Reviewer Review Type Date Requested Status
Paul Larson Approve
Review via email: mp+192734@code.qastaging.launchpad.net

Description of the change

Gets us the ability to send some statsd metrics the mega job in staging

To post a comment you must log in.
lp://qastaging/~doanac/ubuntu-test-cases/statsd updated
98. By Paul Larson

Use bootstrap option on all images

99. By Andy Doan

change order of operations in provision.sh

100. By Andy Doan

increase timeout waiting for device in reboot-and-wait

We see places in automation where the phablet-network script
starts printing tons of:

  error: device not found

The wait-for-device is coming back too fast. ie - its not really ready
yet.

101. By Paul Larson

Modify whoopsie-upload-all in the images we provision so that whoopsie
gets restarted after .crash files are processed to work around a bug,
and use the new utah option to to force running whoopsie-upload-all
at the end of the job

Revision history for this message
Paul Larson (pwlars) wrote :

Looks good, just need to remember to go back and add stats for production once it goes into play :)

review: Approve
lp://qastaging/~doanac/ubuntu-test-cases/statsd updated
102. By Andy Doan

add statsd support for test-execution-service

Revision history for this message
Evan (ev) wrote :

Shouldn't STATSD_KEY only be set if {{statsd_key}} is set?

Revision history for this message
Andy Doan (doanac) wrote :

On 10/29/2013 03:25 PM, Evan Dandrea wrote:
> Shouldn't STATSD_KEY only be set if {{statsd_key}} is set?

setup_jenkins.py will set it to an empty string so "[ -z $STATSD_KEY ]"
should still evaluate properly for people's development setups.

Revision history for this message
Evan (ev) wrote :

Will it?

+export STATSD_KEY={{statsd_key}}.{{imagetype}}

That seems to imply that it will end up as STATSD_KEY=.{{imagetype}}

Revision history for this message
Andy Doan (doanac) wrote :

oh - so embarrassed and have no idea how this worked at home. must have forgotten to regenerate the job.

Revision history for this message
Andy Doan (doanac) wrote :

its fixed in trunk now.

Revision history for this message
Evan (ev) wrote :

:D

Excited for this.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches