ppc64el /usr/bin/ld: error in $WORK/github.com/juju/juju/cmd/juju/_obj/exe/a.out

Bug #1423782 reported by Curtis Hovey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
Critical
Dimiter Naydenov

Bug Description

The ppc64el unittests have not passed in the last three commits to master (1.23) tested, each revision was tested 3 to 5 times. While some tries fail with common bad tests, we are seeing an ld error that implies gccgo/ppc64el is not happy with a recent change to master.

We have 10 failures so fare for three commits, compared with 13 success for the 13 previous commits.

The last success unit test of ppc64el is commit 3150a07.
The failures start with Commit 0621a1a. The suspect commits are:
Commit 18389f1 Merge pull request #1621 from johnweldon/uniter-actions-modecontinue …
Commit 0621a1a Merge pull request #1615 from cherylj/constraints …

Revision history for this message
Curtis Hovey (sinzui) wrote :
Download full text (26.5 KiB)

The error looks like this:

# testmain
/tmp/go-build789795329/github.com/juju/juju/cmd/juju/_obj/exe/a.out:(.rodata+0x6aa08): multiple definition of `__go_imt_I9_ServeHTTPFpN23_net_http.ResponseWriterpN16_net_http.Requesteee__N20_net_http.HandlerFunc'
/tmp/go-build789795329/github.com/juju/juju/libapiserver.a(apiserver.o):(.rodata.__go_imt_I9_ServeHTTPFpN23_net_http.ResponseWriterpN16_net_http.Requesteee__N20_net_http.HandlerFunc[__go_imt_I9_ServeHTTPFpN23_net_http.ResponseWriterpN16_net_http.Requesteee__N20_net_http.HandlerFunc]+0x0): first defined here
/tmp/go-build789795329/github.com/juju/juju/cmd/juju/_obj/exe/a.out:(.rodata+0x21b40): multiple definition of `__go_pimt__I5_WriteFpAN5_uint8eerN3_intN5_erroreee__N7_os.File'
/tmp/go-build789795329/github.com/juju/juju/juju/libtesting.a(testing.o):(.rodata.__go_pimt__I5_WriteFpAN5_uint8eerN3_intN5_erroreee__N7_os.File[__go_pimt__I5_WriteFpAN5_uint8eerN3_intN5_erroreee__N7_os.File]+0x0): first defined here
/tmp/go-build789795329/github.com/juju/juju/cmd/juju/_obj/exe/a.out: In function `_start':
(.text+0x361c): multiple definition of `_start'
/usr/lib/gcc/powerpc64le-linux-gnu/4.9/../../../powerpc64le-linux-gnu/crt1.o:(.text+0x0): first defined here
/tmp/go-build789795329/github.com/juju/juju/cmd/juju/_obj/exe/a.out: In function `_fini':
(.fini+0x0): multiple definition of `_fini'
/usr/lib/gcc/powerpc64le-linux-gnu/4.9/../../../powerpc64le-linux-gnu/crti.o:(.fini+0x0): first defined here
/tmp/go-build789795329/github.com/juju/juju/cmd/juju/_obj/exe/a.out:(.rodata+0x25580): multiple definition of `__go_pimt__I5_WriteFpAN5_uint8eerN3_intN5_erroreee__N12_bytes.Buffer'
/tmp/go-build789795329/gopkg.in/juju/charm.v4/libtesting.a(testing.o):(.rodata.__go_pimt__I5_WriteFpAN5_uint8eerN3_intN5_erroreee__N12_bytes.Buffer[__go_pimt__I5_WriteFpAN5_uint8eerN3_intN5_erroreee__N12_bytes.Buffer]+0x0): first defined here
/tmp/go-build789795329/github.com/juju/juju/cmd/juju/_obj/exe/a.out:(.rodata+0x1e2358): multiple definition of `__go_pimt__I13_EnvironConfigFrpN43_github_com_juju_juju_environs_config.ConfigN5_erroree28_WatchForEnvironConfigChangesFrN40_github_com_juju_juju_state.NotifyWatchereee__N32_github_com_juju_juju_state.State'
/tmp/go-build789795329/github.com/juju/juju/apiserver/libuniter.a(uniter.o):(.rodata.__go_pimt__I13_EnvironConfigFrpN43_github_com_juju_juju_environs_config.ConfigN5_erroree28_WatchForEnvironConfigChangesFrN40_github_com_juju_juju_state.NotifyWatchereee__N32_github_com_juju_juju_state.State[__go_pimt__I13_EnvironConfigFrpN43_github_com_juju_juju_environs_config.ConfigN5_erroree28_WatchForEnvironConfigChangesFrN40_github_com_juju_juju_state.NotifyWatchereee__N32_github_com_juju_juju_state.State]+0x0): first defined here
/tmp/go-build789795329/github.com/juju/juju/cmd/juju/_obj/exe/a.out:(.rodata+0x2020f8): multiple definition of `__go_pimt__I5_CloseFrN5_erroree9_LocalAddrFrN8_net.Addree4_ReadFpAN5_uint8eerN3_intN5_erroree10_RemoteAddrFrN8_net.Addree11_SetDeadlineFpN9_time.TimeerN5_erroree15_SetReadDeadlineFpN9_time.TimeerN5_erroree16_SetWriteDeadlineFpN9_time.TimeerN5_erroree5_WriteFpAN5_uint8eerN3_intN5_erroreee__N15_crypto_tls.Conn'
/tmp/go-build789795329/github.com/juju/libtesti...

description: updated
Curtis Hovey (sinzui)
description: updated
Revision history for this message
Dimiter Naydenov (dimitern) wrote :

I've investigated various bug reports for gccgo/ppc64el to see if this might be a compiler issue. No likely causes found.
Since jw4 already reverted his https://github.com/juju/juju/pull/1621 PR I decided to also revert cherylj's https://github.com/juju/juju/pull/1615 PR to see if it might fix the issue.

Revert PR: https://github.com/juju/juju/pull/1643

Revision history for this message
Dimiter Naydenov (dimitern) wrote :

As the revert of #1615 has landed, I'm marking this as Fix Committed. Hopefully, the ppc64 jobs will show in a few hours whether it worked or not.

Changed in juju-core:
assignee: nobody → Dimiter Naydenov (dimitern)
status: Triaged → Fix Committed
Revision history for this message
Dimiter Naydenov (dimitern) wrote :

Since http://juju-ci.vapour.ws:8080/job/run-unit-tests-trusty-ppc64el/2401/ passed OK I'm marking this as Fix Released.

Changed in juju-core:
status: Fix Committed → Fix Released
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.23 → 1.23-beta1
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.