Description

If you have

o.Expect(err).NotTo(o.HaveOccurred(), err)

it will Panic because it is expecting a string in the NotTo section
 
The replacement is

o.Expect(err).NotTo(o.HaveOccurred(), fmt.Sprintf("%v",err))

Additional helpful info

The Panic only happens if there is an actual error, making it intermittent.

The test fails either way.  Making the change means we get useful logs