Skip to content

Fix for parsing Disk.fromLsblk on older Linux#11

Merged
AngeloAvv merged 4 commits intoAngeloAvv:mainfrom
filipsky:main
Aug 26, 2025
Merged

Fix for parsing Disk.fromLsblk on older Linux#11
AngeloAvv merged 4 commits intoAngeloAvv:mainfrom
filipsky:main

Conversation

@filipsky
Copy link
Contributor

@filipsky filipsky commented Apr 8, 2024

lsblk in Ubuntu 18.04 (in WSL) returns all data as strings.

changed parsing of bool and int properties to work on this format, too

{
  "blockdevices": [
    {
      "name": "/dev/sda",
      "kname": "/dev/sda",
      "maj:min": "8:0",
      "fstype": null,
      "mountpoint": null,
      "label": null,
      "uuid": null,
      "parttype": null,
      "partlabel": null,
      "partuuid": null,
      "partflags": null,
      "ra": "128",
      "ro": "1",
      "rm": "0",
      "hotplug": "0",
      "model": "Virtual Disk    ",
      "serial": null,
      "size": "407379968",
      "state": "running",
      "owner": "root",
      "group": "root",
      "mode": "brw-------",
      "alignment": "0",
      "min-io": "512",
      "opt-io": "0",
      "phy-sec": "512",
      "log-sec": "512",
      "rota": "1",
      "sched": "none",
      "rq-size": "950",
      "type": "disk",
      "disc-aln": "0",
      "disc-gran": "2097152",
      "disc-max": "4294966784",
      "disc-zero": "0",
      "wsame": "0",
      "wwn": null,
      "rand": "1",
      "pkname": null,
      "hctl": "0:0:0:0",
      "tran": null,
      "subsystems": "block:scsi:vmbus:acpi",
      "rev": "1.0 ",
      "vendor": "Msft    ",
      "zoned": "none"
    }
  ]
}

@AngeloAvv
Copy link
Owner

Hey @filipsky , can you rebase against main please?

@filipsky
Copy link
Contributor Author

Hi, should be done

@AngeloAvv AngeloAvv merged commit c5f56c6 into AngeloAvv:main Aug 26, 2025
8 checks passed
@AngeloAvv
Copy link
Owner

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants