Bileto

Unable to run some retries

Bug #1542239 reported by Timo Jyrinki
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Auto Package Testing
Fix Released
Low
Martin Pitt
Bileto
Invalid
Undecided
Unassigned

Bug Description

The new retry feature is working great otherwise, but I'm getting an error "You are not allowed to upload qtcreator-plugin-ubuntu or qtdeclarative-opensource-src-gles to Ubuntu, thus you are not allowed to use this service." when opening https://autopkgtest.ubuntu.com/retry.cgi?release=xenial&arch=amd64&package=qtcreator-plugin-ubuntu&trigger=qtdeclarative-opensource-src-gles%2F5.5.1-2ubuntu4&ppa=ci-train-ppa-service%2Fstable-phone-overlay&ppa=ci-train-ppa-service%2Flanding-023

Which is weird because a) both packages are in universe b) I'm a MOTU c) I was able to retry the same qtcreator-plugin-ubuntu tests from under the qtdeclarative-opensource-src vs. this being qtdeclarative-opensource-src-gles. The non-gles qtdeclarative is in main (and I've PPU rights for that which is why it worked) while the gles qtdeclarative is in universe.

Revision history for this message
Robert Bruce Park (robru) wrote :

That's super weird because it calls the same checkUpload code that the train does during publishing of those packages, there shouldn't be any difference.

Changed in bileto:
assignee: nobody → Martin Pitt (pitti)
Revision history for this message
Martin Pitt (pitti) wrote :

The actual checkUpload() call seems to work fine:

wget -O- 'https://api.launchpad.net/1.0/ubuntu/+archive/primary?ws.op=checkUpload&distroseries=https%3A//api.launchpad.net/1.0/ubuntu/xenial&person=https%3A//api.launchpad.net/1.0/~timo-jyrinki&component=universe&pocket=Proposed&sourcepackagename=qtcreator-plugin-ubuntu'

So I suppose what happens is that in autopkgtest-retrier the call is being done with "&component=main".

The component is determined from the archive that you request the retry in, but that's a PPA in this case -- and PPAs always have "main" as their only component.

Asking the PPA about upload rights doesn't make much sense IMHO, as direct upload rights are also quite limited. So I guess I'll change this to always query the component from the Ubuntu archive, and separate this out from is_valid_package_version().

Changed in auto-package-testing:
status: New → In Progress
assignee: nobody → Martin Pitt (pitti)
Revision history for this message
Martin Pitt (pitti) wrote :

Not a bug in bileto.

Changed in bileto:
assignee: Martin Pitt (pitti) → nobody
status: New → Invalid
Changed in auto-package-testing:
importance: Undecided → Low
Revision history for this message
Robert Bruce Park (robru) wrote :

Yeah that makes sense, in the train we also only ever call checkUpload against the archive even when copying to PPAs. The convention is to pretend that these PPAs are equivalent to the archive.

Revision history for this message
Martin Pitt (pitti) wrote :

Should be fixed with https://git.launchpad.net/~canonical-foundations/+git/autopkgtest-retrier/commit/?id=7b2485291 . Timo, can you double-check please, and reopen if you still have trouble?

Changed in auto-package-testing:
status: In Progress → Fix Released
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Still getting the same message when opening https://autopkgtest.ubuntu.com/retry.cgi?release=xenial&arch=amd64&package=qtcreator-plugin-ubuntu&trigger=qtdeclarative-opensource-src-gles%2F5.5.1-2ubuntu4&ppa=ci-train-ppa-service%2Fstable-phone-overlay&ppa=ci-train-ppa-service%2Flanding-023 and logging in.

Changed in auto-package-testing:
status: Fix Released → Confirmed
Revision history for this message
Martin Pitt (pitti) wrote :

Fixed harder in https://git.launchpad.net/~canonical-foundations/+git/autopkgtest-retrier/commit/?id=365b761b1 . Retry, please?

Changed in auto-package-testing:
status: Confirmed → In Progress
status: In Progress → Fix Released
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

This fixes it, thank you!

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.