Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

fix(shipyard-controller): Reduce log noise for sequence watcher component #5458

Merged
merged 1 commit into from Oct 4, 2021

Conversation

bacherfl
Copy link
Member

@bacherfl bacherfl commented Oct 4, 2021

The log output of the sequence watcher was getting too verbose when too many events with a time stamp that does not match the default time format are available. To reduce the noise in the logs, an error is now only logged if the timestamp can not be logged at all.

…nent

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@bacherfl bacherfl requested a review from a team as a code owner October 4, 2021 06:01
@sonarcloud
Copy link

sonarcloud bot commented Oct 4, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@bacherfl bacherfl merged commit eb2ef25 into master Oct 4, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Oct 4, 2021

Docker Build Report

  • Pushed keptn/shipyard-controller:0.9.3-dev-PR-5458.202110040602 and keptn/shipyard-controller:0.9.3-dev-PR-5458 (Source: shipyard-controller/)

@bacherfl bacherfl deleted the patch/sequence-watcher-logs branch October 4, 2021 06:41
RealAnna pushed a commit to RealAnna/keptn that referenced this pull request Oct 4, 2021
…nent (keptn#5458)

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
bhamail pushed a commit to bhamail/keptn that referenced this pull request Oct 6, 2021
…nent (keptn#5458)

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Dan Rollo <danrollo@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants