~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/lockdir.py

  • Committer: Martin
  • Date: 2011-08-04 00:17:53 UTC
  • mto: This revision was merged to the branch mainline in revision 6055.
  • Revision ID: gzlist@googlemail.com-20110804001753-plgpwcpsxcum16yb
Make tests raising KnownFailure use the knownFailure method instead

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2006-2011 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
92
92
>>> # do something here
93
93
>>> l.unlock()
94
94
 
 
95
Some classes of stale locks can be predicted by checking: the host name is the
 
96
same as the local host name; the user name is the same as the local user; the
 
97
process id no longer exists.  The check on user name is not strictly necessary
 
98
but helps protect against colliding host names.
95
99
"""
96
100
 
97
101
 
103
107
# the existing locking code and needs a new format of the containing object.
104
108
# -- robertc, mbp 20070628
105
109
 
 
110
import errno
106
111
import os
107
112
import time
108
113
 
109
114
from bzrlib import (
 
115
    config,
110
116
    debug,
111
117
    errors,
112
118
    lock,
113
119
    osutils,
 
120
    ui,
 
121
    urlutils,
114
122
    )
115
 
import bzrlib.config
116
123
from bzrlib.decorators import only_raises
117
124
from bzrlib.errors import (
118
125
        DirectoryNotEmpty,
120
127
        LockBreakMismatch,
121
128
        LockBroken,
122
129
        LockContention,
 
130
        LockCorrupt,
123
131
        LockFailed,
124
132
        LockNotHeld,
125
133
        NoSuchFile,
129
137
        )
130
138
from bzrlib.trace import mutter, note
131
139
from bzrlib.osutils import format_delta, rand_chars, get_host_name
132
 
import bzrlib.ui
133
140
 
134
141
from bzrlib.lazy_import import lazy_import
135
142
lazy_import(globals(), """
151
158
# files/dirs created.
152
159
 
153
160
 
154
 
_DEFAULT_TIMEOUT_SECONDS = 300
 
161
_DEFAULT_TIMEOUT_SECONDS = 30
155
162
_DEFAULT_POLL_SECONDS = 1.0
156
163
 
157
164
 
161
168
 
162
169
    __INFO_NAME = '/info'
163
170
 
164
 
    def __init__(self, transport, path, file_modebits=0644, dir_modebits=0755):
 
171
    def __init__(self, transport, path, file_modebits=0644, dir_modebits=0755,
 
172
        extra_holder_info=None):
165
173
        """Create a new LockDir object.
166
174
 
167
175
        The LockDir is initially unlocked - this just creates the object.
170
178
 
171
179
        :param path: Path to the lock within the base directory of the
172
180
            transport.
 
181
 
 
182
        :param extra_holder_info: If passed, {str:str} dict of extra or
 
183
            updated information to insert into the info file when the lock is
 
184
            taken.
173
185
        """
174
186
        self.transport = transport
175
187
        self.path = path
180
192
        self._held_info_path = self._held_dir + self.__INFO_NAME
181
193
        self._file_modebits = file_modebits
182
194
        self._dir_modebits = dir_modebits
183
 
 
184
195
        self._report_function = note
 
196
        self.extra_holder_info = extra_holder_info
 
197
        self._warned_about_lock_holder = None
185
198
 
186
199
    def __repr__(self):
187
200
        return '%s(%s%s)' % (self.__class__.__name__,
202
215
        except (TransportError, PathError), e:
203
216
            raise LockFailed(self, e)
204
217
 
205
 
 
206
218
    def _attempt_lock(self):
207
219
        """Make the pending directory and attempt to rename into place.
208
220
 
215
227
 
216
228
        :returns: The nonce of the lock, if it was successfully acquired.
217
229
 
218
 
        :raises LockContention: If the lock is held by someone else.  The exception
219
 
            contains the info of the current holder of the lock.
 
230
        :raises LockContention: If the lock is held by someone else.  The
 
231
            exception contains the info of the current holder of the lock.
220
232
        """
221
233
        self._trace("lock_write...")
222
234
        start_time = time.time()
225
237
        except (errors.TransportError, PathError), e:
226
238
            self._trace("... failed to create pending dir, %s", e)
227
239
            raise LockFailed(self, e)
228
 
        try:
229
 
            self.transport.rename(tmpname, self._held_dir)
230
 
        except (errors.TransportError, PathError, DirectoryNotEmpty,
231
 
                FileExists, ResourceBusy), e:
232
 
            self._trace("... contention, %s", e)
233
 
            self._remove_pending_dir(tmpname)
234
 
            raise LockContention(self)
235
 
        except Exception, e:
236
 
            self._trace("... lock failed, %s", e)
237
 
            self._remove_pending_dir(tmpname)
238
 
            raise
 
240
        while True:
 
241
            try:
 
242
                self.transport.rename(tmpname, self._held_dir)
 
243
                break
 
244
            except (errors.TransportError, PathError, DirectoryNotEmpty,
 
245
                    FileExists, ResourceBusy), e:
 
246
                self._trace("... contention, %s", e)
 
247
                other_holder = self.peek()
 
248
                self._trace("other holder is %r" % other_holder)
 
249
                try:
 
250
                    self._handle_lock_contention(other_holder)
 
251
                except:
 
252
                    self._remove_pending_dir(tmpname)
 
253
                    raise
 
254
            except Exception, e:
 
255
                self._trace("... lock failed, %s", e)
 
256
                self._remove_pending_dir(tmpname)
 
257
                raise
239
258
        # We must check we really got the lock, because Launchpad's sftp
240
259
        # server at one time had a bug were the rename would successfully
241
260
        # move the new directory into the existing directory, which was
243
262
        # have a similar bug allowing someone to think they got the lock
244
263
        # when it's already held.
245
264
        #
246
 
        # See <https://bugs.edge.launchpad.net/bzr/+bug/498378> for one case.
 
265
        # See <https://bugs.launchpad.net/bzr/+bug/498378> for one case.
247
266
        #
248
267
        # Strictly the check is unnecessary and a waste of time for most
249
268
        # people, but probably worth trapping if something is wrong.
261
280
                (time.time() - start_time) * 1000)
262
281
        return self.nonce
263
282
 
 
283
    def _handle_lock_contention(self, other_holder):
 
284
        """A lock we want to take is held by someone else.
 
285
 
 
286
        This function can: tell the user about it; possibly detect that it's
 
287
        safe or appropriate to steal the lock, or just raise an exception.
 
288
 
 
289
        If this function returns (without raising an exception) the lock will
 
290
        be attempted again.
 
291
 
 
292
        :param other_holder: A LockHeldInfo for the current holder; note that
 
293
            it might be None if the lock can be seen to be held but the info
 
294
            can't be read.
 
295
        """
 
296
        if (other_holder is not None):
 
297
            if other_holder.is_lock_holder_known_dead():
 
298
                if self.get_config().get_user_option_as_bool(
 
299
                    'locks.steal_dead',
 
300
                    default=False):
 
301
                    ui.ui_factory.show_user_warning(
 
302
                        'locks_steal_dead',
 
303
                        lock_url=urlutils.join(self.transport.base, self.path),
 
304
                        other_holder_info=unicode(other_holder))
 
305
                    self.force_break(other_holder)
 
306
                    self._trace("stole lock from dead holder")
 
307
                    return
 
308
        raise LockContention(self)
 
309
 
264
310
    def _remove_pending_dir(self, tmpname):
265
311
        """Remove the pending directory
266
312
 
286
332
            self.create(mode=self._dir_modebits)
287
333
            # After creating the lock directory, try again
288
334
            self.transport.mkdir(tmpname)
289
 
        self.nonce = rand_chars(20)
290
 
        info_bytes = self._prepare_info()
 
335
        info = LockHeldInfo.for_this_process(self.extra_holder_info)
 
336
        self.nonce = info.get('nonce')
291
337
        # We use put_file_non_atomic because we just created a new unique
292
338
        # directory so we don't have to worry about files existing there.
293
339
        # We'll rename the whole directory into place to get atomic
294
340
        # properties
295
341
        self.transport.put_bytes_non_atomic(tmpname + self.__INFO_NAME,
296
 
                                            info_bytes)
 
342
            info.to_bytes())
297
343
        return tmpname
298
344
 
299
345
    @only_raises(LockNotHeld, LockBroken)
343
389
    def break_lock(self):
344
390
        """Break a lock not held by this instance of LockDir.
345
391
 
346
 
        This is a UI centric function: it uses the bzrlib.ui.ui_factory to
 
392
        This is a UI centric function: it uses the ui.ui_factory to
347
393
        prompt for input if a lock is detected and there is any doubt about
348
 
        it possibly being still active.
 
394
        it possibly being still active.  force_break is the non-interactive
 
395
        version.
 
396
 
 
397
        :returns: LockResult for the broken lock.
349
398
        """
350
399
        self._check_not_locked()
351
 
        holder_info = self.peek()
 
400
        try:
 
401
            holder_info = self.peek()
 
402
        except LockCorrupt, e:
 
403
            # The lock info is corrupt.
 
404
            if ui.ui_factory.get_boolean(u"Break (corrupt %r)" % (self,)):
 
405
                self.force_break_corrupt(e.file_data)
 
406
            return
352
407
        if holder_info is not None:
353
 
            lock_info = '\n'.join(self._format_lock_info(holder_info))
354
 
            if bzrlib.ui.ui_factory.get_boolean("Break %s" % lock_info):
355
 
                self.force_break(holder_info)
 
408
            if ui.ui_factory.confirm_action(
 
409
                u"Break %(lock_info)s",
 
410
                'bzrlib.lockdir.break',
 
411
                dict(lock_info=unicode(holder_info))):
 
412
                result = self.force_break(holder_info)
 
413
                ui.ui_factory.show_message(
 
414
                    "Broke lock %s" % result.lock_url)
356
415
 
357
416
    def force_break(self, dead_holder_info):
358
417
        """Release a lock held by another process.
361
420
        it still thinks it has the lock there will be two concurrent writers.
362
421
        In general the user's approval should be sought for lock breaks.
363
422
 
364
 
        dead_holder_info must be the result of a previous LockDir.peek() call;
365
 
        this is used to check that it's still held by the same process that
366
 
        the user decided was dead.  If this is not the current holder,
367
 
        LockBreakMismatch is raised.
368
 
 
369
423
        After the lock is broken it will not be held by any process.
370
424
        It is possible that another process may sneak in and take the
371
425
        lock before the breaking process acquires it.
 
426
 
 
427
        :param dead_holder_info:
 
428
            Must be the result of a previous LockDir.peek() call; this is used
 
429
            to check that it's still held by the same process that the user
 
430
            decided was dead.  If this is not the current holder,
 
431
            LockBreakMismatch is raised.
 
432
 
 
433
        :returns: LockResult for the broken lock.
372
434
        """
373
 
        if not isinstance(dead_holder_info, dict):
 
435
        if not isinstance(dead_holder_info, LockHeldInfo):
374
436
            raise ValueError("dead_holder_info: %r" % dead_holder_info)
375
437
        self._check_not_locked()
376
438
        current_info = self.peek()
394
456
                                 current_info.get('nonce'))
395
457
        for hook in self.hooks['lock_broken']:
396
458
            hook(result)
 
459
        return result
 
460
 
 
461
    def force_break_corrupt(self, corrupt_info_lines):
 
462
        """Release a lock that has been corrupted.
 
463
 
 
464
        This is very similar to force_break, it except it doesn't assume that
 
465
        self.peek() can work.
 
466
 
 
467
        :param corrupt_info_lines: the lines of the corrupted info file, used
 
468
            to check that the lock hasn't changed between reading the (corrupt)
 
469
            info file and calling force_break_corrupt.
 
470
        """
 
471
        # XXX: this copes with unparseable info files, but what about missing
 
472
        # info files?  Or missing lock dirs?
 
473
        self._check_not_locked()
 
474
        tmpname = '%s/broken.%s.tmp' % (self.path, rand_chars(20))
 
475
        self.transport.rename(self._held_dir, tmpname)
 
476
        # check that we actually broke the right lock, not someone else;
 
477
        # there's a small race window between checking it and doing the
 
478
        # rename.
 
479
        broken_info_path = tmpname + self.__INFO_NAME
 
480
        broken_content = self.transport.get_bytes(broken_info_path)
 
481
        broken_lines = osutils.split_lines(broken_content)
 
482
        if broken_lines != corrupt_info_lines:
 
483
            raise LockBreakMismatch(self, broken_lines, corrupt_info_lines)
 
484
        self.transport.delete(broken_info_path)
 
485
        self.transport.rmdir(tmpname)
 
486
        result = lock.LockResult(self.transport.abspath(self.path))
 
487
        for hook in self.hooks['lock_broken']:
 
488
            hook(result)
397
489
 
398
490
    def _check_not_locked(self):
399
491
        """If the lock is held by this instance, raise an error."""
425
517
 
426
518
        peek() reads the info file of the lock holder, if any.
427
519
        """
428
 
        return self._parse_info(self.transport.get_bytes(path))
 
520
        return LockHeldInfo.from_info_file_bytes(
 
521
            self.transport.get_bytes(path))
429
522
 
430
523
    def peek(self):
431
524
        """Check if the lock is held by anyone.
444
537
    def _prepare_info(self):
445
538
        """Write information about a pending lock to a temporary file.
446
539
        """
447
 
        # XXX: is creating this here inefficient?
448
 
        config = bzrlib.config.GlobalConfig()
449
 
        try:
450
 
            user = config.user_email()
451
 
        except errors.NoEmailInUsername:
452
 
            user = config.username()
453
 
        s = rio.Stanza(hostname=get_host_name(),
454
 
                   pid=str(os.getpid()),
455
 
                   start_time=str(int(time.time())),
456
 
                   nonce=self.nonce,
457
 
                   user=user,
458
 
                   )
459
 
        return s.to_string()
460
 
 
461
 
    def _parse_info(self, info_bytes):
462
 
        stanza = rio.read_stanza(osutils.split_lines(info_bytes))
463
 
        if stanza is None:
464
 
            # see bug 185013; we fairly often end up with the info file being
465
 
            # empty after an interruption; we could log a message here but
466
 
            # there may not be much we can say
467
 
            return {}
468
 
        else:
469
 
            return stanza.as_dict()
470
540
 
471
541
    def attempt_lock(self):
472
542
        """Take the lock; fail if it's already held.
486
556
            hook(hook_result)
487
557
        return result
488
558
 
 
559
    def lock_url_for_display(self):
 
560
        """Give a nicely-printable representation of the URL of this lock."""
 
561
        # As local lock urls are correct we display them.
 
562
        # We avoid displaying remote lock urls.
 
563
        lock_url = self.transport.abspath(self.path)
 
564
        if lock_url.startswith('file://'):
 
565
            lock_url = lock_url.split('.bzr/')[0]
 
566
        else:
 
567
            lock_url = ''
 
568
        return lock_url
 
569
 
489
570
    def wait_lock(self, timeout=None, poll=None, max_attempts=None):
490
571
        """Wait a certain period for a lock.
491
572
 
515
596
        deadline_str = None
516
597
        last_info = None
517
598
        attempt_count = 0
 
599
        lock_url = self.lock_url_for_display()
518
600
        while True:
519
601
            attempt_count += 1
520
602
            try:
535
617
                else:
536
618
                    start = 'Lock owner changed for'
537
619
                last_info = new_info
538
 
                formatted_info = self._format_lock_info(new_info)
 
620
                msg = u'%s lock %s %s.' % (start, lock_url, new_info)
539
621
                if deadline_str is None:
540
622
                    deadline_str = time.strftime('%H:%M:%S',
541
 
                                                 time.localtime(deadline))
542
 
                lock_url = self.transport.abspath(self.path)
543
 
                # See <https://bugs.edge.launchpad.net/bzr/+bug/250451>
544
 
                # the URL here is sometimes not one that is useful to the
545
 
                # user, perhaps being wrapped in a lp-%d or chroot decorator,
546
 
                # especially if this error is issued from the server.
547
 
                self._report_function('%s %s\n'
548
 
                    '%s\n' # held by
549
 
                    '%s\n' # locked ... ago
550
 
                    'Will continue to try until %s, unless '
551
 
                    'you press Ctrl-C.\n'
552
 
                    'See "bzr help break-lock" for more.',
553
 
                    start,
554
 
                    formatted_info[0],
555
 
                    formatted_info[1],
556
 
                    formatted_info[2],
557
 
                    deadline_str,
558
 
                    )
559
 
 
 
623
                                                    time.localtime(deadline))
 
624
                if timeout > 0:
 
625
                    msg += ('\nWill continue to try until %s, unless '
 
626
                        'you press Ctrl-C.'
 
627
                        % deadline_str)
 
628
                msg += '\nSee "bzr help break-lock" for more.'
 
629
                self._report_function(msg)
560
630
            if (max_attempts is not None) and (attempt_count >= max_attempts):
561
631
                self._trace("exceeded %d attempts")
562
632
                raise LockContention(self)
564
634
                self._trace("waiting %ss", poll)
565
635
                time.sleep(poll)
566
636
            else:
 
637
                # As timeout is always 0 for remote locks
 
638
                # this block is applicable only for local
 
639
                # lock contention
567
640
                self._trace("timeout after waiting %ss", timeout)
568
 
                raise LockContention(self)
 
641
                raise LockContention('(local)', lock_url)
569
642
 
570
643
    def leave_in_place(self):
571
644
        self._locked_via_token = True
614
687
            raise LockContention(self)
615
688
        self._fake_read_lock = True
616
689
 
617
 
    def _format_lock_info(self, info):
618
 
        """Turn the contents of peek() into something for the user"""
619
 
        lock_url = self.transport.abspath(self.path)
620
 
        start_time = info.get('start_time')
621
 
        if start_time is None:
622
 
            time_ago = '(unknown)'
623
 
        else:
624
 
            time_ago = format_delta(time.time() - int(info['start_time']))
625
 
        return [
626
 
            'lock %s' % (lock_url,),
627
 
            'held by %s on host %s [process #%s]' %
628
 
                tuple([info.get(x, '<unknown>') for x in ['user', 'hostname', 'pid']]),
629
 
            'locked %s' % (time_ago,),
630
 
            ]
631
 
 
632
690
    def validate_token(self, token):
633
691
        if token is not None:
634
692
            info = self.peek()
646
704
        if 'lock' not in debug.debug_flags:
647
705
            return
648
706
        mutter(str(self) + ": " + (format % args))
 
707
 
 
708
    def get_config(self):
 
709
        """Get the configuration that governs this lockdir."""
 
710
        # XXX: This really should also use the locationconfig at least, but
 
711
        # that seems a bit hard to hook up at the moment. -- mbp 20110329
 
712
        return config.GlobalConfig()
 
713
 
 
714
 
 
715
class LockHeldInfo(object):
 
716
    """The information recorded about a held lock.
 
717
 
 
718
    This information is recorded into the lock when it's taken, and it can be
 
719
    read back by any process with access to the lockdir.  It can be used, for
 
720
    example, to tell the user who holds the lock, or to try to detect whether
 
721
    the lock holder is still alive.
 
722
 
 
723
    Prior to bzr 2.4 a simple dict was used instead of an object.
 
724
    """
 
725
 
 
726
    def __init__(self, info_dict):
 
727
        self.info_dict = info_dict
 
728
 
 
729
    def __repr__(self):
 
730
        """Return a debugging representation of this object."""
 
731
        return "%s(%r)" % (self.__class__.__name__, self.info_dict)
 
732
 
 
733
    def __unicode__(self):
 
734
        """Return a user-oriented description of this object."""
 
735
        d = self.to_readable_dict()
 
736
        return (
 
737
            u'held by %(user)s on %(hostname)s (process #%(pid)s), '
 
738
            u'acquired %(time_ago)s' % d)
 
739
 
 
740
    def to_readable_dict(self):
 
741
        """Turn the holder info into a dict of human-readable attributes.
 
742
 
 
743
        For example, the start time is presented relative to the current time,
 
744
        rather than as seconds since the epoch.
 
745
 
 
746
        Returns a list of [user, hostname, pid, time_ago] all as readable
 
747
        strings.
 
