~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/lockdir.py

  • Committer: Vincent Ladeuil
  • Date: 2011-09-09 13:30:12 UTC
  • mfrom: (5609.48.11 2.3)
  • mto: (6015.33.3 2.4)
  • mto: This revision was merged to the branch mainline in revision 6134.
  • Revision ID: v.ladeuil+lp@free.fr-20110909133012-jc1d1zyqgak57123
Merge 2.3 into 2.4

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006, 2007, 2008 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,
 
119
    osutils,
 
120
    ui,
 
121
    urlutils,
113
122
    )
114
 
import bzrlib.config
 
123
from bzrlib.decorators import only_raises
115
124
from bzrlib.errors import (
116
125
        DirectoryNotEmpty,
117
126
        FileExists,
118
127
        LockBreakMismatch,
119
128
        LockBroken,
120
129
        LockContention,
 
130
        LockCorrupt,
121
131
        LockFailed,
122
132
        LockNotHeld,
123
133
        NoSuchFile,
127
137
        )
128
138
from bzrlib.trace import mutter, note
129
139
from bzrlib.osutils import format_delta, rand_chars, get_host_name
130
 
import bzrlib.ui
131
140
 
132
141
from bzrlib.lazy_import import lazy_import
133
142
lazy_import(globals(), """
149
158
# files/dirs created.
150
159
 
151
160
 
152
 
_DEFAULT_TIMEOUT_SECONDS = 300
 
161
_DEFAULT_TIMEOUT_SECONDS = 30
153
162
_DEFAULT_POLL_SECONDS = 1.0
154
163
 
155
164
 
159
168
 
160
169
    __INFO_NAME = '/info'
161
170
 
162
 
    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):
163
173
        """Create a new LockDir object.
164
174
 
165
175
        The LockDir is initially unlocked - this just creates the object.
168
178
 
169
179
        :param path: Path to the lock within the base directory of the
170
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.
171
185
        """
172
186
        self.transport = transport
173
187
        self.path = path
178
192
        self._held_info_path = self._held_dir + self.__INFO_NAME
179
193
        self._file_modebits = file_modebits
180
194
        self._dir_modebits = dir_modebits
181
 
 
182
195
        self._report_function = note
 
196
        self.extra_holder_info = extra_holder_info
 
197
        self._warned_about_lock_holder = None
183
198
 
184
199
    def __repr__(self):
185
200
        return '%s(%s%s)' % (self.__class__.__name__,
200
215
        except (TransportError, PathError), e:
201
216
            raise LockFailed(self, e)
202
217
 
203
 
 
204
218
    def _attempt_lock(self):
205
219
        """Make the pending directory and attempt to rename into place.
206
220
 
213
227
 
214
228
        :returns: The nonce of the lock, if it was successfully acquired.
215
229
 
216
 
        :raises LockContention: If the lock is held by someone else.  The exception
217
 
            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.
218
232
        """
219
233
        self._trace("lock_write...")
220
234
        start_time = time.time()
223
237
        except (errors.TransportError, PathError), e:
224
238
            self._trace("... failed to create pending dir, %s", e)
225
239
            raise LockFailed(self, e)
226
 
        try:
227
 
            self.transport.rename(tmpname, self._held_dir)
228
 
        except (errors.TransportError, PathError, DirectoryNotEmpty,
229
 
                FileExists, ResourceBusy), e:
230
 
            self._trace("... contention, %s", e)
231
 
            self._remove_pending_dir(tmpname)
232
 
            raise LockContention(self)
233
 
        except Exception, e:
234
 
            self._trace("... lock failed, %s", e)
235
 
            self._remove_pending_dir(tmpname)
236
 
            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
237
258
        # We must check we really got the lock, because Launchpad's sftp
238
259
        # server at one time had a bug were the rename would successfully
239
260
        # move the new directory into the existing directory, which was
240
261
        # incorrect.  It's possible some other servers or filesystems will
241
262
        # have a similar bug allowing someone to think they got the lock
242
263
        # when it's already held.
 
264
        #
 
265
        # See <https://bugs.launchpad.net/bzr/+bug/498378> for one case.
 
266
        #
 
267
        # Strictly the check is unnecessary and a waste of time for most
 
268
        # people, but probably worth trapping if something is wrong.
243
269
        info = self.peek()
244
270
        self._trace("after locking, info=%r", info)
245
 
        if info['nonce'] != self.nonce:
 
271
        if info is None:
 
272
            raise LockFailed(self, "lock was renamed into place, but "
 
273
                "now is missing!")
 
274
        if info.get('nonce') != self.nonce:
246
275
            self._trace("rename succeeded, "
247
276
                "but lock is still held by someone else")
248
277
            raise LockContention(self)
251
280
                (time.time() - start_time) * 1000)
252
281
        return self.nonce
253
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
 
254
310
    def _remove_pending_dir(self, tmpname):
255
311
        """Remove the pending directory
256
312
 
276
332
            self.create(mode=self._dir_modebits)
277
333
            # After creating the lock directory, try again
278
334
            self.transport.mkdir(tmpname)
279
 
        self.nonce = rand_chars(20)
280
 
        info_bytes = self._prepare_info()
 
335
        info = LockHeldInfo.for_this_process(self.extra_holder_info)
 
336
        self.nonce = info.get('nonce')
281
337
        # We use put_file_non_atomic because we just created a new unique
282
338
        # directory so we don't have to worry about files existing there.
283
339
        # We'll rename the whole directory into place to get atomic
284
340
        # properties
285
341
        self.transport.put_bytes_non_atomic(tmpname + self.__INFO_NAME,
286
 
                                            info_bytes)
 
342
            info.to_bytes())
