~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/check.py

merge merge tweaks from aaron, which includes latest .dev

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006 Canonical Ltd
2
 
#
 
1
# Copyright (C) 2004, 2005 by Martin Pool
 
2
# Copyright (C) 2005 by Canonical Ltd
 
3
 
3
4
# This program is free software; you can redistribute it and/or modify
4
5
# it under the terms of the GNU General Public License as published by
5
6
# the Free Software Foundation; either version 2 of the License, or
6
7
# (at your option) any later version.
7
 
#
 
8
 
8
9
# This program is distributed in the hope that it will be useful,
9
10
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
11
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
12
# GNU General Public License for more details.
12
 
#
 
13
 
13
14
# You should have received a copy of the GNU General Public License
14
15
# along with this program; if not, write to the Free Software
15
16
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
17
 
17
 
# TODO: Check ancestries are correct for every revision: includes
18
 
# every committed so far, and in a reasonable order.
19
 
 
20
 
# TODO: Also check non-mainline revisions mentioned as parents.
21
 
 
22
 
# TODO: Check for extra files in the control directory.
23
 
 
24
 
# TODO: Check revision, inventory and entry objects have all 
25
 
# required fields.
26
 
 
27
 
# TODO: Get every revision in the revision-store even if they're not
28
 
# referenced by history and make sure they're all valid.
29
 
 
30
 
# TODO: Perhaps have a way to record errors other than by raising exceptions;
31
 
# would perhaps be enough to accumulate exception objects in a list without
32
 
# raising them.  If there's more than one exception it'd be good to see them
33
 
# all.
34
 
 
35
 
from bzrlib import errors
36
 
from bzrlib import repository as _mod_repository
37
 
from bzrlib import revision
38
 
from bzrlib.errors import BzrCheckError
39
18
import bzrlib.ui
40
 
from bzrlib.trace import log_error, note
41
 
 
42
 
class Check(object):
43
 
    """Check a repository"""
44
 
 
45
 
    # The Check object interacts with InventoryEntry.check, etc.
46
 
 
47
 
    def __init__(self, repository):
48
 
        self.repository = repository
49
 
        self.checked_text_cnt = 0
50
 
        self.checked_rev_cnt = 0
51
 
        self.ghosts = []
52
 
        self.repeated_text_cnt = 0
53
 
        self.missing_parent_links = {}
54
 
        self.missing_inventory_sha_cnt = 0
55
 
        self.missing_revision_cnt = 0
56
 
        # maps (file-id, version) -> sha1; used by InventoryFile._check
57
 
        self.checked_texts = {}
58
 
        self.checked_weaves = {}
59
 
        self.unreferenced_versions = set()
60
 
        self.inconsistent_parents = []
61
 
 
62
 
    def check(self):
63
 
        self.repository.lock_read()
64
 
        self.progress = bzrlib.ui.ui_factory.nested_progress_bar()
 
19
from bzrlib.trace import note, warning
 
20
 
 
21
def _update_store_entry(obj, obj_id, branch, store_name, store):
 
22
    """This is just a meta-function, which handles both revision entries
 
23
    and inventory entries.
 
24
    """
 
25
    from bzrlib.trace import mutter
 
26
    import tempfile, os, errno
 
27
    from osutils import rename
 
28
    obj_tmp = tempfile.TemporaryFile()
 
29
    obj.write_xml(obj_tmp)
 
30
    obj_tmp.seek(0)
 
31
 
 
32
    tmpfd, tmp_path = tempfile.mkstemp(prefix=obj_id, suffix='.gz',
 
33
        dir=branch.controlfilename(store_name))
 
34
    os.close(tmpfd)
 
35
    try:
 
36
        orig_obj_path = branch.controlfilename([store_name, obj_id+'.gz'])
 
37
        # Remove the old entry out of the way
 
38
        rename(orig_obj_path, tmp_path)
65
39
        try:
66
 
            self.progress.update('retrieving inventory', 0, 2)
67
 
            # do not put in init, as it should be done with progess,
68
 
            # and inside the lock.
69
 
            self.inventory_weave = self.repository.get_inventory_weave()
70
 
            self.progress.update('checking revision graph', 1)
71
 
            self.check_revision_graph()
72
 
            self.plan_revisions()
73
 
            revno = 0
74
 
            while revno < len(self.planned_revisions):
75
 
                rev_id = self.planned_revisions[revno]
76
 
                self.progress.update('checking revision', revno,
77
 
                                     len(self.planned_revisions))
78
 
                revno += 1
79
 
                self.check_one_rev(rev_id)
80
 
            # check_weaves is done after the revision scan so that
81
 
            # revision index is known to be valid.
82
 
            self.check_weaves()
83
 
        finally:
84
 
            self.progress.finished()
85
 
            self.repository.unlock()
86
 
 
87
 
    def check_revision_graph(self):
88
 
        if not self.repository.revision_graph_can_have_wrong_parents():
89
 
            # This check is not necessary.
90
 
            self.revs_with_bad_parents_in_index = None
91
 
            return
92
 
        bad_revisions = self.repository._find_inconsistent_revision_parents()
93
 
        self.revs_with_bad_parents_in_index = list(bad_revisions)
94
 
 
95
 
    def plan_revisions(self):
96
 
        repository = self.repository
97
 
        self.planned_revisions = repository.all_revision_ids()
98
 
        self.progress.clear()
99
 
        inventoried = set(self.inventory_weave.versions())
100
 
        awol = set(self.planned_revisions) - inventoried
101
 
        if len(awol) > 0:
102
 
            raise BzrCheckError('Stored revisions missing from inventory'
103
 
                '{%s}' % ','.join([f for f in awol]))
104
 
 
105
 
    def report_results(self, verbose):
106
 
        note('checked repository %s format %s',
107
 
             self.repository.bzrdir.root_transport,
108
 
             self.repository._format)
109
 
        note('%6d revisions', self.checked_rev_cnt)
110
 
        note('%6d file-ids', len(self.checked_weaves))
111
 
        note('%6d unique file texts', self.checked_text_cnt)
112
 
        note('%6d repeated file texts', self.repeated_text_cnt)
113
 
        note('%6d unreferenced text versions',
114
 
             len(self.unreferenced_versions))
115
 
        if self.missing_inventory_sha_cnt:
116
 
            note('%6d revisions are missing inventory_sha1',
117
 
                 self.missing_inventory_sha_cnt)
118
 
        if self.missing_revision_cnt:
119
 
            note('%6d revisions are mentioned but not present',
120
 
                 self.missing_revision_cnt)
121
 
        if len(self.ghosts):
122
 
            note('%6d ghost revisions', len(self.ghosts))
123
 
            if verbose:
124
 
                for ghost in self.ghosts:
125
 
                    note('      %s', ghost)
126
 
        if len(self.missing_parent_links):
127
 
            note('%6d revisions missing parents in ancestry',
128
 
                 len(self.missing_parent_links))
129
 
            if verbose:
130
 
                for link, linkers in self.missing_parent_links.items():
131
 
                    note('      %s should be in the ancestry for:', link)
132
 
                    for linker in linkers:
133
 
                        note('       * %s', linker)
134
 
            if verbose:
135
 
                for file_id, revision_id in self.unreferenced_versions:
136
 
                    log_error('unreferenced version: {%s} in %s', revision_id,
137
 
                        file_id)
138
 
        if len(self.inconsistent_parents):
139
 
            note('%6d inconsistent parents', len(self.inconsistent_parents))
140
 
            if verbose:
141
 
                for info in self.inconsistent_parents:
142
 
                    revision_id, file_id, found_parents, correct_parents = info
143
 
                    note('      * %s version %s has parents %r '
144
 
                         'but should have %r'
145
 
                         % (file_id, revision_id, found_parents,
146
 
                             correct_parents))
147
 
        if self.revs_with_bad_parents_in_index:
148
 
            note('%6d revisions have incorrect parents in the revision index',
149
 
                 len(self.revs_with_bad_parents_in_index))
150
 
            if verbose:
151
 
                for item in self.revs_with_bad_parents_in_index:
152
 
                    revision_id, index_parents, actual_parents = item
153
 
                    note(
154
 
                        '       %s has wrong parents in index: '
155
 
                        '%r should be %r',
156
 
                        revision_id, index_parents, actual_parents)
157
 
 
158
 
    def check_one_rev(self, rev_id):
159
 
        """Check one revision.
160
 
 
161
 
        rev_id - the one to check
162
 
        """
163
 
        rev = self.repository.get_revision(rev_id)
164
 
                
165
 
        if rev.revision_id != rev_id:
166
 
            raise BzrCheckError('wrong internal revision id in revision {%s}'
167
 
                                % rev_id)
168
 
 
169
 
        for parent in rev.parent_ids:
170
 
            if not parent in self.planned_revisions:
171
 
                missing_links = self.missing_parent_links.get(parent, [])
172
 
                missing_links.append(rev_id)
173
 
                self.missing_parent_links[parent] = missing_links
174
 
                # list based so somewhat slow,
175
 
                # TODO have a planned_revisions list and set.
176
 
                if self.repository.has_revision(parent):
