holland advisory lock failures

Bug #1184145 reported by Andrew Garner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
holland-backup
Fix Released
High
Andrew Garner

Bug Description

holland 1.0.8 introduced a bug when two holland process try to run the same backupset simultaneously. This results in an ugly stacktrace along the lines of:

2013-05-22 21:27:59,380 [ERROR] Uncaught exception while running command 'backup': <exceptions.AttributeError instance at 0x1d898998>
Traceback (most recent call last):
  File "/usr/lib/python2.4/site-packages/holland/core/command/command.py", line 209, in dispatch
    return self.run(self.optparser.prog, opts, *args)
  File "/usr/lib/python2.4/site-packages/holland/commands/backup.py", line 95, in run
    lock.path, lock.lock.name)
AttributeError: 'NoneType' object has no attribute 'name'

Andrew Garner (muzazzi)
Changed in holland-backup:
milestone: none → 1.0.10
Revision history for this message
Andrew Garner (muzazzi) wrote :

This was introduced when the lock messages were made a little more user friendly - but obviously there isn't a test for this.

Fixed here:

https://github.com/holland-backup/holland/commit/aeeff5ad92246c1f33a6496c278cbcd37792a89b

The proper output here should be:

Holland 1.0.8 started with pid 26836
--- Starting backup run ---
Another holland backup process is already running backupset 'default'. Aborting.
--- Ending backup run ---

This needs to be added to the test suite.

Changed in holland-backup:
status: New → Triaged
importance: Undecided → High
assignee: nobody → Andrew Garner (muzazzi)
status: Triaged → Fix Committed
Andrew Garner (muzazzi)
Changed in holland-backup:
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.