rpc: TestChangeAPI hits an assertion error with gccgo
Bug #1251076 reported by
Michael Hudson-Doyle
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-core |
Fix Released
|
High
|
Dave Cheney |
Bug Description
Like this:
../../.
Aborted
It's the call to NewlyAvailable after the call to ChangeAPIMethods that explodes.
Related branches
lp://staging/~mwhudson/juju-core/gccgo-reflection-fun
Rejected
for merging
into
lp://staging/~go-bot/juju-core/trunk
- John A Meinel: Approve
-
Diff: 128 lines (+33/-7)4 files modifiedrpc/reflect_test.go (+3/-0)
rpc/rpc_test.go (+15/-4)
rpc/rpcreflect/type.go (+12/-2)
state/apiserver/root.go (+3/-1)
lp://staging/~dave-cheney/juju-core/110-fix-lp-1251076
- Juju Engineering: Pending requested
-
Diff: 18 lines (+7/-1)1 file modifiedrpc/rpcreflect/type.go (+7/-1)
tags: | added: gccgo test-failure |
Changed in juju-core: | |
status: | New → Triaged |
importance: | Undecided → High |
Changed in juju-core: | |
milestone: | none → 1.18.0 |
Changed in juju-core: | |
assignee: | nobody → Dave Cheney (dave-cheney) |
milestone: | 1.20.0 → 2.0 |
status: | Triaged → In Progress |
milestone: | 2.0 → 1.17.8 |
Changed in juju-core: | |
milestone: | 1.17.8 → 1.19.0 |
Changed in juju-core: | |
status: | In Progress → Fix Committed |
Changed in juju-core: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Bug reported upstream, we should probably attempt a workaround.