Merge lp:~xnox/utah/wubi-still-exists into lp:utah

Proposed by Dimitri John Ledkov
Status: Merged
Approved by: Javier Collado
Approved revision: 774
Merged at revision: 774
Proposed branch: lp:~xnox/utah/wubi-still-exists
Merge into: lp:utah
Diff against target: 15 lines (+2/-3)
1 file modified
utah/isotest/iso_static_validation.py (+2/-3)
To merge this branch: bzr merge lp:~xnox/utah/wubi-still-exists
Reviewer Review Type Date Requested Status
Max Brustkern (community) Approve
Javier Collado (community) Approve
Review via email: mp+138851@code.launchpad.net

Description of the change

Initially, raring images did not have wubi (it was not build for
raring yet). But wubi is now present on raring images and should be
tested for. Maybe this static check can be moved to a separate jenkins
job, such that WUBI fail, does not block all other downstream tests.

To post a comment you must log in.
Revision history for this message
Javier Collado (javier.collado) wrote :

Thanks for the changes. I've verified with today's distro for i386 and amd64
and it worked fine.

The idea about moving to a different job tests that can fail for testable
images is good. I'll open a bug about that to follow up.

review: Approve
Revision history for this message
Max Brustkern (nuclearbob) wrote :

I agree with the idea of having separate analysis that does not block other jobs. The job blocking was the reason this test was altered in the first place. :)

review: Approve
Revision history for this message
Javier Collado (javier.collado) wrote :

The bug I opened to follow the conversation about which tests cases should
block smoke testing and which ones should not is: 1088444

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'utah/isotest/iso_static_validation.py'
2--- utah/isotest/iso_static_validation.py 2012-12-06 16:30:34 +0000
3+++ utah/isotest/iso_static_validation.py 2012-12-08 02:20:25 +0000
4@@ -262,9 +262,8 @@
5
6 # Test if wubi is present for desktop or dvd and it is a valid PE
7 @unittest.skipUnless(st_variant == 'desktop' and
8- st_arch != 'powerpc' and st_release < 'raring',
9- "Skipping images that aren't ubiquity-based and "
10- "older than raring")
11+ st_arch != 'powerpc',
12+ "Skipping images that aren't ubiquity-based.")
13 def test_wubi(self):
14 logging.debug('Extracting wubi from the desktop images')
15 (stdout, stderr) = self.iso.extract('./wubi.exe', self.temp_dir)

Subscribers

People subscribed via source and target branches