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
Add grubby to handle latest grub2 changes #797
base: master
Are you sure you want to change the base?
Conversation
a142cf4
to
7d8bca8
Compare
testcases/InstallUpstreamKernel.py
Outdated
@@ -128,8 +128,7 @@ def is_url(path): | |||
time.sleep(10) | |||
if not self.use_kexec: | |||
# FIXME: Handle distributions which do not support grub | |||
con.run_command( | |||
"grub2-mkconfig --output=/boot/grub2/grub.cfg") | |||
con.run_command("grubby --set-default /boot/vmlinuz-%s" % res[-1]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will need code to handle vmlinux vs vmlinuz. Additionally we should also handle different distro flavours.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abdhaleegit can you please address sachin's review comment and we able to conclude these changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes I have the code ready
ec66819
to
2cc7254
Compare
da9d12c
to
9da58c2
Compare
c17b2d0
to
e195d47
Compare
@@ -1475,12 +1475,12 @@ def get_login(self, host, term_obj, pty, prompt): | |||
my_user = host.username() | |||
my_pwd = host.password() | |||
pty.sendline() | |||
rc = pty.expect(['login: ', pexpect.TIMEOUT, pexpect.EOF], timeout=10) | |||
rc = pty.expect(['login: ', pexpect.TIMEOUT, pexpect.EOF], timeout=60) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we have this as class/function variable? so changing one place is easy in future as well
e738300
to
29694da
Compare
With latest grub2 installer, one should set the kernel to be booted explicitly after update or install of latest kernel Signed-off-by: Abdul Haleem <abdhalee@linux.vnet.ibm.com>
When LPAR is loaded with devices and adpater and network load is high, the console, boot and prompt response is slow so increase wait time to handle these Signed-off-by: Abdul Haleem <abdhalee@linux.vnet.ibm.com>
e2505f2
to
f9e4dc0
Compare
With latest grub2 installer, one should set the
kernel to be booted explicitly after update or
install of latest kernel