Skip to content

Add more log when invoking efibootmgr #951

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

HuijingHei
Copy link
Member

Fix #907

@HuijingHei
Copy link
Member Author

HuijingHei commented Jun 17, 2025

When I do testing for this, need to run env BOOTC_BOOTLOADER_DEBUG=1 bootc install to get the debug log, I am not sure if Anaconda will get the log.

Change log::debug to println now.

@HuijingHei HuijingHei force-pushed the log-efibootmgr branch 2 times, most recently from fbfe9a7 to 77f8383 Compare June 20, 2025 01:33
@HuijingHei HuijingHei changed the title Add debug log when invoking efibootmgr Add more log when invoking efibootmgr Jun 20, 2025
@HuijingHei HuijingHei force-pushed the log-efibootmgr branch 4 times, most recently from c604317 to f52445c Compare June 20, 2025 06:08
@HuijingHei
Copy link
Member Author

Tried to test using bootc install to-existing-root and get logs like following, does this make sense?

[root@localhost ~]# podman run \
--rm \
--tls-verify=false \
--privileged \
--pid=host \
-v /:/target \
-v /dev:/dev \
-v /var/lib/containers:/var/lib/containers \
-v /root/.ssh:/output \
--security-opt label=type:unconfined_t \
bootc-test \
bootc install to-existing-root --karg=console=tty0 --karg=console=ttyS0,115200n8 --target-transport containers-storage --acknowledge-destructive

...
> bootupctl backend install --write-uuid --update-firmware --auto --device /dev/vda /target
Exec: "efibootmgr" "-b" "0002" "-B"
BootCurrent: 0001
Timeout: 3 seconds
BootOrder: 0001,0000
Boot0000* UiApp	FvVol(7cb8bdc9-f8eb-4f34-aaea-3ee4af6516a1)/FvFile(462caa21-7614-4503-836e-8ab6f4662331)
Boot0001* UEFI Misc Device	PciRoot(0x0)/Pci(0x2,0x3)/Pci(0x0,0x0){auto_created_boot_option}
Exec: "efibootmgr" "--create" "--disk" "/dev/vda" "--part" "2" "--loader" "\\EFI\\fedora\\shimx64.efi" "--label" "Fedora"
BootCurrent: 0001
Timeout: 3 seconds
BootOrder: 0002,0001,0000
Boot0000* UiApp	FvVol(7cb8bdc9-f8eb-4f34-aaea-3ee4af6516a1)/FvFile(462caa21-7614-4503-836e-8ab6f4662331)
Boot0001* UEFI Misc Device	PciRoot(0x0)/Pci(0x2,0x3)/Pci(0x0,0x0){auto_created_boot_option}
Boot0002* Fedora	HD(2,GPT,fb2a48af-ef37-4f56-940e-109f8989ba2f,0x1800,0x32000)/\EFI\fedora\shimx64.efi
Added 01_users.cfg
Added 10_blscfg.cfg
Added 14_menu_show_once.cfg
Added 30_uefi-firmware.cfg
Added 41_custom.cfg
Installed: grub.cfg
Installed: bootuuid.cfg
Installed: "fedora/grub.cfg"
Installed: "fedora/bootuuid.cfg"
Installation complete!

Copy link
Member

@travier travier left a comment

Choose a reason for hiding this comment

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

One question but otherwise LGTM

@travier
Copy link
Member

travier commented Jun 20, 2025

When I do testing for this, need to run env BOOTC_BOOTLOADER_DEBUG=1 bootc install to get the debug log, I am not sure if Anaconda will get the log.

Change log::debug to println now.

Ah, maybe log::info should do it? We should be printing info logs by default.

@HuijingHei
Copy link
Member Author

When I do testing for this, need to run env BOOTC_BOOTLOADER_DEBUG=1 bootc install to get the debug log, I am not sure if Anaconda will get the log.
Change log::debug to println now.

Ah, maybe log::info should do it? We should be printing info logs by default.

I tried to use log::info, but can not get the logs. So change it back to print, let me know if I misunderstood this, thank you!

@HuijingHei HuijingHei enabled auto-merge June 20, 2025 13:44
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.

Log more information when invoking efibootmgr
2 participants