Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

feat(icon): Remove log for missing loading resources. #2257

Merged
merged 1 commit into from
Sep 23, 2021
Merged

feat(icon): Remove log for missing loading resources. #2257

merged 1 commit into from
Sep 23, 2021

Conversation

piotrl
Copy link
Contributor

@piotrl piotrl commented Sep 23, 2021

  • Previous fix PR feat(icon): Retry loading hiccup resources. #2172 improved spam a lot, we have now instead 500 logs daily, only 1-2
  • But still we have them daily, preferably we'd like to not see it again as a hiccup ;)
  • Suggestion: Keep severity as INFO instead WARN

@piotrl
feat(icon): Remove log for missing loading resources.
69dd65e 
* Previous fix PR dynatrace-oss#2172 improved spam a lot, we have now instead 500 logs daily, only 1-2
* But still we have them daily, preferrably we'd like to not see it again as a hiccup ;)
* Suggestion: Keep severity as INFO instead WARN
@thomaspink thomaspink added pr: merge-ready This PR is ready to be merged pr: needs-cherry-pick When a pull request needs manual cherry picking labels Sep 23, 2021
@thomaspink thomaspink merged commit ef80d2f into dynatrace-oss:master Sep 23, 2021
@thomaspink
Copy link
Contributor

LGTM. Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@thomaspink thomaspink thomaspink approved these changes

Assignees
No one assigned
Labels
pr: merge-ready This PR is ready to be merged pr: needs-cherry-pick When a pull request needs manual cherry picking
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@piotrl @thomaspink