can't pull into a --1.9-rich-root format branch

Bug #571055 reported by Michael Hudson-Doyle
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Low
Unassigned
Bazaar Git Plugin
Invalid
Medium
Unassigned
Launchpad itself
Invalid
High
Jelmer Vernooij

Bug Description

mwh@grond:repos$ bzr init --1.9-rich-root etckeeper
Created a standalone tree (format: 1.9-rich-root)
mwh@grond:repos$ bzr pull git://git.kitenet.net/etckeeper -d etckeeper/
bzr: ERROR: An inconsistent delta was supplied involving u'post-apt.d', None
reason: entry with file_id None None

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

I can reproduce this; really weird it works with 2a but not 1.9-rich-root.

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Marking as affects bzr since it would be nice if it could be consistent. If bzr-git is generating invalid inventory delta's (which I suspect it is) it should be consistently raising an error.

Changed in bzr-git:
status: New → Triaged
importance: Undecided → Medium
Changed in bzr:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

For some reason apt-post.d doesn't exist in the base inventory.

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

I've just narrowed this down.

bzr-git is caching RevisionTree's. These trees are not modified in any way after they have been retrieved by bzr-git in any way, other than that their inventory is passed into create_inventory_by_delta.

However, when using a 1.9-rich-root format the inventories seem to change under the hood - when getting a RevisionTree out of the cache after a while sometimes tree.inventory.revision_id no longer matches tree.get_revision_id().

bzr-git now has more appropriate checks for this in place.

Changed in bzr-git:
status: Triaged → Invalid
tags: added: 1.9
Revision history for this message
Martin Pool (mbp) wrote :

Can someone paste the traceback in bzr where this is occurring?

Jelmer Vernooij (jelmer)
Changed in launchpad:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Jelmer Vernooij (jelmer)
tags: added: qa-ok
Jelmer Vernooij (jelmer)
Changed in launchpad:
status: Triaged → Fix Committed
Jelmer Vernooij (jelmer)
Changed in launchpad:
status: Fix Committed → Fix Released
Jelmer Vernooij (jelmer)
Changed in launchpad:
status: Fix Released → Triaged
tags: removed: qa-ok
Changed in launchpad:
importance: Medium → High
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

This will no longer be relevant for Launchpad when bug 432217 is fixed.

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

This is no longer relevant for Launchpad, as a fix for bug 432217 has landed.

Changed in launchpad:
status: Triaged → Invalid
Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.