Ubuntu CI Services

Merge lp://staging/~cjohnston/ubuntu-ci-services-itself/1290790 into lp://staging/ubuntu-ci-services-itself

Proposed by Chris Johnston on 2014-03-11

Status:

Merged

Approved by:

Chris Johnston on 2014-03-13

Approved revision:

358

Merged at revision:

384

Proposed branch:

Merge into:

Diff against target:

5 files modified

webui/index.html (+1/-0)
webui/packages.html (+35/-0)
webui/status.html (+4/-2)
webui/webui.css (+7/-26)
webui/webui_packages.js (+36/-0)

To merge this branch:

bzr merge lp://staging/~cjohnston/ubuntu-ci-services-itself/1290790

Medium

Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
Joe Talbott (community)			Approve on 2014-03-13
PS Jenkins bot (community)	continuous-integration		Approve on 2014-03-13
Chris Johnston (community)			Needs Resubmitting on 2014-03-13
Evan (community)			Needs Fixing on 2014-03-11
Andy Doan (community)		2014-03-11	Approve on 2014-03-11
Review via email: mp+210487@code.staging.launchpad.net

Add page to WebUI with master package list

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-03-11:

PASSED: Continuous integration, rev:355
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/381/
Executed test runs:

review: Approve (continuous-integration)

Revision history for this message

Andy Doan (doanac) wrote on 2014-03-11:

LGTM. I think this is a good step for phase-0

review: Approve

Revision history for this message

Evan (ev) wrote on 2014-03-11:

I have a few successful tickets, but this API call isn't returning any:

http://15.125.115.133/api/v1/binarypackage/?format=json
{"meta": {"limit": 20, "next": null, "offset": 0, "previous": null, "total_count": 0}, "objects": []}

What am I missing?

Revision history for this message

Evan (ev) wrote on 2014-03-11:

197 + } catch (e) {
198 + Y.webui.package_table.showMessage(json_error_msg);
199 + return;
200 + }

209 + failure: function (tx, r, e) {
210 + console.log("failure: ", tx, r, e);
211 + }

I'd use showMessage in both of these cases.

review: Needs Fixing