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

✨ fstab parser #4823

Merged
merged 8 commits into from
Nov 8, 2024
Merged

✨ fstab parser #4823

merged 8 commits into from
Nov 8, 2024

Conversation

slntopp
Copy link
Member

@slntopp slntopp commented Nov 7, 2024

cnquery run -c "fstab.entries { * }"
fstab.entries: [
  0: {
    fsck: 1
    options: "defaults,noatime"
    device: "UUID=7c4e7e0e-ce36-42f9-b456-16f78f3a1eb1"
    fstype: "xfs"
    mountpoint: "/"
    dump: 1
  }
  1: {
    fsck: 2
    options: "defaults,noatime,uid=0,gid=0,umask=0077,shortname=winnt,x-systemd.automount"
    device: "UUID=C155-24D2"
    fstype: "vfat"
    mountpoint: "/boot/efi"
    dump: 0
  }
]

Device connection example

cnquery run device -c "fstab.entries { * }" --device-name sdby
fstab.entries: [
  0: {
    fsck: 1
    options: "discard,commit=30,errors=remount-ro"
    device: "LABEL=cloudimg-rootfs"
    fstype: "ext4"
    mountpoint: "/"
    dump: 0
  }
  1: {
    fsck: 2
    options: "defaults"
    device: "LABEL=BOOT"
    fstype: "ext4"
    mountpoint: "/boot"
    dump: 0
  }
  2: {
    fsck: 1
    options: "umask=0077"
    device: "LABEL=UEFI"
    fstype: "vfat"
    mountpoint: "/boot/efi"
    dump: 0
  }
]

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

github-actions bot commented Nov 7, 2024

Test Results

3 145 tests  +7   3 144 ✅ +7   1m 27s ⏱️ -6s
  371 suites ±0       1 💤 ±0 
   28 files   ±0       0 ❌ ±0 

Results for commit b953065. ± Comparison against base commit 4daa361.

♻️ This comment has been updated with latest results.

@slntopp slntopp marked this pull request as ready for review November 7, 2024 14:27
@slntopp slntopp self-assigned this Nov 7, 2024
Copy link
Contributor

@misterpantz misterpantz left a comment

Choose a reason for hiding this comment

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

Thanks for the great help, @slntopp!

@slntopp slntopp merged commit 971a960 into main Nov 8, 2024
8 checks passed
@slntopp slntopp deleted the mik/fstab branch November 8, 2024 14:29
@github-actions github-actions bot locked and limited conversation to collaborators Nov 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants