Opened 9 years ago

Closed 3 years ago

#1720 closed defect (fixed)

logging cleanup failure prevents session start

Reported by: quozl Owned by: tomeu
Priority: Normal Milestone: Unspecified
Component: Sugar Version: Git as of bugdate
Severity: Minor Keywords:
Cc: Distribution/OS:
Bug Status: New

Description

Logging cleanup creates a directory and moves previous logs into it. This requires available disk space.

When disk space is insufficient, or disk is read-only, or some other I/O error occurs, the sugar-session terminates. On systems that only exist to run sugar-session, the X server is repeatedly restarted. Here is an example from Sugar 0.84 with a full disk:

Traceback (most recent call last):
  File "/usr/bin/sugar-session", line 185, in <module>
    main()
  File "/usr/bin/sugar-session", line 141, in main
    cleanup_logs()
  File "/usr/bin/sugar-session", line 80, in cleanup_logs
    os.mkdir(backup_dir)
OSError: [Errno 28] No queda espacio en el dispositivo: '/home/olpc/.sugar/default/logs/1265854382'

(This example from dev.laptop.org #9623)

Attachments (1)

0001-ignore-logging-cleanup-fail-sugarlabs.org-1720-lapto.patch (2.0 KB) - added by quozl 9 years ago.
patch for review

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by quozl

patch for review

comment:1 Changed 9 years ago by quozl

  • Keywords r? added

This patch not tested.

|testcase|

Fill disk or change ownership of .sugar/default/logs to prevent write by user, then start session. Expected result: session will start. Current result: session fails to start, stderr of sugar-session process shows an IOError exception.

comment:2 Changed 9 years ago by tomeu

  • Keywords r+ added; r? removed
  • Resolution set to fixed
  • Status changed from new to closed

comment:3 Changed 4 years ago by quozl

  • Bug Status changed from Unconfirmed to New
  • Distribution/OS Unspecified deleted
  • Keywords r+ removed
  • Priority changed from Unspecified by Maintainer to Normal
  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:4 Changed 3 years ago by quozl

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.