Launchpad has removed privileges that UDD importer requires to function
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Critical
|
Francis J. Lacoste | ||
Ubuntu Distributed Development |
Fix Released
|
Critical
|
Unassigned |
Bug Description
Launchpad has removed the privilege of ~ubuntu-branches to set series source package official branches, which the importer requires to function, and has not replaced it with an adequate substitute. Many UDD imports are failing as a result.
08:25 < maxb> uhoh, what? Is someone on the case with all those UDD failures?
08:25 < maxb> Please tell me people haven't been removing ~ubuntu-branches without replacing the importer's needs
08:26 < lifeless> the importers needs were discussed with james_w, poole and the techboard
08:26 < lifeless> it should be allowing any uploader to push to any official branch
08:26 < maxb> it's the set_official API call that is getting a 401 unauthorized
08:27 < lifeless> that's also gated on uploader I think, though perhaps that patch hasn't landed yet
08:27 < maxb> (<SourcePackage <Distribution 'Ubuntu' (ubuntu)> <DistroSeries u'lucid'> <SourcePackageName 'linux-
08:27 < lifeless> but if the patch hasn't landed the old rules should be applying
08:28 < lifeless> ah, perhaps frozen series are going to be a head-desk moment.
08:28 < lifeless> wgrant: ^ thoughts?
08:29 < maxb> (<SourcePackage <Distribution 'Debian' (debian)> <DistroSeries u'experimental'> <SourcePackageName 'darcs-monitor'>>, 'setBranch', 'launchpad.Edit')
08:30 < maxb> The importer also needs to be able to set official branches for Debian
08:49 < wgrant> lifeless: By frozen you mean released?
08:52 < lifeless> thingy :P
08:53 < wgrant> lifeless: FROZEN should be OK. CURRENT/
08:54 < lifeless> wgrant: why won't they be?
08:55 < wgrant> lifeless: Because uploads to the release pocket of them are rejected.
08:55 < wgrant> lifeless: So you can't set the branch for the release pocket.
08:56 < lifeless> does soyuz have a test that will francis should use?
08:56 < wgrant> Not sure.
08:56 < wgrant> I need to go now, sorry.
08:57 < lifeless> viao
Related branches
- Julian Edwards (community): Approve
-
Diff: 114 lines (+56/-10)2 files modifiedlib/canonical/launchpad/security.py (+7/-6)
lib/lp/registry/tests/test_sourcepackage.py (+49/-4)
Changed in launchpad: | |
importance: | Undecided → Critical |
Changed in udd: | |
importance: | Undecided → Critical |
tags: | added: regression udd |
Changed in launchpad: | |
status: | New → In Progress |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in launchpad: | |
status: | Fix Committed → Fix Released |
On Tue, 14 Jun 2011 08:50:51 -0000, Max Bowsher <email address hidden> wrote:
> Public bug reported:
>
> Launchpad has removed the privilege of ~ubuntu-branches to set series
> source package official branches, which the importer requires to
> function, and has not replaced it with an adequate substitute. Many UDD
> imports are failing as a result.
Max has a workaround that lessens the impact. Francis has a code change
that should fix some more cases.
<flacoste> james_w: i have a code solution
<james_w> flacoste, cool
<flacoste> for frozen series, we need to check the UPDATE pocket
<james_w> flacoste, that will prevent setting official branches for obsolete series?
<flacoste> probably
<flacoste> i don't know
<flacoste> actually
<flacoste> i don't know the intimates of checkUpload
<flacoste> but would that be a problem?
<james_w> it sounds better than what we have, but I'm wondering if we actually want a different set of rules for this
<flacoste> why should we care about obsolete series
<flacoste> ?
<james_w> c.f. changing the packaging links
<james_w> well, the importer currently sets them up when it does the initial import of an old package
<flacoste> it could simply not set them up
<james_w> obviously most will be done already
<james_w> it could
<flacoste> obsolete is obsolete
<james_w> do you still propose setting me/pkg_import to be maintainer of Debian?
<flacoste> in a way
<flacoste> either that
<james_w> yeah, but it's still useful to easily refer to it sometimes
<flacoste> but i think we should simply mirror the upload privileges from Ubuntu
<james_w> obviously less useful over time
<james_w> that sounds sensible
Thanks,
James