check reports "Missing inventory {('TREE_ROOT'..." for trivial non-rich-root branch
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Bazaar |
Critical
|
Robert Collins | ||
| 2.0 |
Critical
|
Robert Collins |
Bug Description
With current bzr.dev:
andrew@
Created a standalone tree (format: 1.9)
andrew@
andrew@
Committing to: /tmp/non-rr-test/
Committed revision 1.
andrew@
Checking working tree at '/tmp/non-rr-test'.
Checking branch at 'file:/
Checking repository at 'file:/
checked repository <bzrlib.
1 revisions
0 file-ids
Missing inventory {('TREE_ROOT', '<email address hidden>')}
checked branch file://
"bzr check" on a trivial, new branch should not be finding problems! Rich-root formats, like 2a, seem to be unaffected. My guess is that this is a bug in check on non-rich-root repositories (rather than a bug in what is stored in the repository that check discovers). Upgrading to 2a fixes the problem.
(There have been other reports that look like this problem, e.g. <https:/
As this doesn't affect 2a branches (and is corrected by upgrading to 2a) I'm not targetting this to 2.0. But I'm still marking it as High as it seems pretty serious to me.
Related branches
- bzr-core: Pending requested 2009-08-28
- Diff: 1359 lines (has conflicts)
- Martin Pool: Approve on 2009-08-28
- Vincent Ladeuil: Approve on 2009-08-28
- Andrew Bennetts: Approve on 2009-08-28
- Diff: 98 lines (has conflicts)
Neil Martinsen-Burrell (nmb) wrote : | #1 |
Ian Clatworthy (ian-clatworthy) wrote : Re: [Bug 416732] [NEW] check reports "Missing inventory {('TREE_ROOT'..." for trivial non-rich-root branch | #2 |
Andrew Bennetts wrote:
> As this doesn't affect 2a branches (and is corrected by upgrading to 2a)
> I'm not targetting this to 2.0. But I'm still marking it as High as it
> seems pretty serious to me.
And it is going to impact 2.0 support pretty heavily IMO, given the
Upgrade Guide tells users to run check prior to upgrading their
repositories and branches. Every single person doing that will
1. find the problem
2. try reconcile
3. see the problem is still there
4. get confused
5. maybe raise a bug or ping us on IRC.
So I think we definitely need to keep this in scope for 2.0.
Ian C.
Andrew Bennetts (spiv) wrote : | #3 |
Fair enough. I'm targetting to 2.0.
Changed in bzr: | |
milestone: | none → 2.0 |
Changed in bzr: | |
importance: | High → Critical |
Robert Collins (lifeless) wrote : | #4 |
I intend to get this done first thing tomorrow, however I haven't made any actual progress towards it so far.
Changed in bzr: | |
assignee: | nobody → Robert Collins (lifeless) |
Changed in bzr: | |
status: | Confirmed → In Progress |
Danny van Heumen (danny.vanheumen) wrote : | #5 |
It's not much information, but I hope it proves useful:
1. Every revision seems to be reported, or at least: revno: 770 --> 771 errors, in one of my repositories.
2. I've tried bugtracing this a bit, and I've discovered that:
if I set a breakpoint on: bzrlib/knit.py:1416 (absent_keys = keys.difference
I've tried to trace this bug, since I encounterd it a few days ago, but this is way out of my league. (With my very limited experience of python and bzr development.) I'm posting this here so it may possibly help others. (Can't blame a man for trying right? :P)
Changed in bzr: | |
status: | In Progress → Fix Committed |
Martin Pool (mbp) wrote : | #6 |
As explained in https:/
Changed in bzr: | |
status: | Fix Committed → Fix Released |
Edmundo (eantoranz) wrote : | #7 |
I'm seeing this problem on bzr 1.18 (updated, on kubuntu jaunty).
I just branched from another repo and I get a Missing Inventory line for every revision the branch has.
Standalone tree (format: pack-0.92)
Robert Collins (lifeless) wrote : Re: [Bug 416732] Re: check reports "Missing inventory {('TREE_ROOT'..." for trivial non-rich-root branch | #8 |
This is fixed in 2.0
I don't remember the number that its a dupe of, sorry.
status invalid
Changed in bzr: | |
status: | Fix Released → Invalid |
Changed in bzr: | |
status: | Invalid → Fix Released |
status: | Fix Released → Invalid |
Respectfully, I think that this bug is Critical for releasing 2a as the default format. Check should not be spouting irrelevant errors in a properly-working upgrade process. A clean upgrade process is imperative for Bazaar's roll out of a new default format. As it is, rather than all check output being problems, some of the output is irrelevant and can safely be ignored (or so we think). Again, check is part of the upgrade process as described at http:// doc.bazaar- vcs.org/ bzr.dev- html/en/ upgrade- guide/index. html and check has broken its commitment that "The working tree and branch checks will only give output if a problem is detected." As such, the upgrade process is broken and I don't think that 2a should be made the default until the upgrade process is working *right*.
(See https:/ /lists. ubuntu. com/archives/ bazaar/ 2009q3/ 061248. html for another example of the same problem.)