Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[crmsh-4.5] Fix: Raise an exception as a rapid return of ssh-related operations to prevent hang (bsc#1228899) #1508

Merged

Fix: report: When 'core.no_ssh' set to 'yes', crm report works in loc…

e87d325
Select commit
Loading
Failed to load commit list.
Merged

[crmsh-4.5] Fix: Raise an exception as a rapid return of ssh-related operations to prevent hang (bsc#1228899) #1508

Fix: report: When 'core.no_ssh' set to 'yes', crm report works in loc…
e87d325
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 9, 2024 in 0s

66.20% (+13.78%) compared to d44b777

View this Pull Request on Codecov

66.20% (+13.78%) compared to d44b777

Details

Codecov Report

Attention: Patch coverage is 66.00000% with 17 lines in your changes missing coverage. Please review.

Project coverage is 66.20%. Comparing base (d44b777) to head (e87d325).
Report is 41 commits behind head on crmsh-4.5.

Files Patch % Lines
crmsh/ui_cluster.py 46.66% 8 Missing ⚠️
crmsh/report/collect.py 0.00% 4 Missing ⚠️
crmsh/main.py 33.33% 2 Missing ⚠️
crmsh/report/core.py 60.00% 2 Missing ⚠️
crmsh/utils.py 92.85% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff               @@
##           crmsh-4.5    #1508       +/-   ##
==============================================
+ Coverage      52.41%   66.20%   +13.78%     
==============================================
  Files             78       78               
  Lines          25123    25448      +325     
==============================================
+ Hits           13169    16848     +3679     
+ Misses         11954     8600     -3354     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.