177
 
                    missing_ancestry = self.repository.get_ancestry(parent)
178
 
                    for missing in missing_ancestry:
179
 
                        if (missing is not None 
180
 
                            and missing not in self.planned_revisions):
181
 
                            self.planned_revisions.append(missing)
 
40
            # TODO: We may need to handle the case where the old
 
41
            # entry was not compressed (and thus did not end with .gz)
 
42
 
 
43
            store.add(obj_tmp, obj_id) # Add the new one
 
44
            os.remove(tmp_path) # Remove the old name
 
45
            mutter('    Updated %s entry {%s}' % (store_name, obj_id))
 
46
        except:
 
47
            # On any exception, restore the old entry
 
48
            rename(tmp_path, orig_obj_path)
 
49
            raise
 
50
    finally:
 
51
        if os.path.exists(tmp_path):
 
52
            # Unfortunately, the next command might throw
 
53
            # an exception, which will mask a previous exception.
 
54
            os.remove(tmp_path)
 
55
        obj_tmp.close()
 
56
 
 
57
def _update_revision_entry(rev, branch):
 
58
    """After updating the values in a revision, make sure to
 
59
    write out the data, but try to do it in an atomic manner.
 
60
 
 
61
    :param rev:    The Revision object to store
 
62
    :param branch: The Branch object where this Revision is to be stored.
 
63
    """
 
64
    _update_store_entry(rev, rev.revision_id, branch,
 
65
            'revision-store', branch.revision_store)
 
66
 
 
67
def _update_inventory_entry(inv, inv_id, branch):
 
68
    """When an inventory has been modified (such as by adding a unique tree root)
 
69
    this atomically re-generates the file.
 
70
 
 
71
    :param inv:     The Inventory
 
72
    :param inv_id:  The inventory id for this inventory
 
73
    :param branch:  The Branch where this entry will be stored.
 
74
    """
 
75
    _update_store_entry(inv, inv_id, branch,
 
76
            'inventory-store', branch.inventory_store)
 
77
 
 
78
 
 
79
def check(branch):
 
80
    """Run consistency checks on a branch.
 
81
 
 
82
    TODO: Also check non-mainline revisions mentioned as parents.
 
83
 
 
84
    TODO: Check for extra files in the control directory.
 
85
    """
 
86
    from bzrlib.trace import mutter
 
87
    from bzrlib.errors import BzrCheckError, NoSuchRevision
 
88
    from bzrlib.osutils import fingerprint_file
 
89
    from bzrlib.inventory import ROOT_ID
 
90
    from bzrlib.branch import gen_root_id
 
91
 
 
92
    branch.lock_read()
 
93
 
 
94
    try:
 
95
        last_rev_id = None
 
96
 
 
97
        missing_inventory_sha_cnt = 0
 
98
        missing_revision_sha_cnt = 0
 
99
        missing_revision_cnt = 0
 
100
 
 
101
        history = branch.revision_history()
 
102
        revno = 0
 
103
        revcount = len(history)
 
104
        mismatch_inv_id = []
 
105
 
 
106
        # for all texts checked, text_id -> sha1
 
107
        checked_texts = {}
 
108
 
 
109
        progress = bzrlib.ui.ui_factory.progress_bar()
 
110
 
 
111
        for rev_id in history:
 
112
            revno += 1
 
113
            progress.update('checking revision', revno, revcount)
 
114
            # mutter('    revision {%s}' % rev_id)
 
115
            rev = branch.get_revision(rev_id)
 
116
            if rev.revision_id != rev_id:
 
117
                raise BzrCheckError('wrong internal revision id in revision {%s}'
 
118
                                    % rev_id)
 
119
 
 
120
            # check the previous history entry is a parent of this entry
 
121
            if rev.parents:
 
122
                if last_rev_id is None:
 
123
                    raise BzrCheckError("revision {%s} has %d parents, but is the "
 
124
                                        "start of the branch"
 
125
                                        % (rev_id, len(rev.parents)))
 
126
                for prr in rev.parents:
 
127
                    if prr.revision_id == last_rev_id:
 
128
                        break
182
129
                else:
183
 
                    self.ghosts.append(rev_id)
184
 
 
185
 
        if rev.inventory_sha1:
186
 
            inv_sha1 = self.repository.get_inventory_sha1(rev_id)
187
 
            if inv_sha1 != rev.inventory_sha1:
188
 
                raise BzrCheckError('Inventory sha1 hash doesn\'t match'
189
 
                    ' value in revision {%s}' % rev_id)
190
 
        self._check_revision_tree(rev_id)
191
 
        self.checked_rev_cnt += 1
192
 
 
193
 
    def check_weaves(self):
194
 
        """Check all the weaves we can get our hands on.
195
 
        """
196
 
        n_weaves = 1
197
 
        weave_ids = []
198
 
        if self.repository.weave_store.listable():
199
 
            weave_ids = list(self.repository.weave_store)
200
 
            n_weaves = len(weave_ids) + 1
201
 
        self.progress.update('checking versionedfile', 0, n_weaves)
202
 
        self.inventory_weave.check(progress_bar=self.progress)
203
 
        files_in_revisions = {}
204
 
        revisions_of_files = {}
205
 
        weave_checker = self.repository._get_versioned_file_checker()
206
 
        # XXX: for pack repositories, this iteration seems a bit redundant
207
 
        # with iteration also done within the versioned file checker.  also
208
 
        # note that the inventory/revision/signature data is not included in
209
 
        # this enumeration -- mbp 20080528
210
 
        for i, weave_id in enumerate(weave_ids):
211
 
            self.progress.update('checking versionedfile', i, n_weaves)
212
 
            w = self.repository.weave_store.get_weave(weave_id,
213
 
                    self.repository.get_transaction())
214
 
            # No progress here, because it looks ugly.
215
 
            w.check()
216
 
            result = weave_checker.check_file_version_parents(w, weave_id)
217
 
            bad_parents, unused_versions = result
218
 
            bad_parents = bad_parents.items()
219
 
            for revision_id, (weave_parents, correct_parents) in bad_parents:
220
 
                self.inconsistent_parents.append(
221
 
                    (revision_id, weave_id, weave_parents, correct_parents))
222
 
            for revision_id in unused_versions:
223
 
                self.unreferenced_versions.add((weave_id, revision_id))
224
 
            self.checked_weaves[weave_id] = True
225
 
 
226
 
    def _check_revision_tree(self, rev_id):
227
 
        tree = self.repository.revision_tree(rev_id)
228
 
        inv = tree.inventory
229
 
        seen_ids = {}
230
 
        for file_id in inv:
231
 
            if file_id in seen_ids:
232
 
                raise BzrCheckError('duplicated file_id {%s} '
233
 
                                    'in inventory for revision {%s}'
234
 
                                    % (file_id, rev_id))
235
 
            seen_ids[file_id] = True
236
 
        for file_id in inv:
237
 
            ie = inv[file_id]
238
 
            ie.check(self, rev_id, inv, tree)
239
 
        seen_names = {}
240
 
        for path, ie in inv.iter_entries():
241
 
            if path in seen_names:
242
 
                raise BzrCheckError('duplicated path %s '
243
 
                                    'in inventory for revision {%s}'
244
 
                                    % (path, rev_id))
 
130
                    raise BzrCheckError("previous revision {%s} not listed among "
 
131
                                        "parents of {%s}"
 
132
                                        % (last_rev_id, rev_id))
 
133
 
 
134
                for prr in rev.parents:
 
135
                    if prr.revision_sha1 is None:
 
136
                        missing_revision_sha_cnt += 1
 
137
                        continue
 
138
                    prid = prr.revision_id
 
139
                    
 
140
                    try:
 
141
                        actual_sha = branch.get_revision_sha1(prid)
 
142
                    except NoSuchRevision:
 
143
                        missing_revision_cnt += 1
 
144
                        mutter("parent {%s} of {%s} not present in store",
 
145
                               prid, rev_id)
 
146
                        continue
 
147
                        
 
148
                    if prr.revision_sha1 != actual_sha:
 
149
                        raise BzrCheckError("mismatched revision sha1 for "
 
150
                                            "parent {%s} of {%s}: %s vs %s"
 
151
                                            % (prid, rev_id,
 
152
                                               prr.revision_sha1, actual_sha))
 
153
            elif last_rev_id:
 
154
                raise BzrCheckError("revision {%s} has no parents listed but preceded "
 
155
                                    "by {%s}"
 
156
                                    % (rev_id, last_rev_id))
 
157
 
 
158
            if rev.inventory_id != rev_id:
 
159
                mismatch_inv_id.append(rev_id)
 
160
 
 
161
            ## TODO: Check all the required fields are present on the revision.
 
162
 
 
163
            if rev.inventory_sha1:
 
164
                inv_sha1 = branch.get_inventory_sha1(rev.inventory_id)
 
165
                if inv_sha1 != rev.inventory_sha1:
 
166
                    raise BzrCheckError('Inventory sha1 hash doesn\'t match'
 
167
                        ' value in revision {%s}' % rev_id)
 
168
            else:
 
169
                missing_inventory_sha_cnt += 1
 
170
                mutter("no inventory_sha1 on revision {%s}" % rev_id)
 
171
 
 
172
            inv = branch.get_inventory(rev.inventory_id)
 
173
            seen_ids = {}
 
174
            seen_names = {}
 
175
 
 
176
            ## p('revision %d/%d file ids' % (revno, revcount))
 
177
            for file_id in inv:
 
178
                if file_id in seen_ids:
 
179
                    raise BzrCheckError('duplicated file_id {%s} '
 
180
                                        'in inventory for revision {%s}'
 
181
                                        % (file_id, rev_id))
 
182
                seen_ids[file_id] = True
 
183
 
 
184
            i = 0
 
185
            for file_id in inv:
 
186
                i += 1
 
187
                if i & 31 == 0:
 
188
                    progress.tick()
 
189
 
 
190
                ie = inv[file_id]
 
191
 
 
192
                if ie.parent_id != None:
 
193
                    if ie.parent_id not in seen_ids:
 
194
                        raise BzrCheckError('missing parent {%s} in inventory for revision {%s}'
 
195
                                % (ie.parent_id, rev_id))
 
196
 
 
197
                if ie.kind == 'file':
 
198
                    if ie.text_id in checked_texts:
 
199
                        fp = checked_texts[ie.text_id]
 
200
                    else:
 
201
                        if not ie.text_id in branch.text_store:
 
202
                            raise BzrCheckError('text {%s} not in text_store' % ie.text_id)
 
203
 
 
204
                        tf = branch.text_store[ie.text_id]
 
205
                        fp = fingerprint_file(tf)
 
206
                        checked_texts[ie.text_id] = fp
 
207
 
 
208
                    if ie.text_size != fp['size']:
 
209
                        raise BzrCheckError('text {%s} wrong size' % ie.text_id)
 
210
                    if ie.text_sha1 != fp['sha1']:
 
211
                        raise BzrCheckError('text {%s} wrong sha1' % ie.text_id)
 
212
                elif ie.kind == 'directory':
 
213
                    if ie.text_sha1 != None or ie.text_size != None or ie.text_id != None:
 
214
                        raise BzrCheckError('directory {%s} has text in revision {%s}'
 
215
                                % (file_id, rev_id))
 
216
 
 
217
            progress.tick()
 
218
            for path, ie in inv.iter_entries():
 
219
                if path in seen_names:
 
220
                    raise BzrCheckError('duplicated path %s '
 
221
                                        'in inventory for revision {%s}'
 
222
                                        % (path, rev_id))
245
223
            seen_names[path] = True
246
 
 
247
 
 
248
 
def check(branch, verbose):
249
 
    """Run consistency checks on a branch.
250
 
    
251
 
    Results are reported through logging.
252
 
    
253
 
    :raise BzrCheckError: if there's a consistency error.
254
 
    """
255
 
    branch.lock_read()
256
 
    try:
257
 
        branch_result = branch.check()
258
 
        repo_result = branch.repository.check([branch.last_revision()])
 
224
            last_rev_id = rev_id
 
225
 
259
226
    finally:
260
227
        branch.unlock()
261
 
    branch_result.report_results(verbose)
262
 
    repo_result.report_results(verbose)
263
 
 
264
 
 
265
 
 
 
228
 
 
229
    progress.clear()
 
230
 
 
231
    note('checked %d revisions, %d file texts' % (revcount, len(checked_texts)))
 
232
    
 
233
    if missing_inventory_sha_cnt:
 
234
        note('%d revisions are missing inventory_sha1' % missing_inventory_sha_cnt)
 
235
 
 
236
    if missing_revision_sha_cnt:
 
237
        note('%d parent links are missing revision_sha1' % missing_revision_sha_cnt)
 
238
 
 
239
    if missing_revision_cnt:
 
240
        note('%d revisions are mentioned but not present' % missing_revision_cnt)
 
241
 
 
242
    if missing_revision_cnt:
 
243
        print '%d revisions are mentioned but not present' % missing_revision_cnt
 
244
 
 
245
    # stub this out for now because the main bzr branch has references
 
246
    # to revisions that aren't present in the store -- mbp 20050804
 
247
#    if (missing_inventory_sha_cnt
 
248
#        or missing_revision_sha_cnt):
 
249
#        print '  (use "bzr upgrade" to fix them)'
 
250
 
 
251
    if mismatch_inv_id:
 
252
        warning('%d revisions have mismatched inventory ids:' % len(mismatch_inv_id))
 
253
        for rev_id in mismatch_inv_id:
 
254
            warning('  %s', rev_id)