-
Notifications
You must be signed in to change notification settings - Fork 205
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
builder: correctly generate nid for v6 inodes #1304
Conversation
@jiangliu , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/75716 |
The `nid` is not actually used yet, but we should still generate it with correct value. Fixes: dragonflyoss#1301 Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
Codecov Report
@@ Coverage Diff @@
## master #1304 +/- ##
==========================================
- Coverage 45.93% 45.91% -0.02%
==========================================
Files 123 123
Lines 37028 37031 +3
Branches 37028 37031 +3
==========================================
- Hits 17009 17004 -5
- Misses 19125 19130 +5
- Partials 894 897 +3
|
Fix a compilation failure on macos. Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/75717 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
The
nid
is not actually used yet, but we should still generate it with correct value.Relevant Issue (if applicable)
If there are Issues related to this PullRequest, please list it.
Details
Please describe the details of PullRequest.
Types of changes
What types of changes does your PullRequest introduce? Put an
x
in all the boxes that apply:Checklist
Go over all the following points, and put an
x
in all the boxes that apply.