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

fix: Initialization of error monitor plugin #439

Merged
merged 5 commits into from
Jun 18, 2021
Merged

Conversation

achimnol
Copy link
Member

refs lablup/backend.ai#268

  • Let's handle missing root context gracefully but with explicit warning.
    This may indicate a configuration/installation error.

@achimnol achimnol added this to the 20.09 milestone Jun 18, 2021
@achimnol achimnol added the bug label Jun 18, 2021
@codecov
Copy link

codecov bot commented Jun 18, 2021

Codecov Report

Merging #439 (3c5f066) into main (672513b) will decrease coverage by 0.02%.
The diff coverage is 30.76%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #439      +/-   ##
==========================================
- Coverage   48.61%   48.58%   -0.03%     
==========================================
  Files          52       52              
  Lines        8625     8636      +11     
==========================================
+ Hits         4193     4196       +3     
- Misses       4432     4440       +8     
Impacted Files Coverage Δ
src/ai/backend/manager/plugin/error_monitor.py 45.76% <30.76%> (-4.24%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 672513b...3c5f066. Read the comment docs.

@achimnol achimnol merged commit 3b3b493 into main Jun 18, 2021
@achimnol achimnol deleted the fix/error-monitor-init branch June 18, 2021 06:51
achimnol added a commit that referenced this pull request Jun 18, 2021
* fix: Improve debug log messages for error collection

Backported-From: main
Backported-To: 21.03
achimnol added a commit that referenced this pull request Jun 18, 2021
* fix: Improve debug log messages for error collection

Backported-From: main
Backported-To: 20.09
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant