Skip to content

Commit

Permalink
Merge pull request #110 from moira-alert/hotfix/cli-fixes
Browse files Browse the repository at this point in the history
Yep, I made a mistak in CLI converter
  • Loading branch information
titusjaka authored Aug 16, 2018
2 parents 311180f + 652aa4b commit a78643a
Showing 1 changed file with 5 additions and 9 deletions.
14 changes: 5 additions & 9 deletions cmd/cli/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -361,18 +361,14 @@ func updateTriggers(triggers []*moira.Trigger, dataBase moira.Database, logger m
trigger.TriggerType == moira.FallingTrigger ||
trigger.TriggerType == moira.ExpressionTrigger {
logger.Debugf("Trigger %v has '%v' type - no need to convert", trigger.ID, trigger.TriggerType)
continue
}

if err := setProperTriggerType(trigger, logger); err == nil {
logger.Debugf("Trigger %v - save to Database", trigger.ID)
if err := dataBase.SaveTrigger(trigger.ID, trigger); err != nil {
return err
}
} else {
} else if err := setProperTriggerType(trigger, logger); err != nil {
return fmt.Errorf("trigger converter: trigger %v - could not save to Database, error: %v",
trigger.ID, err)
}
logger.Debugf("Trigger %v - save to Database", trigger.ID)
if err := dataBase.SaveTrigger(trigger.ID, trigger); err != nil {
return err
}
}
return nil
}
Expand Down

0 comments on commit a78643a

Please sign in to comment.