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

Driver fail to list datasets under certain circumstances #496

Open
angarod opened this issue Jan 4, 2024 · 2 comments
Open

Driver fail to list datasets under certain circumstances #496

angarod opened this issue Jan 4, 2024 · 2 comments
Labels
Need community involvement Needs community involvement on some action item. need more info More information is needed from user, need reproduction steps
Milestone

Comments

@angarod
Copy link

angarod commented Jan 4, 2024

What steps did you take and what happened:
Create a persistent volume claim and a persistent volume on a server that use zfs as filesystem.
ZFS topology:

$ zfs list
nvme                                242K  9.20G       24K  none
nvme/openebs                        120K  9.20G       24K  none
nvme/openebs/dataset                24K   9.20G       24K  none

What did you expect to happen:
PVC and PV are successfully created without error.

Anything else you would like to add:
The driver cannot list datasets since depth level is hard coded to 1, which prevents the driver from reading my topology. Seems to be related to ba0e174.
openebs-zfs-plugin E1229 09:29:32.162493 1 zfs_util.go:931] zfs: could not list zpools cmd [list -d 1 -s name -o name,guid,available -H -p]: exit status 127

Environment:

  • ZFS-LocalPV version: zfs-localpv-2.3.0
  • Kubernetes version (use kubectl version): v1.28.2
  • Kubernetes installer & version: v1.28.2
  • Cloud provider or hardware configuration: bare-metal
  • OS (e.g. from /etc/os-release): Ubuntu 20.04.6 LTS
@angarod angarod changed the title Driver fail to list zpool under certain circumstances Driver fail to list datasets under certain circumstances Jan 4, 2024
@Val
Copy link

Val commented Mar 22, 2024

@lowjoel what do you propose to meet the need for topology on several levels?

@Abhinandan-Purkait Abhinandan-Purkait added Need community involvement Needs community involvement on some action item. need more info More information is needed from user, need reproduction steps labels Jun 6, 2024
@avishnu avishnu added this to the v4.3 milestone Oct 1, 2024
@avishnu
Copy link
Member

avishnu commented Oct 1, 2024

Yes, this could be the issue. Any takers for fixing? Possible approaches could be determine the depth based on the input dataset or checking for the existence of the input dataset..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Need community involvement Needs community involvement on some action item. need more info More information is needed from user, need reproduction steps
Projects
None yet
Development

No branches or pull requests

4 participants