287
343
        return tmpname
288
344
 
 
345
    @only_raises(LockNotHeld, LockBroken)
289
346
    def unlock(self):
290
347
        """Release a held lock
291
348
        """
332
389
    def break_lock(self):
333
390
        """Break a lock not held by this instance of LockDir.
334
391
 
335
 
        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
336
393
        prompt for input if a lock is detected and there is any doubt about
337
 
        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.
338
398
        """
339
399
        self._check_not_locked()
340
 
        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
341
407
        if holder_info is not None:
342
 
            lock_info = '\n'.join(self._format_lock_info(holder_info))
343
 
            if bzrlib.ui.ui_factory.get_boolean("Break %s" % lock_info):
344
 
                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)
345
415
 
346
416
    def force_break(self, dead_holder_info):
347
417
        """Release a lock held by another process.
350
420
        it still thinks it has the lock there will be two concurrent writers.
351
421
        In general the user's approval should be sought for lock breaks.
352
422
 
353
 
        dead_holder_info must be the result of a previous LockDir.peek() call;
354
 
        this is used to check that it's still held by the same process that
355
 
        the user decided was dead.  If this is not the current holder,
356
 
        LockBreakMismatch is raised.
357
 
 
358
423
        After the lock is broken it will not be held by any process.
359
424
        It is possible that another process may sneak in and take the
360
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.
361
434
        """
362
 
        if not isinstance(dead_holder_info, dict):
 
435
        if not isinstance(dead_holder_info, LockHeldInfo):
363
436
            raise ValueError("dead_holder_info: %r" % dead_holder_info)
364
437
        self._check_not_locked()
365
438
        current_info = self.peek()
383
456
                                 current_info.get('nonce'))
384
457
        for hook in self.hooks['lock_broken']:
385
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)
386
489
 
387
490
    def _check_not_locked(self):
388
491
        """If the lock is held by this instance, raise an error."""
414
517
 
415
518
        peek() reads the info file of the lock holder, if any.
416
519
        """
417
 
        return self._parse_info(self.transport.get(path))
 
520
        return LockHeldInfo.from_info_file_bytes(
 
521
            self.transport.get_bytes(path))
418
522
 
419
523
    def peek(self):
420
524
        """Check if the lock is held by anyone.
421
525
 
422
 
        If it is held, this returns the lock info structure as a rio Stanza,
 
526
        If it is held, this returns the lock info structure as a dict
423
527
        which contains some information about the current lock holder.
424
528
        Otherwise returns None.
425
529
        """
433
537
    def _prepare_info(self):
434
538
        """Write information about a pending lock to a temporary file.
435
539
        """
436
 
        # XXX: is creating this here inefficient?
437
 
        config = bzrlib.config.GlobalConfig()
438
 
        try:
439
 
            user = config.user_email()
440
 
        except errors.NoEmailInUsername:
441
 
            user = config.username()
442
 
        s = rio.Stanza(hostname=get_host_name(),
443
 
                   pid=str(os.getpid()),
444
 
                   start_time=str(int(time.time())),
445
 
                   nonce=self.nonce,
446
 
                   user=user,
447
 
                   )
448
 
        return s.to_string()
449
 
 
450
 
    def _parse_info(self, info_file):
451
 
        return rio.read_stanza(info_file.readlines()).as_dict()
452
540
 
453
541
    def attempt_lock(self):
454
542
        """Take the lock; fail if it's already held.
468
556
            hook(hook_result)
469
557
        return result
470
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
 
471
570
    def wait_lock(self, timeout=None, poll=None, max_attempts=None):
472
571
        """Wait a certain period for a lock.
473
572
 
497
596
        deadline_str = None
498
597
        last_info = None
499
598
        attempt_count = 0
 
599
        lock_url = self.lock_url_for_display()
500
600
        while True:
501
601
            attempt_count += 1
502
602
            try:
517
617
                else:
518
618
                    start = 'Lock owner changed for'
519
619
                last_info = new_info
520
 
                formatted_info = self._format_lock_info(new_info)
 
620
                msg = u'%s lock %s %s.' % (start, lock_url, new_info)
521
621
                if deadline_str is None:
522
622
                    deadline_str = time.strftime('%H:%M:%S',
523
 
                                                 time.localtime(deadline))
524
 
                lock_url = self.transport.abspath(self.path)
525
 
                self._report_function('%s %s\n'
526
 
                                      '%s\n' # held by
527
 
                                      '%s\n' # locked ... ago
528
 
                                      'Will continue to try until %s, unless '
529
 
                                      'you press Ctrl-C\n'
530
 
                                      'If you\'re sure that it\'s not being '
531
 
                                      'modified, use bzr break-lock %s',
532
 
                                      start,
533
 
                                      formatted_info[0],
534
 
                                      formatted_info[1],
535
 
                                      formatted_info[2],
536
 
                                      deadline_str,
537
 
                                      lock_url)
538
 
 
 
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)
539
630
            if (max_attempts is not None) and (attempt_count >= max_attempts):
540
631
                self._trace("exceeded %d attempts")
541
632
                raise LockContention(self)
543
634
                self._trace("waiting %ss", poll)
544
635
                time.sleep(poll)
545
636
            else:
 
637
                # As timeout is always 0 for remote locks
 
638
                # this block is applicable only for local
 
639
                # lock contention
546
640
                self._trace("timeout after waiting %ss", timeout)
547
 
                raise LockContention(self)
 
641
                raise LockContention('(local)', lock_url)
548
642
 
549
643
    def leave_in_place(self):
550
644
        self._locked_via_token = True
593
687
            raise LockContention(self)
594
688
        self._fake_read_lock = True
595
689
 
596
 
    def _format_lock_info(self, info):
597
 
        """Turn the contents of peek() into something for the user"""
598
 
        lock_url = self.transport.abspath(self.path)
599
 
        delta = time.time() - int(info['start_time'])
600
 
        return [
601
 
            'lock %s' % (lock_url,),
602
 
            'held by %(user)s on host %(hostname)s [process #%(pid)s]' % info,
603
 
            'locked %s' % (format_delta(delta),),
604
 
            ]
605
 
 
606
690
    def validate_token(self, token):
607
691
        if token is not None:
608
692
            info = self.peek()
620
704
        if 'lock' not in debug.debug_flags:
621
705
            return
622
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()