Ticket #1720 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

logging cleanup failure prevents session start

Reported by: quozl Owned by: tomeu
Priority: Unspecified by Maintainer Milestone: Unspecified by Release Team
Component: sugar Version: Git as of bugdate
Severity: Minor Keywords: r+
Cc: Distribution/OS: Unspecified
Bug Status: Unconfirmed

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

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

Change History

Changed 3 years ago by quozl

patch for review

Changed 3 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.

Changed 3 years ago by tomeu

  • keywords r+ added; r? removed
  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.