Comment 5 for bug 1747048

Revision history for this message
Anastasia (anastasia-macmood) wrote :

The panic is coming from comparing new and old settings values when storing setting during bootstrap, https://github.com/juju/juju/blob/2.3/state/settings.go#L176

Interestingly, the code snippet between 2.3 and develop varies in the actual comparison implementation.
The code from 2.3 that is creating this panic:

 if reflect.DeepEqual(new, old) {
   continue
  }

The code from develop, which may not able to do the comparison either:

 if new == old {
   continue
  }

Either code path may not behave as expected...