juju names arm arch 'arm' internally, but 'armhf' in tools

Bug #1285410 reported by Jason Hobbs
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
High
Jason Hobbs
1.18
Fix Released
High
John A Meinel

Bug Description

I'm trying to bootstrap juju via maas onto a highbank (armhf) system. If I provide no constraints, juju picks amd64 tools for the node, which obviously won't work.

If I provide a --constraints="arch=armhf" I get:

ubuntu@maas-dev:~$ juju bootstrap -v -v --constraints "tags=calxeda-highbank arch=armhf"
error: invalid value "tags=calxeda-highbank arch=armhf" for flag --constraints: bad "arch" constraint: "armhf" not recognized

It looks like juju-core expects 'arm' for the arm architecture, but that results in no tools being found, as the arch is set to armhf in the released:tools.json here https://streams.canonical.com/juju/tools/streams/v1/com.ubuntu.juju:released:tools.json

This also applies when syncing tools. armhf tools aren't synced because the default tools constraint is:

                logger.Debugf("no architecture specified when finding tools, looking for any")
                toolsConstraint.Arches = []string{"amd64", "i386", "arm"}

This is with 1.17.2-trusty-amd64

Related branches

Curtis Hovey (sinzui)
tags: added: armhf constraints
Changed in juju-core:
status: New → Triaged
importance: Undecided → High
summary: - bad "arch" constraint: "armhf" not recognized
+ juju names arm arch 'arm' internally, but 'armhf' in tools
description: updated
Revision history for this message
Jason Hobbs (jason-hobbs) wrote :

I posted a branch that lets this work for me here lp:~jason-hobbs/juju-core/lp-1285410-fix-arm-armhf-mistmatch

Revision history for this message
Dave Cheney (dave-cheney) wrote : Re: [Bug 1285410] Re: juju names arm arch 'arm' internally, but 'armhf' in tools

Sounds reasonable to me, do the tests pass with this change in place ?
If so, they aren't testing arm/armhf at all :(

Revision history for this message
Jason Hobbs (jason-hobbs) wrote :

The tests won't pass with this change; they are checking for arm, not armhf.

tags: added: server-hwe
tags: added: armhf-hwe
Curtis Hovey (sinzui)
Changed in juju-core:
status: In Progress → Triaged
John A Meinel (jameinel)
Changed in juju-core:
milestone: none → 1.19.0
Go Bot (go-bot)
Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → 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.