~bzr-pqm/bzr/bzr.dev

2052.3.2 by John Arbash Meinel
Change Copyright .. by Canonical to Copyright ... Canonical
1
# Copyright (C) 2005, 2006 Canonical Ltd
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
2
#
1 by mbp at sourcefrog
import from baz patch-364
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
7
#
1 by mbp at sourcefrog
import from baz patch-364
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
12
#
1 by mbp at sourcefrog
import from baz patch-364
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
1335 by Martin Pool
doc
17
# TODO: Check ancestries are correct for every revision: includes
18
# every committed so far, and in a reasonable order.
19
1347 by Martin Pool
- refactor check code into method object
20
# TODO: Also check non-mainline revisions mentioned as parents.
21
22
# TODO: Check for extra files in the control directory.
23
1348 by Martin Pool
- more refactoring of check code
24
# TODO: Check revision, inventory and entry objects have all 
25
# required fields.
26
1185.16.101 by mbp at sourcefrog
todo
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.
1347 by Martin Pool
- refactor check code into method object
29
1616.1.5 by Martin Pool
Cleanup and document some check code
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
2745.6.47 by Andrew Bennetts
Move check_parents out of VersionedFile.
35
from bzrlib import errors
2745.6.16 by Aaron Bentley
Update from review
36
from bzrlib import repository as _mod_repository
2745.6.47 by Andrew Bennetts
Move check_parents out of VersionedFile.
37
from bzrlib import revision
1773.4.1 by Martin Pool
Add pyflakes makefile target; fix many warnings
38
from bzrlib.errors import BzrCheckError
1104 by Martin Pool
- Add a simple UIFactory
39
import bzrlib.ui
2745.6.61 by Andrew Bennetts
Remove some useless mutters, and cope with a file_id that isn't present in a revision's inventory.
40
from bzrlib.trace import log_error, note
1104 by Martin Pool
- Add a simple UIFactory
41
1347 by Martin Pool
- refactor check code into method object
42
class Check(object):
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
43
    """Check a repository"""
1449 by Robert Collins
teach check about ghosts
44
1616.1.5 by Martin Pool
Cleanup and document some check code
45
    # The Check object interacts with InventoryEntry.check, etc.
46
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
47
    def __init__(self, repository):
48
        self.repository = repository
1383 by Martin Pool
- untabify only
49
        self.checked_text_cnt = 0
50
        self.checked_rev_cnt = 0
1449 by Robert Collins
teach check about ghosts
51
        self.ghosts = []
1365 by Martin Pool
- try to avoid checking texts repeatedly
52
        self.repeated_text_cnt = 0
1449 by Robert Collins
teach check about ghosts
53
        self.missing_parent_links = {}
1348 by Martin Pool
- more refactoring of check code
54
        self.missing_inventory_sha_cnt = 0
55
        self.missing_revision_cnt = 0
1616.1.5 by Martin Pool
Cleanup and document some check code
56
        # maps (file-id, version) -> sha1; used by InventoryFile._check
1365 by Martin Pool
- try to avoid checking texts repeatedly
57
        self.checked_texts = {}
1185.50.28 by John Arbash Meinel
Lots of updates for 'bzr check'
58
        self.checked_weaves = {}
2745.6.16 by Aaron Bentley
Update from review
59
        self.revision_versions = _mod_repository._RevisionTextVersionCache(
60
            self.repository)
2745.6.6 by Aaron Bentley
Add unreferenced ancestors to check output
61
        self.unreferenced_ancestors = set()
2745.6.33 by Andrew Bennetts
Add VersionedFile.check_parents, and use it instead of find_bad_ancestors in reconcile.
62
        self.inconsistent_parents = []
676 by Martin Pool
- lock branch while checking
63
1449 by Robert Collins
teach check about ghosts
64
    def check(self):
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
65
        self.repository.lock_read()
1594.1.3 by Robert Collins
Fixup pb usage to use nested_progress_bar.
66
        self.progress = bzrlib.ui.ui_factory.nested_progress_bar()
1449 by Robert Collins
teach check about ghosts
67
        try:
2819.2.3 by Andrew Bennetts
Add test that repo.check will report on wrong parents in the revision graph.
68
            self.progress.update('retrieving inventory', 0, 2)
1510 by Robert Collins
Merge from mpool, adjusting check to retain HTTP support.
69
            # do not put in init, as it should be done with progess,
70
            # and inside the lock.
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
71
            self.inventory_weave = self.repository.get_inventory_weave()
2819.2.3 by Andrew Bennetts
Add test that repo.check will report on wrong parents in the revision graph.
72
            self.progress.update('checking revision graph', 1)
73
            self.check_revision_graph()
1510 by Robert Collins
Merge from mpool, adjusting check to retain HTTP support.
74
            self.plan_revisions()
75
            revno = 0
76
            while revno < len(self.planned_revisions):
77
                rev_id = self.planned_revisions[revno]
78
                self.progress.update('checking revision', revno,
1449 by Robert Collins
teach check about ghosts
79
                                     len(self.planned_revisions))
1510 by Robert Collins
Merge from mpool, adjusting check to retain HTTP support.
80
                revno += 1
1449 by Robert Collins
teach check about ghosts
81
                self.check_one_rev(rev_id)
2745.6.16 by Aaron Bentley
Update from review
82
            # check_weaves is done after the revision scan so that
83
            # revision_versions is pre-populated
2745.6.3 by Aaron Bentley
Implement versionedfile checking for bzr check
84
            self.check_weaves()
1185.35.34 by Aaron Bentley
Made bzr check for stored revisions missing from ancestry
85
        finally:
1594.1.3 by Robert Collins
Fixup pb usage to use nested_progress_bar.
86
            self.progress.finished()
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
87
            self.repository.unlock()
1449 by Robert Collins
teach check about ghosts
88
2819.2.3 by Andrew Bennetts
Add test that repo.check will report on wrong parents in the revision graph.
89
    def check_revision_graph(self):
2819.2.4 by Andrew Bennetts
Add a 'revision_graph_can_have_wrong_parents' method to repository.
90
        if not self.repository.revision_graph_can_have_wrong_parents():
91
            # This check is not necessary.
2819.2.3 by Andrew Bennetts
Add test that repo.check will report on wrong parents in the revision graph.
92
            self.revs_with_bad_parents_in_index = None
93
            return
94
        bad_revisions = self.repository._find_inconsistent_revision_parents()
95
        self.revs_with_bad_parents_in_index = list(bad_revisions)
96
1510 by Robert Collins
Merge from mpool, adjusting check to retain HTTP support.
97
    def plan_revisions(self):
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
98
        repository = self.repository
2745.6.3 by Aaron Bentley
Implement versionedfile checking for bzr check
99
        self.planned_revisions = repository.all_revision_ids()
1563.2.22 by Robert Collins
Move responsibility for repository.has_revision into RevisionStore
100
        self.progress.clear()
1563.2.35 by Robert Collins
cleanup deprecation warnings and finish conversion so the inventory is knit based too.
101
        inventoried = set(self.inventory_weave.versions())
2745.6.3 by Aaron Bentley
Implement versionedfile checking for bzr check
102
        awol = set(self.planned_revisions) - inventoried
1510 by Robert Collins
Merge from mpool, adjusting check to retain HTTP support.
103
        if len(awol) > 0:
104
            raise BzrCheckError('Stored revisions missing from inventory'
105
                '{%s}' % ','.join([f for f in awol]))
106
1449 by Robert Collins
teach check about ghosts
107
    def report_results(self, verbose):
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
108
        note('checked repository %s format %s',
109
             self.repository.bzrdir.root_transport,
110
             self.repository._format)
1365 by Martin Pool
- try to avoid checking texts repeatedly
111
        note('%6d revisions', self.checked_rev_cnt)
2745.6.47 by Andrew Bennetts
Move check_parents out of VersionedFile.
112
        note('%6d file-ids', len(self.checked_weaves))
1365 by Martin Pool
- try to avoid checking texts repeatedly
113
        note('%6d unique file texts', self.checked_text_cnt)
114
        note('%6d repeated file texts', self.repeated_text_cnt)
2745.6.6 by Aaron Bentley
Add unreferenced ancestors to check output
115
        note('%6d unreferenced text ancestors',
116
             len(self.unreferenced_ancestors))
1348 by Martin Pool
- more refactoring of check code
117
        if self.missing_inventory_sha_cnt:
1449 by Robert Collins
teach check about ghosts
118
            note('%6d revisions are missing inventory_sha1',
1383 by Martin Pool
- untabify only
119
                 self.missing_inventory_sha_cnt)
1348 by Martin Pool
- more refactoring of check code
120
        if self.missing_revision_cnt:
1449 by Robert Collins
teach check about ghosts
121
            note('%6d revisions are mentioned but not present',
1383 by Martin Pool
- untabify only
122
                 self.missing_revision_cnt)
1449 by Robert Collins
teach check about ghosts
123
        if len(self.ghosts):
124
            note('%6d ghost revisions', len(self.ghosts))
125
            if verbose:
126
                for ghost in self.ghosts:
127
                    note('      %s', ghost)
128
        if len(self.missing_parent_links):
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
129
            note('%6d revisions missing parents in ancestry',
1449 by Robert Collins
teach check about ghosts
130
                 len(self.missing_parent_links))
131
            if verbose:
132
                for link, linkers in self.missing_parent_links.items():
133
                    note('      %s should be in the ancestry for:', link)
134
                    for linker in linkers:
135
                        note('       * %s', linker)
2745.6.6 by Aaron Bentley
Add unreferenced ancestors to check output
136
            if verbose:
137
                for file_id, revision_id in self.unreferenced_ancestors:
2745.6.47 by Andrew Bennetts
Move check_parents out of VersionedFile.
138
                    log_error('unreferenced ancestor: {%s} in %s', revision_id,
2745.6.6 by Aaron Bentley
Add unreferenced ancestors to check output
139
                        file_id)
2745.6.39 by Andrew Bennetts
Use scenario in test_check too, and make check actually report inconsistent parents to the end user.
140
        if len(self.inconsistent_parents):
141
            note('%6d inconsistent parents', len(self.inconsistent_parents))
142
            if verbose:
143
                for info in self.inconsistent_parents:
144
                    revision_id, file_id, found_parents, correct_parents = info
145
                    note('      * %s version %s has parents %r '
146
                         'but should have %r'
147
                         % (file_id, revision_id, found_parents,
148
                             correct_parents))
2819.2.3 by Andrew Bennetts
Add test that repo.check will report on wrong parents in the revision graph.
149
        if self.revs_with_bad_parents_in_index:
150
            note('%6d revisions have incorrect parents in the revision index',
151
                 len(self.revs_with_bad_parents_in_index))
152
            if verbose:
153
                for item in self.revs_with_bad_parents_in_index:
154
                    revision_id, index_parents, actual_parents = item
155
                    note(
156
                        '       %s has wrong parents in index: '
157
                        '%r should be %r',
158
                        revision_id, index_parents, actual_parents)
1449 by Robert Collins
teach check about ghosts
159
160
    def check_one_rev(self, rev_id):
1383 by Martin Pool
- untabify only
161
        """Check one revision.
162
163
        rev_id - the one to check
164
        """
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
165
        rev = self.repository.get_revision(rev_id)
1449 by Robert Collins
teach check about ghosts
166
                
1383 by Martin Pool
- untabify only
167
        if rev.revision_id != rev_id:
168
            raise BzrCheckError('wrong internal revision id in revision {%s}'
169
                                % rev_id)
170
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
171
        for parent in rev.parent_ids:
172
            if not parent in self.planned_revisions:
173
                missing_links = self.missing_parent_links.get(parent, [])
174
                missing_links.append(rev_id)
175
                self.missing_parent_links[parent] = missing_links
176
                # list based so somewhat slow,
177
                # TODO have a planned_revisions list and set.
178
                if self.repository.has_revision(parent):
179
                    missing_ancestry = self.repository.get_ancestry(parent)
180
                    for missing in missing_ancestry:
181
                        if (missing is not None 
182
                            and missing not in self.planned_revisions):
183
                            self.planned_revisions.append(missing)
1449 by Robert Collins
teach check about ghosts
184
                else:
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
185
                    self.ghosts.append(rev_id)
1383 by Martin Pool
- untabify only
186
187
        if rev.inventory_sha1:
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
188
            inv_sha1 = self.repository.get_inventory_sha1(rev_id)
1383 by Martin Pool
- untabify only
189
            if inv_sha1 != rev.inventory_sha1:
190
                raise BzrCheckError('Inventory sha1 hash doesn\'t match'
191
                    ' value in revision {%s}' % rev_id)
192
        self._check_revision_tree(rev_id)
1362 by Martin Pool
- keep track of number of checked revisions
193
        self.checked_rev_cnt += 1
1349 by Martin Pool
- more refactoring of check code
194
1185.50.28 by John Arbash Meinel
Lots of updates for 'bzr check'
195
    def check_weaves(self):
196
        """Check all the weaves we can get our hands on.
197
        """
198
        n_weaves = 1
199
        weave_ids = []
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
200
        if self.repository.weave_store.listable():
