Please remove badger/armhf binaries

Bug #1884753 reported by Lukas Märdian
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
badger (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Removal request
---------------

Please remove the binaries built from src:badger for all 32bit architectures from Groovy.

The binary package names are:
badger
golang-github-dgraph-io-badger-dev

The (release) arch that should get targeted for removal is:
armhf

It was removed from Debian as well:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=953370

Extended removal reason:

2.0.1-4 included a bunch of patches that made badger not only build
but also pass its own testsuite on 32bit architectures. This was a
mistake, because it was later realized that the trivial patches
created didn't actually make badger practically usable on 32bit
archs - as badger is usually built into another application and if that
application does any memory allocations then badger will not work.

A non-trivial amount of work is needed to fix badger on 32bit architectures and it seems upstream doesn't have interest in this at the moment. There has been no progress on https://github.com/dgraph-io/badger/issues/1384 even though we provided them with a 32bit CI test integration.

Original bug report
-------------------

badger 2.0.3-1 fails to build on armhf, due to running out of memory while executing its test suite inside the armhf build VM, during dh_auto_test stage.

This patch disables running the test suite during the build on armhf. The test suite will still be run via autopkgtest.

With this patch it built successfully in a PPA: https://launchpad.net/~slyon/+archive/ubuntu/testing/+packages

Lukas Märdian (slyon)
description: updated
tags: added: id-5eeb8f9af79fbd71a17226db
Revision history for this message
Lukas Märdian (slyon) wrote :

Actually, there seems to be a bigger, 32bit related issue:
https://buildd.debian.org/status/package.php?p=badger&suite=sid

Let me check that in detail...

Changed in badger (Ubuntu):
status: New → Incomplete
Revision history for this message
Sebastien Bacher (seb128) wrote :

unsubscribing the sponsors team for now while it's being investigated

Revision history for this message
Lukas Märdian (slyon) wrote :

Upstream is aware of the problem of broken 32 bit tests, they did break them repeatedly in the past. They want to setup a 32 bit CI test infrastructure, to keep an eye on this, otherwise they might break it again in the future.

https://github.com/dgraph-io/badger/issues/1384

Lukas Märdian (slyon)
description: updated
Revision history for this message
Lukas Märdian (slyon) wrote :
summary: - Fix armhf build
+ Please remove badger/armhf binaries
Lukas Märdian (slyon)
description: updated
Changed in badger (Ubuntu):
status: Incomplete → New
description: updated
description: updated
description: updated
Revision history for this message
Iain Lane (laney) wrote :

laney@dev> ./remove-package -m "[from Debian] ROM; 32bit archs not supported (LP: #1884753)" -a armhf -s groovy -b badger golang-github-dgraph-io-badger-dev
Removing packages from groovy:
 badger 2.0.1-3 in groovy armhf
 golang-github-dgraph-io-badger-dev 2.0.1-3 in groovy armhf
Comment: [from Debian] ROM; 32bit archs not supported (LP: #1884753)
Remove [y|N]? y
2 packages successfully removed.

Changed in badger (Ubuntu):
status: New → 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.