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

test: add a comment on the effect of device DAXes order #5856

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

janekmi
Copy link
Contributor

@janekmi janekmi commented Aug 7, 2023

Ref: #5615


This change is Reviewable

@janekmi janekmi added the sprint goal This pull request is part of the ongoing sprint label Aug 7, 2023
@janekmi janekmi added this to the 2.0.0 milestone Aug 7, 2023
@codecov
Copy link

codecov bot commented Aug 7, 2023

Codecov Report

Merging #5856 (0d962f8) into master (ee3bb15) will increase coverage by 1.69%.
Report is 6 commits behind head on master.
The diff coverage is n/a.

❗ Current head 0d962f8 differs from pull request most recent head ff488fd. Consider uploading reports for the commit ff488fd to get more accurate results

@@            Coverage Diff             @@
##           master    #5856      +/-   ##
==========================================
+ Coverage   69.29%   70.99%   +1.69%     
==========================================
  Files         131      131              
  Lines       19678    19175     -503     
  Branches     3259     3193      -66     
==========================================
- Hits        13636    13613      -23     
+ Misses       6042     5562     -480     

Copy link
Contributor

@osalyk osalyk left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @janekmi)

@janekmi janekmi merged commit f548135 into pmem:master Aug 7, 2023
5 checks passed
Ref: pmem#5615

Signed-off-by: Jan Michalski <jan.michalski@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sprint goal This pull request is part of the ongoing sprint
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants