Remove Packages-arch-specific use from Soyuz
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Steve Kowalik |
Bug Description
So, after having a long think about this, and many discussions surrounding it, I've come to the conclusion that Packages-
1) Not wanting to waste build time on slow arches where things haven't yet been ported:
Our slow arches have plenty of capacity to build (or fail) the world, and the inverse argument here is that restricting arches from P-a-s hides build failures from the FTBFS list and porters have no idea what needs working on. A failure log is a GOOD thing, in that it tells us what could be fixed/ported.
2) Arch-restrictions in source used to not work and not be respected by build infrastructres and tools:
This hasn't been true for a very long time, and for things that really are completely unportable and never will be ported (ie: nvidia drivers, packages whose sole raison d'etre are driving specific types of CPUs, etc), we can arch-restrict in source, and have no need to use P-a-s, and this is being done in most cases today.
There are plenty of spots, I'm led to understand, where soyuz doesn't always respect P-a-s today, and there are, as far as I can tell, no reasons to keep using it at all. Removing it would reduce complexity in a fair few areas, so this is something we should probably just suck up, tear it all out, QA, and rejoice.
NOTE: this is not about replacing P-a-s with some DB-driven madness, or improving on it, but removing it entirely. It's outlived its usefulness. Also, fixing this bug will obsolete https:/
Related branches
- William Grant: Approve (code)
-
Diff: 676 lines (+53/-389)7 files modifiedlib/lp/soyuz/adapters/buildarch.py (+29/-160)
lib/lp/soyuz/adapters/tests/test_buildarch.py (+10/-159)
lib/lp/soyuz/interfaces/publishing.py (+1/-4)
lib/lp/soyuz/model/publishing.py (+5/-8)
lib/lp/soyuz/model/queue.py (+1/-6)
lib/lp/soyuz/scripts/add_missing_builds.py (+6/-13)
lib/lp/soyuz/scripts/tests/test_add_missing_builds.py (+1/-39)
tags: | added: lp-soyuz soyuz-build soyuz-core |
Changed in launchpad: | |
importance: | Undecided → High |
Changed in launchpad: | |
assignee: | nobody → Steve Kowalik (stevenk) |
status: | New → In Progress |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in launchpad: | |
status: | Fix Committed → Fix Released |
Fixed in stable r16661 <http:// bazaar. launchpad. net/~launchpad- pqm/launchpad/ stable/ revision/ 16661>.