Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: smoke test-performance env var set up failed #1495

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

Desiki-high
Copy link
Member

@Desiki-high Desiki-high commented Nov 29, 2023

Details

We skip the performance test in smoke test.
image

Types of changes

What types of changes does your PullRequest introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation Update (if none of the other choices apply)

Checklist

Go over all the following points, and put an x in all the boxes that apply.

  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.

Signed-off-by: Yadong Ding <ding_yadong@foxmail.com>
@Desiki-high Desiki-high requested a review from a team as a code owner November 29, 2023 08:38
@Desiki-high Desiki-high requested review from imeoer, jiangliu and changweige and removed request for a team November 29, 2023 08:38
Copy link

codecov bot commented Nov 29, 2023

Codecov Report

Merging #1495 (58c4af1) into master (0916979) will increase coverage by 0.04%.
Report is 2 commits behind head on master.
The diff coverage is 85.57%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1495      +/-   ##
==========================================
+ Coverage   62.71%   62.75%   +0.04%     
==========================================
  Files         123      123              
  Lines       43248    43340      +92     
  Branches    43248    43340      +92     
==========================================
+ Hits        27122    27199      +77     
- Misses      14817    14827      +10     
- Partials     1309     1314       +5     
Files Coverage Δ
utils/src/compress/mod.rs 97.83% <100.00%> (ø)
utils/src/compress/zlib_random.rs 91.03% <85.29%> (-0.84%) ⬇️

... and 2 files with indirect coverage changes

@jiangliu jiangliu merged commit 8ad7ae5 into dragonflyoss:master Nov 29, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants