Setup a piuparts script to run automated packaging QA tests

Bug #497704 reported by Free Ekanayaka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Landscape Client
Fix Released
Medium
Free Ekanayaka

Bug Description

As part of our SRU QA procedure we want to make sure that
landscape-client packages can be installed, upgraded, removed and purged
cleanly.

Running piuparts against the candidate packages can automatically catch
problems in any of the operations above. We need a wrapper script to run
piuparts against a bzr branch and plug it into buildbot as a build step.

 affects landscape-client
 status inprogress
 importance medium
 assignee free.ekanayaka
 milestone 1.4.3

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

The attached branch implements a new dev/run-piuparts script which takes care of building the packages from the current branch, and run piuparts against them.

Note that to test this for real you'll need the latest pbuilder and piuparts packages from Karmic, earlier versions might not work, so backports would be needed. If so, please let me know if you need help with that.

tags: added: 1.5-sru-process review
Revision history for this message
Thomas Herve (therve) wrote :

[1] Apparently reloading dbus in the chroot crashes X. Using "pidof X" and aborting may be a solution.

[2] It may be nice to check the version of piuparts, too, as apparently only the lucid version works.

+1!

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

Thanks Thomas, [1] and [2] fixed!

Changed in landscape-client:
status: In Progress → Fix Committed
Changed in landscape-client:
status: Fix Committed → In Progress
Revision history for this message
Kevin McDermott (bigkevmcd) wrote :

#1

+The script checks if packaging changes (i.e. modifications debian/ files) have

s/modifications debian/modifications to debian/

#2

+# Workaround a bug in policykit not puring correctly

What does "not puring correctly" mean?

I can't test it, so if it works for you ;-) +1 with the above fixes.

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

So, it worked, but there were so many errors displayed during the whole process that it's difficult to say with confidence that everything was tested and is ok.

The end at least says this:
4m17.5s INFO: PASS: All tests.
4m17.5s INFO: piuparts run ends.

Jamu Kakar (jkakar)
tags: removed: review
Changed in landscape-client:
status: In Progress → Fix Committed
tags: added: needs-testing
tags: removed: needs-testing
Changed in landscape-client:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.