juju-core

panic: reflect: Call of unexported method ppc64el/gccgo unit tests

Bug #1331744 reported by Curtis Hovey on 2014-06-18

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	juju-core	Fix Released	Critical	John A Meinel	juju-core 1.19.4

Bug Description

Merge in the last few hours broke unit tests on trusty ppc64el (gccgo)

The last successful run was a few hours ago
Revision build: 1487 gitbranch:master:github.com/juju/juju 9a97f86d

The issue is probably compiler related and was introduced in:
    commit 8e75f2b "Merge pull request #114 from voidspace/download-backup"
    commit 587b60a "Merge pull request #123 from jameinel/api-versioning"
    commit 67931d7 "Merge pull request #100 from jameinel/api-versioning-4-login-describes"

Tags:

Revision history for this message

Curtis Hovey (sinzui) wrote on 2014-06-18:

Test output Edit (795.1 KiB, text/html)

John A Meinel (jameinel) on 2014-06-18

Changed in juju-core:
assignee:	nobody → John A Meinel (jameinel)

Revision history for this message

Curtis Hovey (sinzui) wrote on 2014-06-18:

local deploy failure Edit (16.9 KiB, text/plain)

This is the ppc64el local deploy test
http://juju-ci.vapour.ws:8080/job/local-deploy-trusty-ppc64/99/console

Revision history for this message

Dave Cheney (dave-cheney) wrote on 2014-06-19:

The bug from #2 is

        status, err := apiclient.Status(c.patterns)
        // Display any error, but continue to print status if some was returned
        if err != nil {
                fmt.Fprintf(ctx.Stderr, "%v\n", err)
        }
        result := newStatusFormatter(status).format()

There is no contract to specify what the contents of status will be if there is an error

Revision history for this message

Dave Cheney (dave-cheney) wrote on 2014-06-19:

Fix for #2 https://github.com/juju/juju/pull/127