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

Save pdu phase metrics #464

Merged
merged 1 commit into from
Jun 22, 2020
Merged

Save pdu phase metrics #464

merged 1 commit into from
Jun 22, 2020

Conversation

Pengpengwanga
Copy link
Contributor

Signed-off-by: Pengpeng Wang pengpengw@vmware.com

@codecov-commenter
Copy link

codecov-commenter commented Jun 18, 2020

Codecov Report

Merging #464 into master will increase coverage by 2.21%.
The diff coverage is 72.76%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #464      +/-   ##
============================================
+ Coverage     26.72%   28.94%   +2.21%     
- Complexity      757      842      +85     
============================================
  Files           181      181              
  Lines          9089     9336     +247     
  Branches       1143     1198      +55     
============================================
+ Hits           2429     2702     +273     
+ Misses         6330     6277      -53     
- Partials        330      357      +27     
Flag Coverage Δ Complexity Δ
#aggregator 12.23% <ø> (ø) 28.00 <ø> (ø)
#api 51.92% <ø> (ø) 323.00 <ø> (ø)
#infoblox 42.34% <ø> (ø) 36.00 <ø> (ø)
#labsdb 24.84% <ø> (ø) 30.00 <ø> (ø)
#management 5.67% <ø> (ø) 9.00 <ø> (ø)
#nlyte 26.50% <72.91%> (+8.82%) 165.00 <25.00> (+59.00)
#poweriq 33.28% <72.36%> (+2.93%) 217.00 <0.00> (+26.00)
#vc 6.63% <ø> (ø) 18.00 <ø> (ø)
#vro 5.48% <ø> (ø) 16.00 <ø> (ø)
Impacted Files Coverage Δ Complexity Δ
...lowgate/nlyteworker/restclient/NlyteAPIClient.java 2.68% <0.00%> (-0.06%) 1.00 <0.00> (ø)
...te/nlyteworker/scheduler/job/NlyteDataService.java 25.71% <0.00%> (-0.57%) 35.00 <0.00> (ø)
...lowgate/poweriqworker/client/PowerIQAPIClient.java 0.75% <ø> (ø) 1.00 <0.00> (ø)
...re/flowgate/poweriqworker/jobs/PowerIQService.java 38.88% <72.36%> (+1.88%) 70.00 <0.00> (+1.00)
...teworker/scheduler/job/common/HandleAssetUtil.java 49.61% <79.54%> (+31.84%) 49.00 <25.00> (+37.00)
...com/vmware/flowgate/poweriqworker/model/Inlet.java 64.00% <0.00%> (-4.00%) 9.00% <0.00%> (-1.00%)
...re/flowgate/poweriqworker/model/OutletReading.java 27.14% <0.00%> (+1.42%) 13.00% <0.00%> (+1.00%)
.../vmware/flowgate/nlyteworker/model/NlyteAsset.java 22.14% <0.00%> (+4.02%) 23.00% <0.00%> (+4.00%)
...a/com/vmware/flowgate/poweriqworker/model/Pdu.java 14.78% <0.00%> (+4.22%) 14.00% <0.00%> (+4.00%)
... and 5 more

@Pengpengwanga Pengpengwanga force-pushed the powerIq branch 3 times, most recently from a3898a6 to 1c4b860 Compare June 22, 2020 02:16
Signed-off-by: Pengpeng Wang <pengpengw@vmware.com>
Copy link
Contributor

@yixingjia yixingjia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yixingjia yixingjia merged commit 34ff34c into vmware:master Jun 22, 2020
@Pengpengwanga
Copy link
Contributor Author

Closed #455

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.

3 participants