748
        """
 
749
        start_time = self.info_dict.get('start_time')
 
750
        if start_time is None:
 
751
            time_ago = '(unknown)'
 
752
        else:
 
753
            time_ago = format_delta(
 
754
                time.time() - int(self.info_dict['start_time']))
 
755
        user = self.info_dict.get('user', '<unknown>')
 
756
        hostname = self.info_dict.get('hostname', '<unknown>')
 
757
        pid = self.info_dict.get('pid', '<unknown>')
 
758
        return dict(
 
759
            user=user,
 
760
            hostname=hostname,
 
761
            pid=pid,
 
762
            time_ago=time_ago)
 
763
 
 
764
    def get(self, field_name):
 
765
        """Return the contents of a field from the lock info, or None."""
 
766
        return self.info_dict.get(field_name)
 
767
 
 
768
    @classmethod
 
769
    def for_this_process(cls, extra_holder_info):
 
770
        """Return a new LockHeldInfo for a lock taken by this process.
 
771
        """
 
772
        info = dict(
 
773
            hostname=get_host_name(),
 
774
            pid=str(os.getpid()),
 
775
            nonce=rand_chars(20),
 
776
            start_time=str(int(time.time())),
 
777
            user=get_username_for_lock_info(),
 
778
            )
 
779
        if extra_holder_info is not None:
 
780
            info.update(extra_holder_info)
 
781
        return cls(info)
 
782
 
 
783
    def to_bytes(self):
 
784
        s = rio.Stanza(**self.info_dict)
 
785
        return s.to_string()
 
786
 
 
787
    @classmethod
 
788
    def from_info_file_bytes(cls, info_file_bytes):
 
789
        """Construct from the contents of the held file."""
 
790
        lines = osutils.split_lines(info_file_bytes)
 
791
        try:
 
792
            stanza = rio.read_stanza(lines)
 
793
        except ValueError, e:
 
794
            mutter('Corrupt lock info file: %r', lines)
 
795
            raise LockCorrupt("could not parse lock info file: " + str(e),
 
796
                lines)
 
797
        if stanza is None:
 
798
            # see bug 185013; we fairly often end up with the info file being
 
799
            # empty after an interruption; we could log a message here but
 
800
            # there may not be much we can say
 
801
            return cls({})
 
802
        else:
 
803
            return cls(stanza.as_dict())
 
804
 
 
805
    def __cmp__(self, other):
 
806
        """Value comparison of lock holders."""
 
807
        return (
 
808
            cmp(type(self), type(other))
 
809
            or cmp(self.info_dict, other.info_dict))
 
810
 
 
811
    def is_locked_by_this_process(self):
 
812
        """True if this process seems to be the current lock holder."""
 
813
        return (
 
814
            self.get('hostname') == get_host_name()
 
815
            and self.get('pid') == str(os.getpid())
 
816
            and self.get('user') == get_username_for_lock_info())
 
817
 
 
818
    def is_lock_holder_known_dead(self):
 
819
        """True if the lock holder process is known to be dead.
 
820
 
 
821
        False if it's either known to be still alive, or if we just can't tell.
 
822
 
 
823
        We can be fairly sure the lock holder is dead if it declared the same
 
824
        hostname and there is no process with the given pid alive.  If people
 
825
        have multiple machines with the same hostname this may cause trouble.
 
826
 
 
827
        This doesn't check whether the lock holder is in fact the same process
 
828
        calling this method.  (In that case it will return true.)
 
829
        """
 
830
        if self.get('hostname') != get_host_name():
 
831
            return False
 
832
        if self.get('hostname') == 'localhost':
 
833
            # Too ambiguous.
 
834
            return False
 
835
        if self.get('user') != get_username_for_lock_info():
 
836
            # Could well be another local process by a different user, but
 
837
            # just to be safe we won't conclude about this either.
 
838
            return False
 
839
        pid_str = self.info_dict.get('pid', None)
 
840
        if not pid_str:
 
841
            mutter("no pid recorded in %r" % (self, ))
 
842
            return False
 
843
        try:
 
844
            pid = int(pid_str)
 
845
        except ValueError:
 
846
            mutter("can't parse pid %r from %r"
 
847
                % (pid_str, self))
 
848
            return False
 
849
        return osutils.is_local_pid_dead(pid)
 
850
 
 
851
 
 
852
def get_username_for_lock_info():
 
853
    """Get a username suitable for putting into a lock.
 
854
 
 
855
    It's ok if what's written here is not a proper email address as long
 
856
    as it gives some clue who the user is.
 
857
    """
 
858
    try:
 
859
        return config.GlobalConfig().username()
 
860
    except errors.NoWhoami:
 
861
        return osutils.getuser_unicode()