codetree barfs unnecessarily if parent branch incorrectly configured

Bug #1247028 reported by Tom Haddon
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
codetree
Fix Released
High
David Ames

Bug Description

<mthaddon> wgrant: hi, can you explain this to me? https://pastebin.canonical.com/99697/ I'm not quite sure what the error means
<wgrant> mthaddon: The parent_location defined in the branch.conf that was pushed to Launchpad is invalid. That shouldn't affect any remote operations other than bzr info.
<wgrant> Do you need bzr info for any particular reason?
<wgrant> It's not usually very interesting on a remote brnach.
<wgrant> Also, is that a very old Canonical server or just a conveniently named desktop/laptop?
<mthaddon> wgrant: it's confusing codetree - https://ci.admin.canonical.com/job/mojo-cdo-omnibus/10/console
<mthaddon> wgrant: it's my laptop :)
<mthaddon> I guess I need to check what codetree is actually doing here...
<wgrant> What is codetree?
<mthaddon> it's part of the mojo project - kind of like config-manager
<wgrant> Weird, I wonder why it's doing that.
<wgrant> Anyway, you can fix it with 'bzr config -d lp:goamz --remove parent_location'
<mthaddon> oh cool
<mthaddon> do I need any special perms for that?
<mthaddon> this seems to be the problem: https://pastebin.canonical.com/99699/ (in codetree)
<wgrant> You need to be able to write to the branch on LP
<wgrant> Hum
<wgrant> Is it just doing that to check whether it exists? :/
<wgrant> I'd consider just doing bzr revno instead.
<mthaddon> ok, interesting, thanks
<mthaddon> that does indeed seem to have fixed it - thanks muchly
<wgrant> revno should be like two HPSS calls, so much faster
<wgrant> Though from the DC that matters less, I guess.
<mthaddon> seems to be another branch with the same issue. I'll fix it as above, but file a bug against codetree
<wgrant> (parent_location is mostly used for determining the default for 'bzr pull' or 'bzr merge', so it's pretty much useless having it in a branch on LP, so it's fine to remove, particularly if it's invalid)

David Ames (thedac)
Changed in codetree:
assignee: nobody → David Ames (thedac)
importance: Undecided → High
status: New → Fix Committed
status: Fix Committed → Incomplete
status: Incomplete → Fix Committed
Revision history for this message
David Ames (thedac) wrote :
David Ames (thedac)
Changed in codetree:
status: Fix Committed → Fix Released
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.