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

List host_nqns added to subsystem with subsystem list command #1086

Open
rahullepakshi opened this issue Feb 4, 2025 · 2 comments
Open

List host_nqns added to subsystem with subsystem list command #1086

rahullepakshi opened this issue Feb 4, 2025 · 2 comments

Comments

@rahullepakshi
Copy link
Contributor

Proposing to list hostNQN's column along with 'Any Host' at subsystem list output to enhance usability. With namespace masking and subsystem host add , user will always benefit when host NQNs added to subsystem are displayed at subsystem list and hence compared with NQNs masked with namespace to eliminate confusion and invalid configuration when host visibility is set to both subsystem and namespace. WDYT?

[root@ceph-rlepaksh-8-1-065w2y-node3 ~]# podman run --rm quay.io/ceph/nvmeof-cli:latest --server-address 10.0.66.114 subsystem list
Subsystems:
╒═══════════╤═══════════════════════════════════╤════════════════════╤══════════════════╤═════════════╤══════════════╤════════════╤══════════╕
│ Subtype   │ NQN                               │ Serial             │ Controller IDs   │   Namespace │          Max │ Allow      │ DHCHAP   │
│           │                                   │ Number             │                  │       Count │   Namespaces │ Any Host   │ Key      │
╞═══════════╪═══════════════════════════════════╪════════════════════╪══════════════════╪═════════════╪══════════════╪════════════╪══════════╡
│ NVMe      │ nqn.2016-06.io.spdk:cnode1.group1 │ Ceph23571808504978 │ 1-2040           │           8 │         1400 │ Yes        │ No       │
├───────────┼───────────────────────────────────┼────────────────────┼──────────────────┼─────────────┼──────────────┼────────────┼──────────┤
│ NVMe      │ nqn.2016-06.io.spdk:cnode2.group1 │ Ceph67858855260085 │ 1-2040           │           8 │         1400 │ No         │ No       │
╘═══════════╧═══════════════════════════════════╧════════════════════╧══════════════════╧═════════════╧══════════════╧════════════╧══════════╛


@gbregman
Copy link
Collaborator

gbregman commented Feb 4, 2025

We have a "host list" command for that. We can't pack too much information on the subsystem list. It's already very packed.

@gbregman
Copy link
Collaborator

gbregman commented Feb 4, 2025

We can add a "show all" option to the subsystem list command. If this option is set the CLI will automatically execute all the commands needed to show the hosts and listeners for each subsystem. It's not feasible to have all the information in one table. What we can have is displaying the tables one after the other.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🆕 New
Development

No branches or pull requests

2 participants