I am wondering if this code,
// First check if mountCandidate matches rootDevice
if mountCandidate != "" {
if rootChecker(p, mountCandidate, rootDevice) {
p.active = mountCandidate
log.Debugf("Setting active partition from mount candidate: %s", p.active)
return p.active, nil
}
// If mount candidate does not match root device check if we have a match in ENV
if checkBootEnvAndRootPartitionMatch(bootEnvBootPart, mountCandidate) {
p.active = mountCandidate
log.Debug("Setting active partition: ", mountCandidate)
return p.active, nil
}
// If not see if we are lucky somewhere else
}
Should be converted to:
// First check if mountCandidate matches rootDevice and ENV
if mountCandidate != "" {
if rootChecker(p, mountCandidate, rootDevice) && checkBootEnvAndRootPartitionMatch(bootEnvBootPart, mountCandidate) {
p.active = mountCandidate
log.Debugf("Setting active partition from mount candidate: %s", p.active)
return p.active, nil
}
// If not see if we are lucky somewhere else
}
@kacf, any input?