← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/check.py

update TODO

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/check.py
15
15
# along with this program; if not, write to the Free Software
16
16
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
17
17
 
18
 
 
19
 
 
20
 
######################################################################
21
 
# consistency checks
22
 
 
23
 
import sys
24
 
from sets import Set
25
 
 
26
 
from trace import mutter
27
 
from errors import bailout
28
 
import osutils
29
 
 
30
 
def check(branch, progress=True):
31
 
    out = sys.stdout
32
 
 
33
 
    # TODO: factor out
34
 
    if not (hasattr(out, 'isatty') and out.isatty()):
35
 
        progress=False
36
 
 
37
 
    if progress:
38
 
        def p(m):
39
 
            mutter('checking ' + m)
40
 
            out.write('\rchecking: %-50.50s' % m)
41
 
            out.flush()
42
 
    else:
43
 
        def p(m):
44
 
            mutter('checking ' + m)
45
 
 
46
 
    p('history of %r' % branch.base)
47
 
    last_ptr = None
48
 
    checked_revs = Set()
49
 
    
50
 
    history = branch.revision_history()
51
 
    revno = 0
52
 
    revcount = len(history)
53
 
 
54
 
    checked_texts = {}
55
 
    
56
 
    for rid in history:
57
 
        revno += 1
58
 
        p('revision %d/%d' % (revno, revcount))
59
 
        mutter('    revision {%s}' % rid)
60
 
        rev = branch.get_revision(rid)
61
 
        if rev.revision_id != rid:
62
 
            bailout('wrong internal revision id in revision {%s}' % rid)
63
 
        if rev.precursor != last_ptr:
64
 
            bailout('mismatched precursor in revision {%s}' % rid)
65
 
        last_ptr = rid
66
 
        if rid in checked_revs:
67
 
            bailout('repeated revision {%s}' % rid)
68
 
        checked_revs.add(rid)
69
 
 
70
 
        ## TODO: Check all the required fields are present on the revision.
71
 
 
72
 
        inv = branch.get_inventory(rev.inventory_id)
73
 
        seen_ids = Set()
74
 
        seen_names = Set()
75
 
 
76
 
        p('revision %d/%d file ids' % (revno, revcount))
 
18
# TODO: Check ancestries are correct for every revision: includes
 
19
# every committed so far, and in a reasonable order.
 
20
 
 
21
# TODO: Also check non-mainline revisions mentioned as parents.
 
22
 
 
23
# TODO: Check for extra files in the control directory.
 
24
 
 
25
# TODO: Check revision, inventory and entry objects have all 
 
26
# required fields.
 
27
 
 
28
# TODO: Get every revision in the revision-store even if they're not
 
29
# referenced by history and make sure they're all valid.
 
30
 
 
31
import bzrlib.ui
 
32
from bzrlib.trace import note, warning
 
33
from bzrlib.osutils import rename, sha_string, fingerprint_file
 
34
from bzrlib.trace import mutter
 
35
from bzrlib.errors import BzrCheckError, NoSuchRevision
 
36
from bzrlib.inventory import ROOT_ID
 
37
 
 
38
 
 
39
class Check(object):
 
40
    """Check a branch"""
 
41
 
 
42
    def __init__(self, branch):
 
43
        self.branch = branch
 
44
        self.checked_text_cnt = 0
 
45
        self.checked_rev_cnt = 0
 
46
        self.ghosts = []
 
47
        self.repeated_text_cnt = 0
 
48
        self.missing_parent_links = {}
 
49
        self.missing_inventory_sha_cnt = 0
 
50
        self.missing_revision_cnt = 0
 
51
        # maps (file-id, version) -> sha1
 
52
        self.checked_texts = {}
 
53
        self.checked_weaves = {}
 
54
 
 
55
    def check(self):
 
56
        self.branch.lock_read()
 
57
        self.progress = bzrlib.ui.ui_factory.progress_bar()
 
58
        try:
 
59
            self.progress.update('retrieving inventory', 0, 0)
 
60
            # do not put in init, as it should be done with progess,
 
61
            # and inside the lock.
 
62
            self.inventory_weave = self.branch._get_inventory_weave()
 
63
            self.history = self.branch.revision_history()
 
64
            if not len(self.history):
 
65
                # nothing to see here
 
66
                return
 
67
            self.plan_revisions()
 
68
            revno = 0
 
69
            self.check_weaves()
 
70
            while revno < len(self.planned_revisions):
 
71
                rev_id = self.planned_revisions[revno]
 
72
                self.progress.update('checking revision', revno,
 
73
                                     len(self.planned_revisions))
 
74
                revno += 1
 
75
                self.check_one_rev(rev_id)
 
76
        finally:
 
77
            self.progress.clear()
 
78
            self.branch.unlock()
 
79
 
 
80
    def plan_revisions(self):
 
81
        if not self.branch.revision_store.listable():
 
82
            self.planned_revisions = self.branch.get_ancestry(self.history[-1])
 
83
            self.planned_revisions.remove(None)
 
84
            # FIXME progress bars should support this more nicely.
 
85
            self.progress.clear()
 
86
            print ("Checking reachable history -"
 
87
                   " for a complete check use a local branch.")
 
88
            return
 
89
        
 
90
        self.planned_revisions = set(self.branch.revision_store)
 
91
        inventoried = set(self.inventory_weave.names())
 
92
        awol = self.planned_revisions - inventoried
 
93
        if len(awol) > 0:
 
94
            raise BzrCheckError('Stored revisions missing from inventory'
 
95
                '{%s}' % ','.join([f for f in awol]))
 
96
        self.planned_revisions = list(self.planned_revisions)
 
97
 
 
98
    def report_results(self, verbose):
 
99
        note('checked branch %s format %s',
 
100
             self.branch.base, 
 
101
             self.branch._branch_format)
 
102
 
 
103
        note('%6d revisions', self.checked_rev_cnt)
 
104
        note('%6d unique file texts', self.checked_text_cnt)
 
105
        note('%6d repeated file texts', self.repeated_text_cnt)
 
106
        note('%6d weaves', len(self.checked_weaves))
 
107
        if self.missing_inventory_sha_cnt:
 
108
            note('%6d revisions are missing inventory_sha1',
 
109
                 self.missing_inventory_sha_cnt)
 
110
        if self.missing_revision_cnt:
 
111
            note('%6d revisions are mentioned but not present',
 
112
                 self.missing_revision_cnt)
 
113
        if len(self.ghosts):
 
114
            note('%6d ghost revisions', len(self.ghosts))
 
115
            if verbose:
 
116
                for ghost in self.ghosts:
 
117
                    note('      %s', ghost)
 
118
        if len(self.missing_parent_links):
 
119
            note('%6d revisions missing parents in ancestry', 
 
120
                 len(self.missing_parent_links))
 
121
            if verbose:
 
122
                for link, linkers in self.missing_parent_links.items():
 
123
                    note('      %s should be in the ancestry for:', link)
 
124
                    for linker in linkers:
 
125
                        note('       * %s', linker)
 
126
 
 
127
    def check_one_rev(self, rev_id):
 
128
        """Check one revision.
 
129
 
 
130
        rev_id - the one to check
 
131
 
 
132
        last_rev_id - the previous one on the mainline, if any.
 
133
        """
 
134
 
 
135
        # mutter('    revision {%s}', rev_id)
 
136
        branch = self.branch
 
137
        try:
 
138
            rev_history_position = self.history.index(rev_id)
 
139
        except ValueError:
 
140
            rev_history_position = None
 
141
        last_rev_id = None
 
142
        if rev_history_position:
 
143
            rev = branch.get_revision(rev_id)
 
144
            if rev_history_position > 0:
 
145
                last_rev_id = self.history[rev_history_position - 1]
 
146
        else:
 
147
            rev = branch.get_revision(rev_id)
 
148
                
 
149
        if rev.revision_id != rev_id:
 
150
            raise BzrCheckError('wrong internal revision id in revision {%s}'
 
151
                                % rev_id)
 
152
 
 
153
        # check the previous history entry is a parent of this entry
 
154
        if rev.parent_ids:
 
155
            if last_rev_id is not None:
 
156
                for parent_id in rev.parent_ids:
 
157
                    if parent_id == last_rev_id:
 
158
                        break
 
159
                else:
 
160
                    raise BzrCheckError("previous revision {%s} not listed among "
 
161
                                        "parents of {%s}"
 
162
                                        % (last_rev_id, rev_id))
 
163
            for parent in rev.parent_ids:
 
164
                if not parent in self.planned_revisions:
 
165
                    missing_links = self.missing_parent_links.get(parent, [])
 
166
                    missing_links.append(rev_id)
 
167
                    self.missing_parent_links[parent] = missing_links
 
168
                    # list based so somewhat slow,
 
169
                    # TODO have a planned_revisions list and set.
 
170
                    if self.branch.has_revision(parent):
 
171
                        missing_ancestry = self.branch.get_ancestry(parent)
 
172
                        for missing in missing_ancestry:
 
173
                            if (missing is not None 
 
174
                                and missing not in self.planned_revisions):
 
175
                                self.planned_revisions.append(missing)
 
176
                    else:
 
177
                        self.ghosts.append(rev_id)
 
178
        elif last_rev_id:
 
179
            raise BzrCheckError("revision {%s} has no parents listed "
 
180
                                "but preceded by {%s}"
 
181
                                % (rev_id, last_rev_id))
 
182
 
 
183
        if rev.inventory_sha1:
 
184
            inv_sha1 = branch.get_inventory_sha1(rev_id)
 
185
            if inv_sha1 != rev.inventory_sha1:
 
186
                raise BzrCheckError('Inventory sha1 hash doesn\'t match'
 
187
                    ' value in revision {%s}' % rev_id)
 
188
        else:
 
189
            missing_inventory_sha_cnt += 1
 
190
            mutter("no inventory_sha1 on revision {%s}", rev_id)
 
191
        self._check_revision_tree(rev_id)
 
192
        self.checked_rev_cnt += 1
 
193
 
 
194
    def check_weaves(self):
 
195
        """Check all the weaves we can get our hands on.
 
196
        """
 
197
        n_weaves = 1
 
198
        weave_ids = []
 
199
        if self.branch.weave_store.listable():
 
200
            weave_ids = list(self.branch.weave_store)
 
201
            n_weaves = len(weave_ids)
 
202
        self.progress.update('checking weave', 0, n_weaves)
 
203
        self.inventory_weave.check(progress_bar=self.progress)
 
204
        for i, weave_id in enumerate(weave_ids):
 
205
            self.progress.update('checking weave', i, n_weaves)
 
206
            w = self.branch.weave_store.get_weave(weave_id,
 
207
                    self.branch.get_transaction())
 
208
            # No progress here, because it looks ugly.
 
209
            w.check()
 
210
            self.checked_weaves[weave_id] = True
 
211
 
 
212
    def _check_revision_tree(self, rev_id):
 
213
        tree = self.branch.revision_tree(rev_id)
 
214
        inv = tree.inventory
 
215
        seen_ids = {}
77
216
        for file_id in inv:
78
217
            if file_id in seen_ids:
79
 
                bailout('duplicated file_id {%s} in inventory for revision {%s}'
80
 
                        % (file_id, rid))
81
 
            seen_ids.add(file_id)
82
 
 
83
 
        i = 0
84
 
        len_inv = len(inv)
 
218
                raise BzrCheckError('duplicated file_id {%s} '
 
219
                                    'in inventory for revision {%s}'
 
220
                                    % (file_id, rev_id))
 
221
            seen_ids[file_id] = True
85
222
        for file_id in inv:
86
 
            i += 1
87
 
            if (i % 100) == 0:
88
 
                p('revision %d/%d file text %d/%d' % (revno, revcount, i, len_inv))
89
 
 
90
223
            ie = inv[file_id]
91
 
 
92
 
            if ie.parent_id != None:
93
 
                if ie.parent_id not in seen_ids:
94
 
                    bailout('missing parent {%s} in inventory for revision {%s}'
95
 
                            % (ie.parent_id, rid))
96
 
 
97
 
            if ie.kind == 'file':
98
 
                if ie.text_id in checked_texts:
99
 
                    fp = checked_texts[ie.text_id]
100
 
                else:
101
 
                    if not ie.text_id in branch.text_store:
102
 
                        bailout('text {%s} not in text_store' % ie.text_id)
103
 
 
104
 
                    tf = branch.text_store[ie.text_id]
105
 
                    fp = osutils.fingerprint_file(tf)
106
 
                    checked_texts[ie.text_id] = fp
107
 
 
108
 
                if ie.text_size != fp['size']:
109
 
                    bailout('text {%s} wrong size' % ie.text_id)
110
 
                if ie.text_sha1 != fp['sha1']:
111
 
                    bailout('text {%s} wrong sha1' % ie.text_id)
112
 
            elif ie.kind == 'directory':
113
 
                if ie.text_sha1 != None or ie.text_size != None or ie.text_id != None:
114
 
                    bailout('directory {%s} has text in revision {%s}'
115
 
                            % (file_id, rid))
116
 
 
117
 
        p('revision %d/%d file paths' % (revno, revcount))
 
224
            ie.check(self, rev_id, inv, tree)
 
225
        seen_names = {}
118
226
        for path, ie in inv.iter_entries():
119
227
            if path in seen_names:
120
 
                bailout('duplicated path %r in inventory for revision {%s}' % (path, revid))
121
 
            seen_names.add(path)
122
 
 
123
 
 
124
 
    p('done')
125
 
    if progress:
126
 
        print 
127
 
    print 'checked %d revisions, %d file texts' % (revcount, len(checked_texts))
128
 
 
 
228
                raise BzrCheckError('duplicated path %s '
 
229
                                    'in inventory for revision {%s}'
 
230
                                    % (path, rev_id))
 
231
            seen_names[path] = True
 
232
 
 
233
 
 
234
def check(branch, verbose):
 
235
    """Run consistency checks on a branch."""
 
236
    checker = Check(branch)
 
237
    checker.check()
 
238
    checker.report_results(verbose)