Uploaded image for project: 'RHEL'
  1. RHEL
  2. RHEL-11541

[python-blivet] Respect HostNQN used or set in the dracut phase

Linking RHIVOS CVEs to...Migration: Automation ...SWIFT: POC ConversionSync from "Extern...XMLWordPrintable

    • Icon: Story Story
    • Resolution: Done-Errata
    • Icon: Major Major
    • rhel-9.4
    • rhel-9.4
    • python-blivet
    • None
    • python-blivet-3.6.0-11.el9
    • None
    • rhel-storage-management
    • ssg_filesystems_storage_and_HA
    • 15
    • 20
    • 2
    • QE ack, Dev ack
    • False
    • False
    • Hide

      None

      Show
      None
    • No
    • None
    • None

      In case RHEL-2252 gets included, then the proposal is this:

      As an administrator, I want the installer to respect:

      • any rd.nvmf.hostnqn and rd.nvmf.hostid overrides supplied on the installer kernel commandline
      • any hostnqn/hostid values parsed out or set and used by the 95nvmf dracut module

      For example, HostNQN and HostID values parsed from the NBFT table and used to establish a connection need to be taken over to the installer.

      See also https://github.com/redhat-plumbers/dracut-rhel9/blob/85ba5ac39bede0995720c7d3382239420c49f010/modules.d/95nvmf/parse-nvmf-boot-connections.sh#L300
      See also related discussion in RHELPLAN-110903

      In other words: take any /etc/nvme/host* files over from initramfs to the installer userspace. The code in RHEL-1254 should make use of it. Although in most cases the IDs used in the dracut phase should match the fallback values autogenerated from DMI UUID, we should probably think of this scenario.

      Additional info:

      • Turned out in most cases there aren't any /etc/nvme/host* files. The files are written down when rd.nvmf.hostnqn or rd.nvmf.hostid arguments are present. In theory, since this kind of override will get used for connection, it might be sufficient to just take over any /sys/class/nvme-fabrics/ctl/nvme0/hostnqn and /sys/class/nvme-fabrics/ctl/nvme0/hostid values, if present at Anaconda startup.

              tbzatek Tomáš Bžatek
              tbzatek Tomáš Bžatek
              Tomáš Bžatek Tomáš Bžatek
              Jan Stodola Jan Stodola
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: