disk_io script doesn't work, strips last digit of device

Bug #572855 reported by Johan van Dijk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu
Fix Released
Medium
Dustin Kirkland 
byobu (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

The /usr/lib/byobu/disk_io script doesn't work on my computer.

In line 29, the last digit(s) of the device name are stripped.
In normal cases (/dev/sdaX, /dev/hdaX, etc.) this works, but when you try to monitor a MDADM device (/dev/mdX) this fails.
It tries to read /sys/block/md/stat which doesn't exist. It should read /sys/block/md1/stat instead.

Is it possible to change the script, so it doesn't strip the last digits when monitoring a mdadm device?

apt-cache policy byobu
byobu:
  Installed: 2.73-0ubuntu1~ppa4
  Candidate: 2.73-0ubuntu1~ppa4
  Version table:
 *** 2.73-0ubuntu1~ppa4 0
        500 http://ppa.launchpad.net karmic/main Packages
        100 /var/lib/dpkg/status
     2.38-0ubuntu3 0
        500 http://nl.archive.ubuntu.com karmic/main Packages

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

Thanks for the report!

I'll get this fixed soon.

Changed in byobu:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Dustin Kirkland (kirkland)
Changed in byobu:
status: Triaged → Fix Committed
Changed in byobu (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package byobu - 2.74-0ubuntu1

---------------
byobu (2.74-0ubuntu1) maverick; urgency=low

  * usr/lib/byobu/disk_io: fix regex to support RAID devices, LP: #572855
  * debian/postinst: make debconf question failsafe, LP: #569041
  * usr/bin/byobu, usr/share/byobu/profiles/byoburc: add support for
    different window sets, LP: #517796
  * usr/bin/byobu-select-session: use screen -ls, less overhead
  * usr/share/man/man1/byobu.1: document the different window sets

byobu (2.73-0ubuntu1) maverick; urgency=low

  [ Dustin Kirkland ]
  * usr/bin/byobu-reconnect-sockets: look for gpg sessions quietly
  * usr/bin/byobu-launcher: ensure that if user does not want a nested
    session, they still get a local shell

  [ Jon Bernard ]
  * etc/byobu/statusrc, usr/bin/byobu-config, usr/lib/byobu/rcs_cost,
    usr/share/byobu/profiles/common, usr/share/man/man1/byobu.1:
    - Rackspace cost estimation (based on the ec2 one)

byobu (2.72-0ubuntu1) maverick; urgency=low

  * usr/bin/byobu: support nethack screen mode, LP: #568751
  * usr/share/man/man1/byobu.1: document a sample custom script using
    custom colors, LP: #568967
  * usr/bin/byobu: LANG=C before grepping for text
  * usr/bin/byobu: handle unset term in tput 256 color check
  * bin/byobu-config, lib/byobu/disk_io, share/byobu/profiles/common,
    share/man/man1/byobu.1: added a disk throughput notification
  * usr/bin/byobu-janitor: add hyphenation to the regex to allow for
    disk_io
  * usr/lib/byobu/disk_io: improve MONITORED_DISK overriding
  * usr/lib/byobu/disk: minor simplification

byobu (2.71-0ubuntu1) maverick; urgency=low

  * usr/bin/byobu-launch, usr/bin/byobu-launcher-install,
    usr/bin/byobu-janitor: exit 0 after launching byobu; this is superior
    to previously reverted implementations where we exec'd byobu, since
    we'll only exit 0 if the byobu/screen session exits cleanly; this will
    keep the user from having to do the double-exit when detaching,
    LP: #568306

byobu (2.70-0ubuntu1) maverick; urgency=low

  * usr/bin/byobu-launcher: fix regression in last upload

byobu (2.69-0ubuntu1) lucid; urgency=low

  * usr/bin/byobu-launcher: prompt on nested screen connection, LP: #565398
  * debian/po/POTFILES.in, debian/po/de.po, debian/po/en_GB.po,
    debian/po/es.po, debian/po/pt.po, debian/po/ru.po,
    debian/po/templates.pot, po/bg.po, po/cs.po, po/de.po, po/es.po,
    po/fr.po, po/hu.po, po/is.po, po/it.po, po/la.po, po/lt.po, po/nl.po,
    po/pt.po, po/pt_BR.po, po/ru.po, po/sv.po, po/tr.po, po/uk.po:
    - translations merged, LP: #567309
  * bin/byobu-janitor, share/byobu/keybindings/f-keys,
    share/byobu/profiles/common, share/man/man1/byobu.1: add a keybinding,
    ctrl-a ~ to write the current window's buffer to a file, LP: #565399
  * debian/control, debian/screen-profiles.postinst, debian/preinst,
    debian/prerm, debian/rules, usr/bin/byobu-janitor:
    - remove more remnants of screen-profiles
  * debian/rules: drop color symlinks, no longer necessary, should be
    cleaned up by byobu-janitor now
 -- Dustin Kirkland <email address hidden> Wed, 28 Apr 2010 23:11:14 -0500

Changed in byobu (Ubuntu):
status: Fix Committed → Fix Released
Changed in byobu:
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.