Merge lp://staging/~deeptik/lava-dashboard/kernel-ci-data-veiw-report into lp://staging/~linaro-validation/lava-dashboard/data-views-and-reports

Proposed by Deepti B. Kalakeri
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 29
Merged at revision: 29
Proposed branch: lp://staging/~deeptik/lava-dashboard/kernel-ci-data-veiw-report
Merge into: lp://staging/~linaro-validation/lava-dashboard/data-views-and-reports
Diff against target: 224 lines (+196/-0)
3 files modified
reports/kernel-ci-data.html (+54/-0)
reports/kernel-ci-data.xml (+4/-0)
views/kernel-ci-data.xml (+138/-0)
To merge this branch: bzr merge lp://staging/~deeptik/lava-dashboard/kernel-ci-data-veiw-report
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+74563@code.staging.launchpad.net

Description of the change

Adding the xml and html files to get a consolidated view of the kernel test run and
kernel build.

Please note that there are some repetitive feilds and needs some improvisation.
Feedback will help me to improve it further and its welcome.

Thanks and Regards,
Deepti.

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

This is good to land as is but needs some more subsequent work:

1) The query should be fine tuned, in particular ths e join between build results and test result should permit broken builds to be listed (without any test data) (the outer left join vs inner join)

2) The data view documentation and summary should have non-dummy values. Summary is a short one-line description that shows up in help text and various pages. Documentation is a longer free form text.

3) The report page should be extended to explain what is being observed, how to interpret the data, which bundle streams are being analyzed and how the data is being produced. This would make it a very worthwhile target for being displayed or linked to on the validation front page.

review: Approve

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