mantle/kola/testiso: fixup comments about signature checking
Add newlines to the end of the print statements
Adjust logic so only one of the statements get printed
- If baseInst.Insecure is already set no need to set it again
@@ -300,14 +300,14 @@ func runTestIso(cmd *cobra.Command, args []string) error {
if instInsecure {
baseInst.Insecure = true
- fmt.Printf("Ignoring verification of signature on metal image")
+ fmt.Printf("Ignoring verification of signature on metal image\n")
}
// Ignore signing verification by default when running with development build
// https://github.com/coreos/fedora-coreos-tracker/issues/908
- if strings.Contains(kola.CosaBuild.Meta.BuildID, ".dev.") {
+ if !baseInst.Insecure && strings.Contains(kola.CosaBuild.Meta.BuildID, ".dev.") {
baseInst.Insecure = true
- fmt.Printf("Detected development build; disabling signature verification")
+ fmt.Printf("Detected development build; disabling signature verification\n")
}
ranTest := false
