Skip to content

Conversation

@fmoehler
Copy link

@fmoehler fmoehler commented Dec 9, 2025

This change fixes cloudfoundry/bosh-deployment#486 .

On noble the command changed compared to jammy.
Jammy

# /usr/sbin/nvme id-ctrl -v "/dev/nvme1n1" | sed -n -E '/0000:/s/.*"([^.]+).*".*/\1/p'
sdb

Noble

# /usr/sbin/nvme id-ctrl -v "/dev/nvme1n1" | sed -n -E '/0000:/s/.*"([^.]+).*".*/\1/p'

This change is also documented here:
https://manpages.ubuntu.com/manpages/jammy/man1/nvme-id-ctrl.1.html vs https://manpages.ubuntu.com/manpages/noble/man1/nvme-id-ctrl.1.html

Testing this on noble works succesfully:

# /usr/sbin/nvme id-ctrl -V "/dev/nvme1n1" | sed -n -E '/0000:/s/.*"([^.]+).*".*/\1/p'
sdb

@fmoehler fmoehler changed the title Update nvme-id Fix output of nvme id-ctrl to show vendor-specific fields for noble Dec 9, 2025
@fmoehler fmoehler changed the title Fix output of nvme id-ctrl to show vendor-specific fields for noble Fix output of "nvme id-ctrl" to show vendor-specific fields for noble Dec 9, 2025
@fmoehler fmoehler requested review from a team, KauzClay and ragaskar and removed request for a team December 9, 2025 15:03
Copy link
Member

@selzoc selzoc left a comment

Choose a reason for hiding this comment

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

Nice catch! LGTM

@aramprice aramprice moved this from Inbox to Pending Review | Discussion in Foundational Infrastructure Working Group Dec 11, 2025
@github-project-automation github-project-automation bot moved this from Pending Review | Discussion to Pending Merge | Prioritized in Foundational Infrastructure Working Group Dec 11, 2025
@aramprice aramprice merged commit 9b74751 into ubuntu-noble Dec 11, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Dec 11, 2025
@aramprice aramprice deleted the fix-device-name-for-noble branch December 11, 2025 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants