keel-hq
/
keel
mirror of https://github.com/keel-hq/keel/tree/master/chart/keel
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge pull request #286 from plumdog/more_version_comparison_logging 

More version comparison logging
feature/policy_from_annotations
Karolis Rusenas 2018-10-09 22:02:55 +01:00 committed by GitHub
parent f571b82cf4 98f4ff38d3
commit 4e1b35679a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
trigger/poll/watcher.go
View File

@ -407,9 +407,8 @@ func (j *WatchRepositoryTagsJob) Run() {
return return
} }
log.Debugf("new tag '%s' available", latestVersion)
if newAvailable { if newAvailable {
log.Debugf("new tag '%s' available", latestVersion)
// updating current latest // updating current latest
j.details.latest = latestVersion j.details.latest = latestVersion
event := types.Event{ event := types.Event{
@ -424,5 +423,7 @@ func (j *WatchRepositoryTagsJob) Run() {
"new_tag": latestVersion, "new_tag": latestVersion,
}).Info("trigger.poll.WatchRepositoryTagsJob: submiting event to providers") }).Info("trigger.poll.WatchRepositoryTagsJob: submiting event to providers")
j.providers.Submit(event) j.providers.Submit(event)
} else {
log.Debugf("no new tag available")
} }
} }

2
util/version/version.go
View File

@ -109,7 +109,9 @@ func NewAvailable(current string, tags []string) (newVersion string, newAvailabl
sort.Sort(sort.Reverse(semver.Collection(vs))) sort.Sort(sort.Reverse(semver.Collection(vs)))
if currentVersion.LessThan(vs[0]) { if currentVersion.LessThan(vs[0]) {
log.WithFields(log.Fields{"currentVersion": currentVersion, "latestAvailable": vs[0]}).Debug("latest available is newer than current")
return vs[0].Original(), true, nil return vs[0].Original(), true, nil
} }
log.WithFields(log.Fields{"currentVersion": currentVersion, "latestAvailable": vs[0]}).Debug("latest available is not newer than current")
return "", false, nil return "", false, nil
} }