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

fix: Correct log level for storing root events #5075

Merged
merged 2 commits into from Aug 31, 2021
Merged

Conversation

agrimmer
Copy link
Member

Signed-off-by: Andreas Grimmer andreas.grimmer@dynatrace.com

This PR

corrects the log level to info for print messages that a root event already exists in the database.

Signed-off-by: Andreas Grimmer <andreas.grimmer@dynatrace.com>
@agrimmer agrimmer requested a review from a team as a code owner August 30, 2021 12:12
bacherfl
bacherfl previously approved these changes Aug 30, 2021
@codecov
Copy link

codecov bot commented Aug 30, 2021

Codecov Report

Merging #5075 (d1294a9) into master (e789b86) will decrease coverage by 0.48%.
The diff coverage is 0.00%.

❗ Current head d1294a9 differs from pull request most recent head 4fc0c94. Consider uploading reports for the commit 4fc0c94 to get more accurate results

@@            Coverage Diff             @@
##           master    #5075      +/-   ##
==========================================
- Coverage   52.23%   51.74%   -0.49%     
==========================================
  Files         299      293       -6     
  Lines       17581    17310     -271     
  Branches      692      692              
==========================================
- Hits         9183     8957     -226     
+ Misses       7536     7499      -37     
+ Partials      862      854       -8     
Impacted Files Coverage Δ
mongodb-datastore/handlers/events.go 25.46% <0.00%> (ø)
secret-service/pkg/backend/secretbackend_k8s.go
secret-service/pkg/handler/secrethandler.go
secret-service/pkg/common/utils.go
secret-service/pkg/repository/scopes.go
secret-service/pkg/backend/registry.go
secret-service/pkg/handler/common.go

Signed-off-by: Andreas Grimmer <andreas.grimmer@dynatrace.com>
@sonarcloud
Copy link

sonarcloud bot commented Aug 30, 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
0.0% 0.0% Duplication

@agrimmer agrimmer changed the title Correct log message for storing root events fix(mongodb-datastore): Correct log message for storing root events Aug 30, 2021
@agrimmer agrimmer changed the title fix(mongodb-datastore): Correct log message for storing root events fix: Correct log message for storing root events Aug 31, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Aug 31, 2021

Docker Build Report

  • Pushed keptn/mongodb-datastore:0.8.8-dev-PR-5075.202108310623 and keptn/mongodb-datastore:0.8.8-dev-PR-5075 (Source: mongodb-datastore/)

@thisthat thisthat changed the title fix: Correct log message for storing root events fix: Correct log level for storing root events Aug 31, 2021
@thisthat thisthat merged commit 78e55bb into master Aug 31, 2021
@thisthat thisthat deleted the patch/log-level branch August 31, 2021 07:52
laneli pushed a commit that referenced this pull request Sep 9, 2021
Signed-off-by: Elisabeth Lang <elisabeth.lang@dynatrace.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

3 participants