201
            weave_ids = list(self.repository.weave_store)
2592.3.63 by Robert Collins
Supply weave_store.__iter__ for compatibility with check.
202
            n_weaves = len(weave_ids) + 1
2745.6.8 by Aaron Bentley
Clean up text
203
        self.progress.update('checking versionedfile', 0, n_weaves)
1185.50.28 by John Arbash Meinel
Lots of updates for 'bzr check'
204
        self.inventory_weave.check(progress_bar=self.progress)
2745.6.33 by Andrew Bennetts
Add VersionedFile.check_parents, and use it instead of find_bad_ancestors in reconcile.
205
        files_in_revisions = {}
206
        revisions_of_files = {}
1185.50.28 by John Arbash Meinel
Lots of updates for 'bzr check'
207
        for i, weave_id in enumerate(weave_ids):
2745.6.8 by Aaron Bentley
Clean up text
208
            self.progress.update('checking versionedfile', i, n_weaves)
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
209
            w = self.repository.weave_store.get_weave(weave_id,
210
                    self.repository.get_transaction())
1185.50.28 by John Arbash Meinel
Lots of updates for 'bzr check'
211
            # No progress here, because it looks ugly.
212
            w.check()
2745.6.47 by Andrew Bennetts
Move check_parents out of VersionedFile.
213
214
            weave_checker = self.repository.get_versioned_file_checker(
215
                self.planned_revisions, self.revision_versions)
2745.6.49 by Andrew Bennetts
Get rid of bzrlib.repository._RevisionParentsProvider.
216
            result = weave_checker.check_file_version_parents(w, weave_id)
2745.6.47 by Andrew Bennetts
Move check_parents out of VersionedFile.
217
2745.6.33 by Andrew Bennetts
Add VersionedFile.check_parents, and use it instead of find_bad_ancestors in reconcile.
218
            for revision_id, (weave_parents,correct_parents) in result.items():
219
                self.inconsistent_parents.append(
220
                    (revision_id, weave_id, weave_parents, correct_parents))
2745.6.50 by Andrew Bennetts
Remove find_bad_ancestors; it's not needed anymore.
221
                unreferenced_parents = set(weave_parents)-set(correct_parents)
222
                for unreferenced_parent in unreferenced_parents:
223
                    self.unreferenced_ancestors.add(
224
                        (weave_id, unreferenced_parent))
1185.50.28 by John Arbash Meinel
Lots of updates for 'bzr check'
225
            self.checked_weaves[weave_id] = True
226
1349 by Martin Pool
- more refactoring of check code
227
    def _check_revision_tree(self, rev_id):
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
228
        tree = self.repository.revision_tree(rev_id)
2745.6.53 by Andrew Bennetts
Some more changes suggested by review.
229
        self.revision_versions.add_revision_text_versions(tree)
1383 by Martin Pool
- untabify only
230
        inv = tree.inventory
231
        seen_ids = {}
232
        for file_id in inv:
233
            if file_id in seen_ids:
234
                raise BzrCheckError('duplicated file_id {%s} '
235
                                    'in inventory for revision {%s}'
236
                                    % (file_id, rev_id))
237
            seen_ids[file_id] = True
238
        for file_id in inv:
1092.2.20 by Robert Collins
symlink and weaves, whaddya know
239
            ie = inv[file_id]
240
            ie.check(self, rev_id, inv, tree)
1383 by Martin Pool
- untabify only
241
        seen_names = {}
242
        for path, ie in inv.iter_entries():
243
            if path in seen_names:
244
                raise BzrCheckError('duplicated path %s '
245
                                    'in inventory for revision {%s}'
246
                                    % (path, rev_id))
247
            seen_names[path] = True
1349 by Martin Pool
- more refactoring of check code
248
1347 by Martin Pool
- refactor check code into method object
249
1449 by Robert Collins
teach check about ghosts
250
def check(branch, verbose):
1732.2.4 by Martin Pool
Split check into Branch.check and Repository.check
251
    """Run consistency checks on a branch.
252
    
253
    Results are reported through logging.
254
    
255
    :raise BzrCheckError: if there's a consistency error.
256
    """
257
    branch.lock_read()
258
    try:
259
        branch_result = branch.check()
260
        repo_result = branch.repository.check([branch.last_revision()])
261
    finally:
262
        branch.unlock()
263
    branch_result.report_results(verbose)
264
    repo_result.report_results(verbose)
2745.6.47 by Andrew Bennetts
Move check_parents out of VersionedFile.
265
266
267