Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

don't build debootstrap from source #17227

Closed
wants to merge 1 commit into from

Conversation

k-v1
Copy link
Contributor

@k-v1 k-v1 commented Nov 19, 2023

Why I did it

test build

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@saiarcot895
Copy link
Contributor

FYI I've pulled in this change into #17234.

@k-v1
Copy link
Contributor Author

k-v1 commented Nov 20, 2023

FYI I've pulled in this change into #17234.

@saiarcot895
Thank you. When I tested this patch locally on my PC (Ubuntu 22.04) I got a build error in files/build_templates/sonic_debian_extension.j2:

for kernel_release in $(ls $FILESYSTEM_ROOT/lib/modules/); do
    sudo LANG=C chroot $FILESYSTEM_ROOT /etc/kernel/postinst.d/kdump-tools $kernel_release
    sudo LANG=C chroot $FILESYSTEM_ROOT kdump-config symlinks $kernel_release
done
+ cat files/image_config/kdump/kdump-tools
+ sudo tee -a ./fsroot-vs/etc/default/kdump-tools
++ ls ./fsroot-vs/lib/modules/
+ for kernel_release in $(ls $FILESYSTEM_ROOT/lib/modules/)
+ sudo LANG=C chroot ./fsroot-vs /etc/kernel/postinst.d/kdump-tools 6.1.0-11-2-amd64
kdump-tools: Generating /var/lib/kdump/initrd.img-6.1.0-11-2-amd64
mkinitramfs: failed to determine device for /
mkinitramfs: workaround is MODULES=most, check:
grep -r MODULES /var/lib/kdump/initramfs-tools

Error please report bug on initramfs-tools
Include the output of 'mount' and 'cat /proc/mounts'
update-initramfs: failed for /var/lib/kdump/initrd.img-6.1.0-11-2-amd64 with 1.
+ clean_sys

That's why I open this draft instread of PR.
But probably this bug is not related with debootstrap.

@saiarcot895
Copy link
Contributor

That looks like some local environment issue on your PC, maybe that your root filesystem is some type that's not handled by mkinitramfs? Not entirely sure.

@k-v1 k-v1 closed this Nov 22, 2023
@k-v1 k-v1 deleted the debootstrap-upstream branch November 22, 2023 08:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants