TestNewContainerManage executed but not supported by gccgo-go

Bug #1545046 reported by Curtis Hovey
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
Critical
Tycho Andersen

Bug Description

As seen in
    http://reports.vapour.ws/releases/issue/56ba0ab9749a567fb286bb9b

factory_test.go:53:
    c.Assert(err, jc.ErrorIsNil)
... value *errors.Err = &errors.Err{message:"LXD containers not supported in go 1.2", cause:error(nil), previous:error(nil), file:"github.com/juju/juju/container/lxd/lxd_go12.go", line:24} ("LXD containers not supported in go 1.2")
... error stack:
 github.com/juju/juju/container/lxd/lxd_go12.go:24: LXD containers not supported in go 1.2

Curtis Hovey (sinzui)
Changed in juju-core:
status: New → Incomplete
Curtis Hovey (sinzui)
Changed in juju-core:
status: Incomplete → Triaged
importance: Undecided → Critical
Revision history for this message
Tycho Andersen (tycho-s) wrote :

Looks like here the tests might be run on a different go version than they're built on?

Revision history for this message
Tycho Andersen (tycho-s) wrote :

I can send a patch, but can anyone confirm that's the case or is something else going on?

Revision history for this message
Cheryl Jennings (cherylj) wrote :
Revision history for this message
John A Meinel (jameinel) wrote :

Tycho, I think the issue is that you were looking for the "runtime.Version" to start with "go1.2" I'm guessing that GCC-go uses a different version string. Something like gccgo-1.2. I don't really know what it would be. But it isn't that we compile on one and use another, just that we use a different *compiler* and that didn't match your original check.

Revision history for this message
Curtis Hovey (sinzui) wrote :

The gccgo version string always has "xgcc" in it
    $ go version
    go version xgcc (Ubuntu 4.9.3-0ubuntu4) 4.9.3 linux/ppc64le

Changed in juju-core:
status: Triaged → Fix Committed
assignee: nobody → Tycho Andersen (tycho-s)
milestone: none → 2.0-beta2
no longer affects: juju-core/lxd-container-type
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
tags: added: 2.0-count
affects: juju-core → juju
Changed in juju:
milestone: 2.0-beta2 → none
milestone: none → 2.0-beta2
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.