TestDrive

unpleasant failure if iso is missing

Bug #491600 reported by Alan Pope 🍺🐧🐱 🦄
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
TestDrive
Fix Released
Low
Dustin Kirkland 
testdrive (Ubuntu)
Fix Released
Low
Dustin Kirkland 

Bug Description

Just tried to test the server iso, and got this:-

alan@wopr64:~$ testdrive
  1. Ubuntu Desktop (lucid)
     +-cache--> [2009-11-29 08:15:28] lucid-desktop-amd64.iso
  2. Ubuntu Server (lucid)
  3. Ubuntu Alternate (lucid)
  4. Ubuntu DVD (lucid)
  5. Ubuntu Netbook Remix (lucid)
     +-cache--> [2009-11-30 03:05:40] lucid-netbook-remix-i386.iso
  6. Other (prompt for ISO URL)

Select an ISO to testdrive: 2

WARNING: Your CPU's lack of VT will impact the performance of your VMs.
INFO: Using VirtualBox for virtual machine hosting...

INFO: Syncing the specified ISO...
      rsync://cdimage.ubuntu.com/cdimage/ubuntu-server/daily/current/lucid-server-amd64.iso
receiving file list ...
rsync: link_stat "/ubuntu-server/daily/current/lucid-server-amd64.iso" (in cdimage) failed: No such file or directory (2)
0 files to consider

sent 4 bytes received 17 bytes 42.00 bytes/sec
total size is 0 speedup is 0.00
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1523) [receiver=3.0.6]
alan@wopr64:~$

Visited http://cdimage.ubuntu.com/cdimage/ubuntu-server/daily/current/ and found there was no image. It might be preferable for testdrive to fail more gracefully when this error condition occurs.

Dustin Kirkland  (kirkland)
Changed in testdrive:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Hey Popey-

Yeah, I noticed this too... Daily cd images do seem to be broken at the moment.

Personally, I was able to glean this from the errors that rsync spit out. I suppose I could have testdrive print a generic error if the rsync|zsync|wget fails, but I think the details provided by the rsync|zsync|wget command should be relatively straightfoward, no?

Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :

Yeah, it just looked a bit ugly, that's all. I'm looking at this from a non-tech user point of view. Someone wanting to test drive an Ubuntu iso might give up after seeing that, not realising the image is missing but thinking something is broken at their end.

Maybe do an http HEAD on the file first to see if it's there. If not, give up with a nice error. If it is, start rsync/wget/whatever.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Hey popey, how do I "do an http HEAD on the file first"?

Happen to have a code snippet or hint?

Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :

curl -I could do this:-

alan@wopr64:~$ curl -I http://cdimage.ubuntu.com/cdimage/daily-live/current/lucid-desktop-amd64.iso 2>/dev/null | grep "HTTP/1.0 200 OK"
HTTP/1.0 200 OK
alan@wopr64:~$ curl -I http://cdimage.ubuntu.com/cdimage/daily-live/current/does_not_exist_lucid-desktop-amd64.iso 2>/dev/null | grep "HTTP/1.0 200 OK"
alan@wopr64:~$

Revision history for this message
Nafallo Bjälevik (nafallo) wrote :

You can also use wget -o /dev/stdout --spider -S http://cdimage.ubuntu.com/cdimage/daily-live/current/lucid-desktop-amd64.iso which removed the dependency on curl... but I also believe python has to have some nice standard way of doing this without having to resort to calling command line tools.

Dustin Kirkland  (kirkland)
Changed in testdrive:
status: Confirmed → Triaged
Dustin Kirkland  (kirkland)
Changed in testdrive:
status: Triaged → In Progress
assignee: nobody → Dustin Kirkland (kirkland)
Dustin Kirkland  (kirkland)
Changed in testdrive:
status: In Progress → Fix Committed
Changed in testdrive (Ubuntu):
status: New → Fix Committed
importance: Undecided → Low
assignee: nobody → Dustin Kirkland (kirkland)
Dustin Kirkland  (kirkland)
Changed in testdrive:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package testdrive - 1.22-0ubuntu1

---------------
testdrive (1.22-0ubuntu1) lucid; urgency=low

  * testdrive:
    - fix run() return value checking, LP: #525438
    - more graceful error if ISO url 404's, LP: #491600
    - remove unnecessary warning when running VB, LP: #486884
    - offer to launch usb-creator after running VM, LP: #507537
    - add a -v|--version option, LP: #526548
    - unconditional print message about cleaning cache, LP: #526546
    - fix typo, LP: #526570
    - clean up output and prompt at the end of a desktop run
  * testdrive-select-iso, doc/testdrive-select-iso.1, debian/install,
    debian/rules:
    - move this file into testdrive, now that testdrive is in python too
  * testdriverc: quote sample CACHE value, LP: #524241
  * debian/control:
    - allow for non-free virtualbox in package dependencies, LP: #524704
    - drop suggests of testdrivegtk package, LP: #526490
 -- Dustin Kirkland <email address hidden> Thu, 18 Feb 2010 22:31:34 -0600

Changed in testdrive (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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