Launchpad itself

Initializing a distroseries that doesn't use the parent's archindep arch will fail

Bug #869221 reported by Julian Edwards
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Critical
Raphaël Badin

Bug Description

Traceback (most recent call last):
  File "/srv/launchpad.net/production/launchpad-rev-14085.1/lib/lp/services/job/runner.py", line 217, in runJob
    job.run()
  File "/srv/launchpad.net/production/launchpad-rev-14085.1/lib/lp/soyuz/model/initializedistroseriesjob.py", line 198, in run
    ids.initialize()
  File "/srv/launchpad.net/production/launchpad-rev-14085.1/lib/lp/soyuz/scripts/initialize_distroseries.py", line 243, in initialize
    self._copy_architectures()
  File "/srv/launchpad.net/production/launchpad-rev-14085.1/lib/lp/soyuz/scripts/initialize_distroseries.py", line 354, in _copy_architectures
    self.derivation_parents[0].nominatedarchindep.architecturetag]
  File "/srv/launchpad.net/production/launchpad-rev-14085.1/lib/lp/registry/model/distroseries.py", line 350, in __getitem__
    return self.getDistroArchSeries(archtag)
  File "/srv/launchpad.net/production/launchpad-rev-14085.1/lib/lp/registry/model/distroseries.py", line 364, in getDistroArchSeries
    archtag, self.distribution.name, self.name))
NotFoundError: u'Unknown architecture i386 for ubuntu-leb oneiric'

We need to force the user to specify a different architecture if the parents archindep arch is not being copied.

Related branches

lp:~rvb/launchpad/bug-869221
Julian Edwards (community): Approve
Diff: 171 lines (+77/-11)
4 files modified
lib/lp/registry/stories/webservice/xx-derivedistroseries.txt (+3/-0)
lib/lp/registry/tests/test_initderiveddistroseries.py (+5/-3)
lib/lp/soyuz/scripts/initialize_distroseries.py (+28/-3)
lib/lp/soyuz/scripts/tests/test_initialize_distroseries.py (+41/-5)
lp:~rvb/launchpad/bug-869221-expose-nominatedarchindep
Benji York (community): Approve (code)
Diff: 108 lines (+27/-9)
5 files modified
lib/canonical/launchpad/interfaces/_schema_circular_imports.py (+7/-5)
lib/lp/registry/browser/tests/test_distroseries_webservice.py (+10/-0)
lib/lp/registry/interfaces/distroseries.py (+8/-3)
lib/lp/registry/stories/webservice/xx-distroseries.txt (+1/-0)
lib/lp/soyuz/scripts/initialize_distroseries.py (+1/-1)
lp:~rvb/launchpad/bug-869221-ui
Benji York (community): Approve (code)
Diff: 433 lines (+293/-11)
5 files modified
lib/lp/app/javascript/formwidgets/formwidgets.js (+69/-3)
lib/lp/app/javascript/formwidgets/tests/test_formwidgets.js (+80/-6)
lib/lp/registry/javascript/distroseries/initseries.js (+18/-2)
lib/lp/registry/javascript/distroseries/tests/test_widgets.js (+67/-0)
lib/lp/registry/javascript/distroseries/widgets.js (+59/-0)
lp:~rvb/launchpad/bug-869221-forcederivedarchinep
Julian Edwards (community): Approve
Diff: 486 lines (+177/-53)
7 files modified
lib/lp/registry/interfaces/distroseries.py (+15/-5)
lib/lp/registry/model/distroseries.py (+6/-5)
lib/lp/registry/tests/test_initderiveddistroseries.py (+2/-2)
lib/lp/soyuz/model/initializedistroseriesjob.py (+11/-5)
lib/lp/soyuz/scripts/initialize_distroseries.py (+35/-14)
lib/lp/soyuz/scripts/tests/test_initialize_distroseries.py (+57/-16)
lib/lp/soyuz/tests/test_initializedistroseriesjob.py (+51/-6)
lp:~rvb/launchpad/bug-869221-archindepwidget
Benji York (community): Approve (code)
Diff: 1055 lines (+539/-95)
7 files modified
lib/lp/app/javascript/formwidgets/formwidgets.js (+62/-15)
lib/lp/app/javascript/formwidgets/tests/test_formwidgets.js (+56/-8)
lib/lp/registry/javascript/distroseries/initseries.js (+83/-11)
lib/lp/registry/javascript/distroseries/tests/test_initseries.html (+18/-0)
lib/lp/registry/javascript/distroseries/tests/test_initseries.js (+131/-32)
lib/lp/registry/javascript/distroseries/tests/test_widgets.js (+107/-22)
lib/lp/registry/javascript/distroseries/widgets.js (+82/-7)
Julian Edwards (julian-edwards)
Changed in launchpad:
status: New → Triaged
importance: Undecided → Critical
tags: added: derivation
Raphaël Badin (rvb)
Changed in launchpad:
assignee: nobody → Raphaël Victor Badin (rvb)
status: Triaged → In Progress
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

r14121 in stable (http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/14121) is part of this bug's fix.

tags: added: qa-needstesting
William Grant (wgrant)
tags: added: qa-ok
removed: qa-needstesting
Revision history for this message
Ricardo Salveti (rsalveti) wrote :

Just tried to create a derived series by selecting just ARM and I got:
"The distroseries has no architectures selected to build architecture independent binaries."

So it seems that the fix is in place at least. In my case I guess the only way would be to also require the series to be built with i386 and armel.

Revision history for this message
Raphaël Badin (rvb) wrote :

A fifth branch is coming up to allow you to force the architecture used for arch indep packages. Right now, it's picked up from the parents'.

Revision history for this message
Julian Edwards (julian-edwards) wrote : Re: [Bug 869221] Re: Initializing a distroseries that doesn't use the parent's archindep arch will fail

On Monday 17 October 2011 01:42:57 you wrote:
> Just tried to create a derived series by selecting just ARM and I got:
> "The distroseries has no architectures selected to build architecture
> independent binaries."
>
> So it seems that the fix is in place at least. In my case I guess the
> only way would be to also require the series to be built with i386 and
> armel.

The fix is not completely in place yet, please don't initialise. Also, you
may have noticed that it failed because it went over the 4 hour transaction
limit. I think this is because you selected no packagesets and we've not
catered for that scenario performance-wise. I'll talk to you about this when
you are online.

Raphaël Badin (rvb)
tags: removed: qa-ok
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

Fixed in stable r14166 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/14166>.

tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Raphaël Badin (rvb)
tags: added: qa-ok
removed: qa-needstesting
William Grant (wgrant)
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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