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

kerndat: Don't fail on NETLINK/nsid support missing. #2213

Merged
merged 3 commits into from
Jul 5, 2023

Conversation

osctobe
Copy link
Contributor

@osctobe osctobe commented Jun 22, 2023

If not dumping netns nor connections, nsid support is not used. Don't fail the run as if the support is needed, the dumping process will fail later.

@osctobe osctobe force-pushed the net-kerndat branch 2 times, most recently from a818bdb to f162280 Compare June 24, 2023 01:32
@codecov-commenter
Copy link

Codecov Report

Patch coverage: 25.00% and project coverage change: -0.16 ⚠️

Comparison is base (f5cd44f) 70.68% compared to head (a4d4320) 70.53%.

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

Additional details and impacted files
@@             Coverage Diff              @@
##           criu-dev    #2213      +/-   ##
============================================
- Coverage     70.68%   70.53%   -0.16%     
============================================
  Files           133      133              
  Lines         33248    33508     +260     
============================================
+ Hits          23501    23634     +133     
- Misses         9747     9874     +127     
Impacted Files Coverage Δ
criu/util.c 64.44% <0.00%> (ø)
criu/kerndat.c 57.73% <33.33%> (ø)

... and 15 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

If the error is ignored it is not important enough - make it a warning
instead.

From: Mian Luo <mianl@google.com>
Change-Id: If2641c3d4e0a4d57fdf04e4570c49be55f526535
Signed-off-by: Michał Mirosław <emmir@google.com>
kerndat_nsid() is not used outside kerndat.c. Make it static.

Change-Id: I52e518ecb7c627cc1866e373411b2be3f71a2c9d
Signed-off-by: Michał Mirosław <emmir@google.com>
If not dumping netns nor connections, nsid support is not used. Don't
fail the run as if the support is needed, the dumping process will fail
later.

Change-Id: I39a086756f6d520c73bb6b21eaf6d9fb49a18879
Signed-off-by: Michał Mirosław <emmir@google.com>
@osctobe osctobe requested a review from avagin June 27, 2023 18:01
@avagin avagin merged commit a5939b0 into checkpoint-restore:criu-dev Jul 5, 2023
@osctobe osctobe deleted the net-kerndat branch July 18, 2023 17:20
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