[ubuntu] Support non-standard kernel version naming#602
[ubuntu] Support non-standard kernel version naming#602jfernandez wants to merge 1 commit intoNVIDIA:mainfrom
Conversation
a70822e to
88ef378
Compare
88ef378 to
0f9b25c
Compare
|
@shivakunv ready for another review. |
|
@jfernandez Please share the test logs for both types of kernels, if available. @rajathagasthya LGTM. PTAL |
|
@shivakunv I will verify the latest version with our infra this week. I should get you test logs before the end of the week. |
|
@jfernandez : Have you had a chance to test it? |
Query apt-cache for linux-headers package info once and attempt to parse the version using the existing major.minor.patch-revision regex. When the regex does not match (e.g. custom/distro kernels with non-standard naming), keep KERNEL_VERSION unchanged instead of failing. This preserves the existing resolution logic for standard Ubuntu kernels while allowing custom kernels to work without modification. Applied to all Ubuntu/Debian variants and vgpu-manager Ubuntu variants. Closes: NVIDIA#601 Signed-off-by: Jose Fernandez <josef@netflix.com>
0f9b25c to
bbf2bf3
Compare
|
@shivakunv Testing surfaced a gap in the initial patch: the regex Logic is working as expected after this change, here are the logs for both our custom kernel and standard Ubuntu: Custom kernel (Ubuntu 22.04, Standard Ubuntu kernel (Ubuntu 22.04 VM, |
Query apt-cache for linux-headers package info once and attempt to parse the version using the existing major.minor.patch-revision regex. When the regex does not match (e.g. custom/distro kernels with non-standard naming), keep KERNEL_VERSION unchanged instead of failing. This preserves the existing resolution logic for standard Ubuntu kernels while allowing custom kernels to work without modification.
Applied to all Ubuntu/Debian variants and vgpu-manager Ubuntu variants.
Closes: #601