vsphere: environ_availzones invalid memory address or nil pointer dereference

Bug #1623136 reported by Curtis Hovey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
High
Horacio Durán

Bug Description

As seen at
    http://reports.vapour.ws/releases/issue/57d808b1749a56567fd47cfd

Bootstrap failed

INFO juju.provider.vmware environ_availzones.go:118 found 1 zones: [{10.245.0.130 []}]
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x1308884]

goroutine 1 [running]:
panic(0x2118980, 0xc8200140e0)
 /usr/lib/go-1.6/src/runtime/panic.go:464 +0x3e6
github.com/juju/govmomi/property.Wait(0x7f00ef775138, 0xc8201aa5f0, 0xc821cf8d98, 0xc82153c388, 0x4, 0xc82153c3b0, 0x9, 0xc82153c3e0, 0x1, 0x1, ...)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/govmomi/property/wait.go:74 +0x734
github.com/juju/govmomi/task.Wait(0x7f00ef775138, 0xc8201aa5f0, 0xc82153c388, 0x4, 0xc82153c3b0, 0x9, 0xc821cf8d98, 0x0, 0x0, 0x0, ...)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/govmomi/task/wait.go:120 +0x1bb
github.com/juju/govmomi/object.(*Task).WaitForResult(0xc821185f50, 0x7f00ef775138, 0xc8201aa5f0, 0x0, 0x0, 0x0, 0x0, 0x0)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/govmomi/object/task.go:51 +0x1c7
github.com/juju/juju/provider/vsphere.(*client).CreateInstance(0xc82047b580, 0xc820401b10, 0xc821cf9200, 0xc821542f00, 0x0, 0x0)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/provider/vsphere/client.go:103 +0x24e
github.com/juju/juju/provider/vsphere.(*environ).newRawInstance(0xc82047c780, 0xc820393ef0, 0x24, 0x0, 0x0, 0x0, 0x0, 0xc82041d718, 0x0, 0x0, ...)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/provider/vsphere/environ_broker.go:146 +0xc28
github.com/juju/juju/provider/vsphere.(*environ).StartInstance(0xc82047c780, 0xc820393ef0, 0x24, 0x0, 0x0, 0x0, 0x0, 0xc82041d718, 0x0, 0x0, ...)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/provider/vsphere/environ_broker.go:42 +0x189
github.com/juju/juju/provider/common.BootstrapInstance(0x7f00ef782fc0, 0xc821d093d0, 0x7f00ef783990, 0xc82047c780, 0x7ffd73bc0860, 0x7, 0xc82044e3c7, 0x3, 0xc820393f20, 0x0, ...)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/provider/common/bootstrap.go:169 +0xcf6
github.com/juju/juju/provider/common.Bootstrap(0x7f00ef782fc0, 0xc821d093d0, 0x7f00ef783990, 0xc82047c780, 0x7ffd73bc0860, 0x7, 0xc82044e3c7, 0x3, 0xc820393f20, 0x0, ...)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/provider/common/bootstrap.go:45 +0x80
github.com/juju/juju/provider/vsphere.(*environ).Bootstrap(0xc82047c780, 0x7f00ef782fc0, 0xc821d093d0, 0x7ffd73bc0860, 0x7, 0xc82044e3c7, 0x3, 0xc820393f20, 0x0, 0x0, ...)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/provider/vsphere/environ.go:116 +0xb1
github.com/juju/juju/environs/bootstrap.Bootstrap(0x7f00ef782fc0, 0xc821d093d0, 0x7f00ef783990, 0xc82047c780, 0x0, 0x0, 0x0, 0x0, 0xc82041d718, 0x0, ...)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/environs/bootstrap/bootstrap.go:354 +0x13ac
github.com/juju/juju/cmd/juju/commands.bootstrapFuncs.Bootstrap(0x7f00ef782fc0, 0xc821d093d0, 0x7f00ef783990, 0xc82047c780, 0x0, 0x0, 0x0, 0x0, 0xc82041d718, 0x0, ...)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/cmd/juju/commands/bootstrap.go:265 +0x80
github.com/juju/juju/cmd/juju/commands.(*bootstrapFuncs).Bootstrap(0x39fcb08, 0x7f00ef782fc0, 0xc821d093d0, 0x7f00ef783990, 0xc82047c780, 0x0, 0x0, 0x0, 0x0, 0xc82041d718, ...)
 <autogenerated>:86 +0xfb
github.com/juju/juju/cmd/juju/commands.(*bootstrapCommand).Run(0xc8202a18c0, 0xc82029f310, 0x0, 0x0)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/cmd/juju/commands/bootstrap.go:736 +0x4e87
github.com/juju/juju/cmd/modelcmd.(*modelCommandWrapper).Run(0xc8203d2c60, 0xc82029f310, 0x0, 0x0)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/cmd/modelcmd/modelcommand.go:282 +0x4f
github.com/juju/juju/cmd/modelcmd.(*baseCommandWrapper).Run(0xc8202d7670, 0xc82029f310, 0x0, 0x0)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/cmd/modelcmd/base.go:308 +0xd4
github.com/juju/cmd.(*SuperCommand).Run(0xc82038d440, 0xc82029f310, 0x0, 0x0)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/cmd/supercommand.go:456 +0x688
github.com/juju/cmd.Main(0x7f00ef77e1b0, 0xc82038d440, 0xc82029f310, 0xc82000a100, 0xe, 0xe, 0x2c984b0)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/cmd/cmd.go:280 +0x2f0
github.com/juju/juju/cmd/juju/commands.main.Run(0x2c99d50, 0xc82000a0f0, 0xf, 0xf, 0x2600940)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/cmd/juju/commands/main.go:157 +0x850
github.com/juju/juju/cmd/juju/commands.Main(0xc82000a0f0, 0xf, 0xf, 0xc820000180)
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/cmd/juju/commands/main.go:114 +0x48
main.main()
 /workspace/juju-core-2.0-rc1/src/github.com/juju/juju/cmd/juju/main.go:27 +0x3b
2016-09-13 09:53:36 ERROR Command '('juju', '--show-log', 'bootstrap', '--constraints', 'mem=2G', 'vsphere-deploy-trusty-amd64', 'vsphere/dc0', '--config', '/tmp/tmpFeapmb.yaml', '--default-model', 'vsphere-deploy-trusty-amd64', '--agent-version', '2.0-rc1', '--bootstrap-series', 'trusty')' returned non-zero exit status 2

Curtis Hovey (sinzui)
Changed in juju:
milestone: 2.1.0 → 2.0.0
tags: added: ci
Changed in juju:
assignee: nobody → Anastasia (anastasia-macmood)
Changed in juju:
assignee: Anastasia (anastasia-macmood) → nobody
Changed in juju:
assignee: nobody → Alexis Bruemmer (alexis-bruemmer)
Revision history for this message
Andrew Wilkins (axwalk) wrote :

It appears that there was a bug fixed in govmomi, which we forked at some point. I'm going to see if we can ditch the fork, otherwise I'll cherry-pick.

Changed in juju:
status: Triaged → In Progress
assignee: Alexis Bruemmer (alexis-bruemmer) → Andrew Wilkins (axwalk)
Curtis Hovey (sinzui)
Changed in juju:
milestone: 2.0-rc3 → 2.0.0
Revision history for this message
Andrew Wilkins (axwalk) wrote :

This is blocked by updating unit tests. The current tests are somewhat broken, and only working because the old govmomi code was being lenient about server responses that we fake.

Changed in juju:
status: In Progress → Triaged
milestone: 2.0.0 → 2.0.1
Revision history for this message
Horacio Durán (hduran-8) wrote :

The proposal of https://github.com/juju/juju/pull/6490 most likely fixes this issue

Changed in juju:
assignee: Andrew Wilkins (axwalk) → Horacio Durán (hduran-8)
Changed in juju:
status: Triaged → Fix Committed
Curtis Hovey (sinzui)
Changed in juju:
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.