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

Revert "[warm-reboot] Use kexec_file_load instead of kexec_load when … #2833

Merged
merged 1 commit into from
May 10, 2023

Conversation

kellyyeh
Copy link
Contributor

…available (#2608)"

This reverts commit 93c7d43.

What I did

Revert using kexec_file_load only 202012 branch only. Kexec_file_load is causing warm-reboot failure on kvm

admin@vlab-01:~$ sudo warm-reboot -vvvv
Wed 10 May 2023 12:20:58 AM UTC Saving counters folder before warmboot...
kexec_file_load failed: Key was rejected by service
Wed 10 May 2023 12:21:00 AM UTC warm-reboot failure (255) cleanup ...
Wed 10 May 2023 12:21:01 AM UTC Cancel warm-reboot: code (0)

admin@vlab-01:~$ sudo /sbin/kexec -a -l /host/image-202012-14937.266039-f379524f2/boot/vmlinuz-4.19.0-12-2-amd64 --initrd=/host/image-202012-14937.266039-f379524f2/boot/initrd.img-4.19.0-12-2-amd64 --append="BOOT_IMAGE=/image-202012-14937.266039-f379524f2/boot/vmlinuz-4.19.0-12-2-amd64 root=UUID=55f52500-e626-4865-ab33-54d72ca71c22 rw console=tty0 console=ttyS0,115200n8 quiet intel_idle.max_cstate=0 net.ifnames=0 biosdevname=0 loop=image-202012-14937.266039-f379524f2/fs.squashfs loopfstype=squashfs systemd.unified_cgroup_hierarchy=0 apparmor=1 security=apparmor varlog_size=4096 usbcore.autosuspend=-1 SONIC_BOOT_TYPE=warm"
kexec_file_load failed: Key was rejected by service

How I did it

Revert using "-a" flag

How to verify it

Run warm-reboot on kvm without '-a' flag

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@kellyyeh kellyyeh requested a review from saiarcot895 May 10, 2023 20:08
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 10, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

@kellyyeh kellyyeh merged commit 878be48 into sonic-net:202012 May 10, 2023
@kellyyeh kellyyeh deleted the revert-kvm branch May 10, 2023 22:21
@vaibhavhd
Copy link
Contributor

@saiarcot895 does this mean the original issue that you fixed will reappear in 202012 after this revert?

If yes, is it possible to provide a separate fix for 202012, since the cherry picked is not good.

@saiarcot895
Copy link
Contributor

It's possible that the issue might reappear. I'll need to see why the kexec_file_load syscall is failing with 202012 images.

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.

3 participants