ci-core-jenkins

Merge lp://staging/~joetalbott/ci-core-jenkins/fix_names into lp://staging/ci-core-jenkins

fix_names
Merge into trunk

Proposed by Joe Talbott on 2015-06-22

Status:	Merged
Approved by:	Joe Talbott on 2015-06-23
Approved revision:	17
Merged at revision:	16
Proposed branch:	lp://staging/~joetalbott/ci-core-jenkins/fix_names
Merge into:	lp://staging/ci-core-jenkins
Diff against target:	34 lines (+9/-8) 1 file modified workers/fetch_latest_results/__init__.py (+9/-8)
To merge this branch:	bzr merge lp://staging/~joetalbott/ci-core-jenkins/fix_names
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Francis Ginther		Approve on 2015-06-23
Para Siva (community)	2015-06-22	Approve on 2015-06-23
Review via email: mp+262636@code.staging.launchpad.net

Commit message

Make container and result file names match what lp:autopkgtest-result-checker produces.

Description of the change

Make container and result file names match what lp:autopkgtest-result-checker produces.

Revision history for this message

Para Siva (psivaa) wrote on 2015-06-23:

Seems ok to me, not being able to test this though.

review: Approve

Revision history for this message

Para Siva (psivaa) wrote on 2015-06-23:

This also needs the AUTH url change in the config file. I already had an MP for this in-progress for the channel, release name fixes to land. https://code.launchpad.net/~psivaa/ci-core-jenkins/ps4.5-auth-url/+merge/261357

Revision history for this message

Francis Ginther (fginther) wrote on 2015-06-23:

Argh, I missed this in my earlier review, there is no "args.release" being parsed. I'm not opposed to setting this with a default value of "rolling" and limiting the amount of code changes.

review: Needs Fixing

lp://staging/~joetalbott/ci-core-jenkins/fix_names updated on 2015-06-23