[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#740673: apt-cdrom ident started requesting to insert cd even if cd is already mounted



Control: severity -1 grave

Rationale: breaks all cdimage builds.

Gabriele Giacone <1o5g4r8o@gmail.com> (2014-03-04):
> Package: apt
> Version: 0.9.15.5
> Severity: important
> Tags: d-i
> User: debian-hurd@lists.debian.org
> Usertags: hurd

FWIW this isn't hurd-specific.

> On hurd, "apt-cdrom ident" started requesting to insert cdrom even if cdrom is
> already mounted.
> That breaks debian-installer given it's called by load-install-cd.
> Recent debian-installer builds get stuck at "Configuring apt - Scanning the
> CD-ROM".
> See https://bugs.debian.org/728153

Weekly installation image builds are indeed broken; this is slightly
annoying since I was aiming at releasing an alpha 1 image for jessie
soon, so that we can perform regression tests against it.

On a wheezy amd64 installation image, d-i's syslog says:
| Mar  8 00:10:35 apt-setup: umount: can't umount /target/media/cdrom0: Invalid argument
| Mar  8 00:10:35 apt-setup: Using CD-ROM mount point /media/cdrom/
| Mar  8 00:10:35 apt-setup: Identifying.. 
| Mar  8 00:10:35 kernel: [   98.432287] ISO 9660 Extensions: Microsoft Joliet Level 3
| Mar  8 00:10:35 kernel: [   98.432799] ISO 9660 Extensions: RRIP_1991A
| Mar  8 00:10:35 apt-setup: [925dbb4f8c3f5332e76734f22eddc145-2]
| Mar  8 00:10:35 apt-setup: Scanning disc for index files..
| Mar  8 00:10:35 apt-setup: Found 1 package indexes, 0 source indexes, 1 translation indexes and 0 signatures
| Mar  8 00:10:35 apt-setup: This disc is called: 
| Mar  8 00:10:35 apt-setup: 'Debian GNU/Linux 7.4.0 _Wheezy_ - Official amd64 NETINST Binary-1 20140208-13:45'
| Mar  8 00:10:35 apt-setup: Copying package lists...
| Mar  8 00:10:35 apt-setup: ^MReading Package Indexes... 0%^M
| Mar  8 00:10:35 apt-setup: ^MReading Package Indexes... 1%^M
| Mar  8 00:10:35 apt-setup: ^MReading Package Indexes... Done^M
| Mar  8 00:10:35 apt-setup: 
| Mar  8 00:10:35 apt-setup: ^MReading Translation Indexes... 0%^M
| Mar  8 00:10:35 apt-setup: ^MReading Translation Indexes... Done^M
| Mar  8 00:10:35 apt-setup: 
| Mar  8 00:10:35 apt-setup: Writing new source list
| Mar  8 00:10:35 apt-setup: Source list entries for this disc are:
| Mar  8 00:10:35 apt-setup: deb cdrom:[Debian GNU/Linux 7.4.0 _Wheezy_ - Official amd64 NETINST Binary-1 20140208-13:45]/ wheezy main
| Mar  8 00:10:35 apt-setup: Repeat this process for the rest of the CDs in your set.
| Mar  8 00:10:35 kernel: [   98.611009] ISO 9660 Extensions: Microsoft Joliet Level 3
| Mar  8 00:10:35 kernel: [   98.611568] ISO 9660 Extensions: RRIP_1991A

and stuff continues. On a jessie weekly amd64 installation image (built
2014-03-03), one gets this instead:
| Mar  8 00:04:02 apt-setup: umount: can't umount /target/media/cdrom0: Invalid argument
| Mar  8 00:04:02 apt-setup: Using CD-ROM mount point /media/cdrom/
| Mar  8 00:04:02 apt-setup: Unmounting CD-ROM
| Mar  8 00:04:02 apt-setup: Waiting for disc...
| Mar  8 00:04:02 apt-setup: Please insert a Disc in the drive and press enter  
| Mar  8 00:04:02 apt-setup: 
| Mar  8 00:04:02 apt-setup: Mounting CD-ROM...
| Mar  8 00:04:02 apt-setup: Identifying.. 
| Mar  8 00:04:02 kernel: [  103.820796] ISO 9660 Extensions: Microsoft Joliet Level 3
| Mar  8 00:04:02 kernel: [  103.821442] ISO 9660 Extensions: RRIP_1991A
| Mar  8 00:04:02 apt-setup: [6c0eb3539a58e533556f373c12bee688-2]
| Mar  8 00:04:02 apt-setup: Scanning disc for index files..
| Mar  8 00:04:02 apt-setup: Found 1 package indexes, 0 source indexes, 1 translation indexes and 0 signatures
| Mar  8 00:04:02 apt-setup: This disc is called: 
| Mar  8 00:04:02 apt-setup: 'Debian GNU/Linux testing _Jessie_ - Official Snapshot amd64 NETINST Binary-1 20140303-06:00'
| Mar  8 00:04:02 apt-setup: Copying package lists...
| Mar  8 00:04:02 apt-setup: ^MReading Package Indexes... 0%^M
| Mar  8 00:04:02 apt-setup: ^MReading Package Indexes... 1%^M
| Mar  8 00:04:02 apt-setup: ^MReading Package Indexes... Done^M
| Mar  8 00:04:02 apt-setup: 
| Mar  8 00:04:02 apt-setup: ^MReading Translation Indexes... 0%^M
| Mar  8 00:04:02 apt-setup: ^MReading Translation Indexes... Done^M
| Mar  8 00:04:02 apt-setup: 
| Mar  8 00:04:02 apt-setup: Writing new source list 
| Mar  8 00:04:02 apt-setup: Source list entries for this disc are:
| Mar  8 00:04:02 apt-setup: deb cdrom:[Debian GNU/Linux testing _Jessie_ - Official Snapshot amd64 NETINST Binary-1 20140303-06:00]/ jessie main
| Mar  8 00:04:02 apt-setup: Unmounting CD-ROM... 
| Mar  8 00:04:02 apt-setup: Repeat this process for the rest of the CDs in your set.


Your changelog says:
|   [ John Ogness ]
|   * apt-cdrom should succeed if any drive succeeds (Closes: 728153)

And more precisely:
| Author: John Ogness <john.ogness@linutronix.de>
| Date:   Fri Dec 13 20:59:31 2013 +0100
| 
|     apt-cdrom should succeed if any drive succeeds
|     
|     If there are multiple CD-ROM drives, `apt-cdrom add` will abort with an
|     error if any of the drives do not contain a Debian CD which is against
|     the documentation we have saying "a CD-ROM" and also scripts do not
|     expect it this way.
|     
|     This patch modifies apt-cdrom to return success if any of the drives
|     succeeded. If failures occur, apt-cdrom will still continue trying all
|     the drives and report the last failure (if none of them succeeded).
|     
|     The 'ident' command was also changed to match the new 'add' behavior.
|     
|     Closes: 728153

I'm pretty sure making "apt-cdrom ident" hang wasn't part of the plan,
and that's what's happening nonetheless when called from apt-setup's
generators/40cdrom script.

[ Also, if you're going to change semantics, it probably would be nice
to warn your users (e.g. -boot@ in that particular case); heads-up on
topics with possible big consequences are always appreciated. ]

Mraw,
KiBi.

Attachment: signature.asc
Description: Digital signature


Reply to: