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

Fix vc-worker cannot get metricid and cause error #794

Merged
merged 1 commit into from
May 11, 2021

Conversation

YangJiao0817
Copy link
Contributor

Signed-off-by: YangJiao jiaoya@vmware.com

@codecov
Copy link

codecov bot commented May 11, 2021

Codecov Report

Merging #794 (27c96b1) into master (806f711) will increase coverage by 0.01%.
The diff coverage is 50.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #794      +/-   ##
============================================
+ Coverage     40.61%   40.63%   +0.01%     
- Complexity     1543     1545       +2     
============================================
  Files           233      233              
  Lines         12856    12855       -1     
  Branches       1712     1712              
============================================
+ Hits           5222     5223       +1     
+ Misses         6968     6966       -2     
  Partials        666      666              
Flag Coverage Δ Complexity Δ
aggregator 28.11% <ø> (ø) 63.00 <ø> (ø)
api 64.23% <ø> (ø) 590.00 <ø> (ø)
infoblox 54.04% <ø> (ø) 70.00 <ø> (ø)
labsdb 24.84% <ø> (ø) 30.00 <ø> (ø)
management 5.67% <ø> (ø) 9.00 <ø> (ø)
nlyte 30.88% <ø> (ø) 200.00 <ø> (ø)
openmanage 41.76% <ø> (ø) 151.00 <ø> (ø)
poweriq 42.21% <ø> (ø) 269.00 <ø> (ø)
vc 33.33% <50.00%> (+0.09%) 147.00 <0.00> (+2.00)
vro 5.46% <ø> (ø) 16.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
...om/vmware/flowgate/controller/AssetController.java 53.24% <ø> (ø) 60.00 <0.00> (ø)
...flowgate/vcworker/scheduler/job/VCDataService.java 36.68% <50.00%> (+0.16%) 42.00 <0.00> (+2.00)

Signed-off-by: YangJiao <jiaoya@vmware.com>
@YangJiao0817 YangJiao0817 merged commit 85d6e88 into vmware:master May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vc-worker caused a null pointer because it could not get the available MetricId
2 participants