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

Add current time in the log #888

Merged
merged 1 commit into from
Aug 30, 2022
Merged

Conversation

yixingjia
Copy link
Contributor

Sometime the metric timestamp is a future time. the error message is a
little confusing.

Signed-off-by: Yixing Jiayixingj@vmware.com

Sometime the metric timestamp is a future time. the error message is a
little confusing.

Signed-off-by: Yixing Jia<yixingj@vmware.com>
@codecov
Copy link

codecov bot commented Aug 30, 2022

Codecov Report

Merging #888 (33d9380) into master (1011c28) will increase coverage by 0.03%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #888      +/-   ##
============================================
+ Coverage     41.74%   41.78%   +0.03%     
+ Complexity     1580     1533      -47     
============================================
  Files           233      233              
  Lines         12891    12879      -12     
  Branches       1727     1721       -6     
============================================
  Hits           5381     5381              
+ Misses         6823     6818       -5     
+ Partials        687      680       -7     
Flag Coverage Δ
aggregator 28.11% <ø> (ø)
api 65.00% <ø> (+0.20%) ⬆️
infoblox 54.04% <ø> (ø)
labsdb 24.84% <ø> (ø)
management 5.67% <ø> (ø)
nlyte 30.86% <0.00%> (-0.02%) ⬇️
openmanage 42.05% <ø> (+0.03%) ⬆️
poweriq 42.21% <ø> (ø)
vc 41.42% <ø> (-0.04%) ⬇️
vro 5.46% <ø> (ø)

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

Impacted Files Coverage Δ
...te/nlyteworker/scheduler/job/NlyteDataService.java 28.24% <0.00%> (-0.05%) ⬇️
...com/vmware/flowgate/controller/JobsController.java 36.36% <0.00%> (-0.85%) ⬇️
...ava/com/vmware/flowgate/util/FlowgateKeystore.java 43.39% <0.00%> (-0.25%) ⬇️
...flowgate/vcworker/scheduler/job/VCDataService.java 48.09% <0.00%> (-0.19%) ⬇️
...are/flowgate/vcworker/client/InventoryService.java 0.00% <0.00%> (ø)
.../flowgate/openmanage/job/OpenManageJobService.java 52.53% <0.00%> (+0.11%) ⬆️
...main/java/com/vmware/flowgate/auth/AuthVcUser.java 8.92% <0.00%> (+0.15%) ⬆️
...java/com/vmware/flowgate/service/AssetService.java 81.10% <0.00%> (+0.39%) ⬆️
...a/com/vmware/flowgate/util/Pkcs12KeystoreUtil.java 35.71% <0.00%> (+2.38%) ⬆️
...java/com/vmware/flowgate/util/EncryptionGuard.java 78.57% <0.00%> (+2.70%) ⬆️

@yixingjia yixingjia merged commit e6020c4 into vmware:master Aug 30, 2022
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.

None yet

1 participant