Bug 1328787

Summary: NGN can't be bootable in UEFI machine due to failed to write boot loader configuration
Product: [oVirt] ovirt-node Reporter: cshao <cshao>
Component: Installation & UpdateAssignee: Ryan Barry <rbarry>
Status: CLOSED CURRENTRELEASE QA Contact: cshao <cshao>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 4.0CC: bugs, cshao, dfediuck, fdeutsch, huzhao, leiwang, rbarry, vpodzime, weiwang, yaniwang, ycui
Target Milestone: ovirt-4.0.1Flags: rule-engine: ovirt-4.0.z+
ycui: testing_plan_complete?
rule-engine: planning_ack+
rule-engine: devel_ack+
cshao: testing_ack+
Target Release: 4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-08-04 13:32:53 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Node RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1140646    
Attachments:
Description Flags
uefi1-error
none
uefi-2-reboot
none
uefi-3-can't boot
none
blurred screen
none
traceback-log
none
all-log
none
uefi-new-all-logs
none
re-upload-all-log
none
UEFI-BOOT
none
UEFI-BOOT-FAILED
none
june-16-uefi1
none
june-16-uefi2
none
june-16-uefi3
none
uefi.log
none
uefi.png
none
/tmp/all_log
none
uefi_all_log none

Description cshao 2016-04-20 09:45:46 UTC
Created attachment 1148969 [details]
uefi1-error

Description of problem:
NGN can't be bootable in UEFI machine due to failed to write boot loader configuration


Version-Release number of selected component (if applicable):
ovirt-node-ng-installer-ovirt-3.6-2016041820.iso
ovirt-node-ng-image-update-placeholder-3.6.5-0.0.master.20160414085328.gitcedfbf3.el7.noarch
imgbased-0.6-0.201604150305git1e3b28f.el7.centos.noarch
ovirt-release-host-node-3.6.5-0.0.master.20160414085328.gitcedfbf3.el7.noarch

How reproducible:
100%

Steps to Reproduce:
1. Anaconda install NGN 4.0 in UEFI machine.
2. Focus on the installation process, an error occurred.
3. Ignore this error and continue with installation.
4. Reboot NGN 

Actual results:
1. Step2: An error occurred while installing the boot loader.
2. Step3: Installation complete, NGN is now successfully installed and ready for use
4. Step4: NGN can't be bootable in UEFI machine due to failed to write boot loader configuration

Expected results:
NGN can be bootable in UEFI machine.

Additional info:
Can't obtain log info, if need I will provide the test ENV for debug.

Comment 1 cshao 2016-04-20 09:46:29 UTC
Created attachment 1148970 [details]
uefi-2-reboot

Comment 2 cshao 2016-04-20 09:46:58 UTC
Created attachment 1148972 [details]
uefi-3-can't boot

Comment 3 Fabian Deutsch 2016-04-20 09:52:13 UTC
Canyou please also provide the log files from the installation?

Comment 4 cshao 2016-04-20 09:56:05 UTC
(In reply to Fabian Deutsch from comment #3)
> Canyou please also provide the log files from the installation?

I can't obtain log info, but I have sent test env to you by mail.
Is there any shortcut key can enter to shell mode for obtain log?

Comment 5 Fabian Deutsch 2016-04-20 11:54:07 UTC
There are some hints here of how to dbeug anaconda problems

https://fedoraproject.org/wiki/How_to_debug_installation_problems
https://fedoraproject.org/wiki/Anaconda/Logging

In general you can press ctrl-alt-f2 to drop to a shell and retrieve log files.

Comment 6 cshao 2016-04-20 12:16:56 UTC
(In reply to Fabian Deutsch from comment #5)
> There are some hints here of how to dbeug anaconda problems
> 
> https://fedoraproject.org/wiki/How_to_debug_installation_problems
> https://fedoraproject.org/wiki/Anaconda/Logging
> 
> In general you can press ctrl-alt-f2 to drop to a shell and retrieve log
> files.

Seem can't drop to shell by press ctrl-alt-f2 key, blurred screen appear after press those key, I can't input anything on this screen. please see "blurred_screen.png"

I will have a try on other machine, will attach log if I can retrieve log files.

Comment 7 cshao 2016-04-20 12:17:23 UTC
Created attachment 1149070 [details]
blurred screen

Comment 8 Fabian Deutsch 2016-04-20 12:18:27 UTC
Please retry with ctrl-alt-f3 or an other f key.

Comment 9 cshao 2016-04-20 12:24:00 UTC
No such issue on local machine, please ctrl-alt-f2 key can drop to shell directly.

I will send ticket to lab machine admin to ask him help press those host key directly.

Comment 10 cshao 2016-04-21 08:24:17 UTC
Created attachment 1149353 [details]
traceback-log

Capture traceback log info, hope it is useful. Continue working on obtain full log.

Comment 11 cshao 2016-04-21 10:10:38 UTC
Created attachment 1149417 [details]
all-log

Comment 12 Fabian Deutsch 2016-04-26 08:14:11 UTC
From anaconda-tb:

anaconda 21.48.22.56-1 exception report
Traceback (most recent call first):
  File "/tmp/updates/pyanaconda/bootloader.py", line 1573, in write_config
    raise BootLoaderError("failed to write boot loader configuration")
  File "/tmp/updates/pyanaconda/bootloader.py", line 1774, in write
    self.write_config()
  File "/tmp/updates/pyanaconda/bootloader.py", line 2369, in writeBootLoaderFinal
    storage.bootloader.write()
  File "/tmp/updates/pyanaconda/bootloader.py", line 2441, in writeBootLoader
    writeBootLoaderFinal(storage, payload, instClass, ksdata)
  File "/tmp/updates/pyanaconda/install.py", line 254, in doInstall
    writeBootLoader(storage, payload, instClass, ksdata)
  File "/usr/lib64/python2.7/threading.py", line 764, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/tmp/updates/pyanaconda/threads.py", line 227, in run
    threading.Thread.run(self, *args, **kwargs)
BootLoaderError: failed to write boot loader configuration

And further down:

18:05:44,759 INFO program: Running... grub2-mkconfig -o /boot/efi/EFI/fedora/grub.cfg
18:05:47,029 INFO program: /sbin/grub2-mkconfig: line 241: /boot/efi/EFI/fedora/grub.cfg.new: No such file or directory
18:05:47,030 DEBUG program: Return code: 1

Vratislav, have you seen this kind of error?

Comment 13 Fabian Deutsch 2016-04-26 08:18:01 UTC
Actually, I see more:

18:05:42,316 INFO program: rsync: rsync_xal_set: lsetxattr(""/mnt/sysimage/boot/efi"","security.selinux") failed: Operation not supported (95)
18:05:42,318 INFO program: rsync: rsync_xal_set: lsetxattr(""/mnt/sysimage/boot/efi/EFI"","security.selinux") failed: Operation not supported (95)
18:05:42,318 INFO program: rsync: rsync_xal_set: lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos"","security.selinux") failed: Operation not supported (95)
18:05:42,318 INFO program: rsync: rsync_xal_set: lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/.gcdx64.efi.YLn5wU"","security.selinux") failed: Operation not supported (95)
18:05:42,319 INFO program: rsync: rsync_xal_set: lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/.grubenv.x5L8y9"","security.selinux") failed: Operation not supported (95)
18:05:42,320 INFO program: rsync: rsync_xal_set: lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/.grubx64.efi.KZ5dBo"","security.selinux") failed: Operation not supported (95)
18:05:42,321 INFO program: rsync: rsync_xal_set: lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/fonts"","security.selinux") failed: Operation not supported (95)
18:05:42,321 INFO program: rsync: rsync_xal_set: lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/fonts/.unicode.pf2.HlymoF"","security.selinux") failed: Operation not supported (95)
18:05:42,321 INFO program: rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1052) [sender=3.0.9]
18:05:42,321 DEBUG program: Return code: 23
18:05:43,951 INFO program: Running... new-kernel-pkg --rpmposttrans 3.10.0-327.13.1.el7.x86_64
18:05:44,066 DEBUG program: Return code: 0
18:05:44,601 INFO program: Running... efibootmgr
18:05:44,621 ERR program: Error running efibootmgr: No such file or directory
18:05:44,624 INFO program: Running... grub2-set-default 0
18:05:44,758 DEBUG program: Return code: 0
18:05:44,759 INFO program: Running... grub2-mkconfig -o /boot/efi/EFI/fedora/grub.cfg
18:05:47,029 INFO program: /sbin/grub2-mkconfig: line 241: /boot/efi/EFI/fedora/grub.cfg.new: No such file or directory
18:05:47,030 DEBUG program: Return code: 1

Initially
/mnt/sysimage/boot/efi/EFI/centos
is used
then
/boot/efi/EFI/fedora

I'm not concerned about the different prefix (/mnt/sysimage), but about the different OS part (centos cs fedora).

Chen, are you sure that the vmlinux, initrd and stage2 are from CentOS?

Comment 14 cshao 2016-04-27 06:25:11 UTC
(In reply to Fabian Deutsch from comment #13)
> Actually, I see more:
> 
> 18:05:42,316 INFO program: rsync: rsync_xal_set:
> lsetxattr(""/mnt/sysimage/boot/efi"","security.selinux") failed: Operation
> not supported (95)
> 18:05:42,318 INFO program: rsync: rsync_xal_set:
> lsetxattr(""/mnt/sysimage/boot/efi/EFI"","security.selinux") failed:
> Operation not supported (95)
> 18:05:42,318 INFO program: rsync: rsync_xal_set:
> lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos"","security.selinux") failed:
> Operation not supported (95)
> 18:05:42,318 INFO program: rsync: rsync_xal_set:
> lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/.gcdx64.efi.YLn5wU"","security.
> selinux") failed: Operation not supported (95)
> 18:05:42,319 INFO program: rsync: rsync_xal_set:
> lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/.grubenv.x5L8y9"","security.
> selinux") failed: Operation not supported (95)
> 18:05:42,320 INFO program: rsync: rsync_xal_set:
> lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/.grubx64.efi.KZ5dBo"",
> "security.selinux") failed: Operation not supported (95)
> 18:05:42,321 INFO program: rsync: rsync_xal_set:
> lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/fonts"","security.selinux")
> failed: Operation not supported (95)
> 18:05:42,321 INFO program: rsync: rsync_xal_set:
> lsetxattr(""/mnt/sysimage/boot/efi/EFI/centos/fonts/.unicode.pf2.HlymoF"",
> "security.selinux") failed: Operation not supported (95)
> 18:05:42,321 INFO program: rsync error: some files/attrs were not
> transferred (see previous errors) (code 23) at main.c(1052) [sender=3.0.9]
> 18:05:42,321 DEBUG program: Return code: 23
> 18:05:43,951 INFO program: Running... new-kernel-pkg --rpmposttrans
> 3.10.0-327.13.1.el7.x86_64
> 18:05:44,066 DEBUG program: Return code: 0
> 18:05:44,601 INFO program: Running... efibootmgr
> 18:05:44,621 ERR program: Error running efibootmgr: No such file or directory
> 18:05:44,624 INFO program: Running... grub2-set-default 0
> 18:05:44,758 DEBUG program: Return code: 0
> 18:05:44,759 INFO program: Running... grub2-mkconfig -o
> /boot/efi/EFI/fedora/grub.cfg
> 18:05:47,029 INFO program: /sbin/grub2-mkconfig: line 241:
> /boot/efi/EFI/fedora/grub.cfg.new: No such file or directory
> 18:05:47,030 DEBUG program: Return code: 1
> 
> Initially
> /mnt/sysimage/boot/efi/EFI/centos
> is used
> then
> /boot/efi/EFI/fedora
> 
> I'm not concerned about the different prefix (/mnt/sysimage), but about the
> different OS part (centos cs fedora).
> 
> Chen, are you sure that the vmlinux, initrd and stage2 are from CentOS?

Yes. I tested this issue on UEFI machine with virtual-media attached, they are from CnetOS.

Comment 15 Fabian Deutsch 2016-04-27 07:18:57 UTC
Thanks.

Vratislav, do you have any idea on this one?

Comment 16 Sandro Bonazzola 2016-05-02 10:09:05 UTC
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 18 Fabian Deutsch 2016-05-03 10:26:28 UTC
Please retest this with teh upstream build.

It is important to:
1. Use kernel + initrd from centos7 repos
2. Use stage2 squashfs from centos7 repos

Comment 19 Vratislav Podzimek 2016-05-03 10:58:06 UTC
this looks like some weird mixture of fedora and centos stuff

Comment 20 cshao 2016-05-04 09:08:33 UTC
(In reply to Fabian Deutsch from comment #18)
> Please retest this with teh upstream build.
> 
> It is important to:
> 1. Use kernel + initrd from centos7 repos
> 2. Use stage2 squashfs from centos7 repos

We can not access uefi remote console now, I will reply the needinfo asap.

Comment 21 cshao 2016-05-05 08:34:19 UTC
(In reply to Fabian Deutsch from comment #18)
> Please retest this with teh upstream build.
> 
> It is important to:
> 1. Use kernel + initrd from centos7 repos
> 2. Use stage2 squashfs from centos7 repos


Anaconda exception occurs this time.

============================================================================
16:26:14,448 INFO anaconda: Installing boot loader
16:26:14,449 INFO anaconda: boot loader stage1 target device is sda1
16:26:14,449 INFO anaconda: boot loader stage2 target device is sda2
16:26:14,450 DEBUG anaconda: new default image: <pyanaconda.bootloader.LinuxBootLoaderImage object at 0x504bcd0>
16:26:14,628 INFO anaconda: bootloader.py: used boot args: crashkernel=auto rd.lvm.lv=centos00/root00 rd.lvm.lv=centos00/swap rhgb quiet 
16:26:18,655 DEBUG anaconda: running handleException
16:26:18,657 CRIT anaconda: Traceback (most recent call last):

  File "/usr/lib64/python2.7/site-packages/pyanaconda/threads.py", line 227, in run
    threading.Thread.run(self, *args, **kwargs)

  File "/usr/lib64/python2.7/threading.py", line 764, in run
    self.__target(*self.__args, **self.__kwargs)

  File "/usr/lib64/python2.7/site-packages/pyanaconda/install.py", line 254, in doInstall
    writeBootLoader(storage, payload, instClass, ksdata)

  File "/usr/lib64/python2.7/site-packages/pyanaconda/bootloader.py", line 2440, in writeBootLoader
    writeBootLoaderFinal(storage, payload, instClass, ksdata)

  File "/usr/lib64/python2.7/site-packages/pyanaconda/bootloader.py", line 2368, in writeBootLoaderFinal
    storage.bootloader.write()

  File "/usr/lib64/python2.7/site-packages/pyanaconda/bootloader.py", line 1771, in write
    self.install()

  File "/usr/lib64/python2.7/site-packages/pyanaconda/bootloader.py", line 1750, in install
    self.remove_efi_boot_target()

  File "/usr/lib64/python2.7/site-packages/pyanaconda/bootloader.py", line 1706, in remove_efi_boot_target
    buf = self.efibootmgr(capture=True)

  File "/usr/lib64/python2.7/site-packages/pyanaconda/bootloader.py", line 1699, in efibootmgr
    return exec_func("efibootmgr", list(args), **kwargs)

  File "/usr/lib64/python2.7/site-packages/pyanaconda/iutil.py", line 345, in execWithCapture
    filter_stderr=filter_stderr)[1]

  File "/usr/lib64/python2.7/site-packages/pyanaconda/iutil.py", line 259, in _run_program
    env_prune=env_prune)

  File "/usr/lib64/python2.7/site-packages/pyanaconda/iutil.py", line 185, in startProgram
    preexec_fn=preexec, cwd=root, env=env, **kwargs)

  File "/usr/lib64/python2.7/subprocess.py", line 711, in __init__
    errread, errwrite)

  File "/usr/lib64/python2.7/subprocess.py", line 1327, in _execute_child
    raise child_exception

OSError: [Errno 2] No such file or directory

16:26:18,658 DEBUG anaconda: Gtk running, queuing exception handler to the main loop
16:26:18,658 INFO anaconda: Thread Done: AnaInstallThread (139766683113216)
16:26:48,266 INFO anaconda: Running kickstart %%traceback script(s)
16:26:48,266 INFO anaconda: All kickstart %%traceback script(s) have been run
============================================================================




Test version:
ovirt-node-ng-installer-master-2016050300.iso
imgbased-0.6-0.201604241653git1e3b28f.el7.centos.noarch
ovirt-release-host-node-4.0.0-0.3.master.20160428135304.git037679a.el7.noarch
ovirt-node-ng-image-update-placeholder-4.0.0-0.3.master.20160428135304.git037679a.el7.noarch

1. Anaconda install NGN 4.0 in UEFI machine.
2. Focus on the installation process, an error occurred.

Comment 22 Ryan Barry 2016-05-06 03:15:47 UTC
efibootmgr was not included, and is necessary.

There's a patch which resolves this (as soon as it's merged, the next build will work).

However, it's necessary to match it with the appropriate stage2 and/or ISO. I'm not sure if this is or should be an Anaconda bug, but using a RHEL boot ISO as the source with a centos squashfs will lead to a failure during bootloader installation (/boot/efi/EFI/centos instead of /boot/efi/EFI/redhat). The same with Fedora, which is the likely cause of the earlier traceback.

I'm guessing that the Anaconda stage2 assumes the pathnames will match, but efibootmgr/grub2-efi are run outside the context of the install root, and the wrong paths are inferred. Anaconda distro == install distro is a fair assumption, but it's a caveat we need to be aware of for ngn.

Comment 23 Red Hat Bugzilla Rules Engine 2016-05-06 03:15:53 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 24 cshao 2016-05-12 06:17:39 UTC
Still can reproduce this issue on ovirt-node-ng-installer-master-2016051100 build.

Comment 25 Ryan Barry 2016-05-13 18:14:01 UTC
I'll check this again, since I verified the fix (in derive-boot-iso and the addition of efibootmgr). Can you please attach new logs?

Comment 26 cshao 2016-05-16 07:38:23 UTC
Created attachment 1157802 [details]
uefi-new-all-logs

Comment 27 Ryan Barry 2016-05-16 12:50:40 UTC
I cannot reproduce this locally. This appears to be a problem specific to Jenkins builds. I'm looking into why.

Workaround:

Download a CentOS 7 boot ISO
Get ovirt-node-ng-image.squashfs.img from jenkins
Clone ovirt-node-ng

/path/to/ovirt-node-ng/scripts/derive-boot-iso.sh /path/to/centos.iso /path/to/squashfs.img ./ovirt-node-ng.iso

An ISO generated this way will work.

Comment 28 Ryan Barry 2016-05-16 13:18:42 UTC
Note:

The uploaded logs don't work. In testing locally (from jenkins master), I got the same error about /EFI/fedora... as earlier, which leads me to believe that some Fedora parts are being injected somewhere in Jenkins, though I can't find it in the logs or git (I need to test using the ovirt-node-ng-tools RPM)

My development workstation is Fedora, so being in a Fedora root upstream shouldn't affect this, as it works locally...

Comment 29 Ryan Barry 2016-05-16 13:18:51 UTC
Note:

The uploaded logs don't work. In testing locally (from jenkins master), I got the same error about /EFI/fedora... as earlier, which leads me to believe that some Fedora parts are being injected somewhere in Jenkins, though I can't find it in the logs or git (I need to test using the ovirt-node-ng-tools RPM)

My development workstation is Fedora, so being in a Fedora root upstream shouldn't affect this, as it works locally...

Comment 30 cshao 2016-05-16 13:43:27 UTC
Created attachment 1157943 [details]
re-upload-all-log

Comment 31 Ryan Barry 2016-05-16 13:55:20 UTC
(In reply to shaochen from comment #30)
> Created attachment 1157943 [details]
> re-upload-all-log

This shows the same error:

15:28:14,742 INFO program: Running... grub2-mkconfig -o /boot/efi/EFI/fedora/grub.cfg
15:28:15,439 INFO program: /sbin/grub2-mkconfig: line 241: /boot/efi/EFI/fedora/grub.cfg.new: No such file or directory

Please try with the workaround from comment#27

Comment 32 cshao 2016-05-17 07:38:33 UTC
(In reply to Ryan Barry from comment #31)
> (In reply to shaochen from comment #30)
> > Created attachment 1157943 [details]
> > re-upload-all-log
> 
> This shows the same error:
> 
> 15:28:14,742 INFO program: Running... grub2-mkconfig -o
> /boot/efi/EFI/fedora/grub.cfg
> 15:28:15,439 INFO program: /sbin/grub2-mkconfig: line 241:
> /boot/efi/EFI/fedora/grub.cfg.new: No such file or directory
> 
> Please try with the workaround from comment#27

I did the following testing with below Workaround:

Download a CentOS 7 boot ISO
Get ovirt-node-ng-image.squashfs.img (20160511_0_el7_noarch_rpm-master)from jenkins
Clone ovirt-node-ng
/path/to/ovirt-node-ng/scripts/derive-boot-iso.sh /path/to/centos.iso /path/to/squashfs.img ./ovirt-node-ng.iso

Test steps:
1. Anaconda install NGN 4.0 in UEFI machine.
2. Try to finish the installation.
3. Reboot NGN 
4. Try to boot NGN.


Test result:
1. After step 2,3, NGN install can successful on UEFI machine.
2. NGN boot failed on UEFI machine.

Additional  info:
I did the same testing using vintage RHEV-H can boot successful on the same uefi machine.

Comment 33 cshao 2016-05-17 07:39:23 UTC
Created attachment 1158199 [details]
UEFI-BOOT

Comment 34 cshao 2016-05-17 07:39:50 UTC
Created attachment 1158200 [details]
UEFI-BOOT-FAILED

Comment 35 Fabian Deutsch 2016-05-17 10:02:16 UTC
Please attach log files for the failed attempt in comment 32.

Comment 36 Ryan Barry 2016-05-17 13:44:58 UTC
shim.efi appears to be missing.

I'm doing a build to test this, and I'll verify that it boots after installation...

Comment 37 Ryan Barry 2016-05-17 14:33:28 UTC
I've verified that including both efibootmgr and shim produces an image which is installable and bootable.

I'll update redhat-release-rhev-hypervisor as well...

Comment 38 cshao 2016-05-18 06:11:57 UTC
(In reply to Fabian Deutsch from comment #35)
> Please attach log files for the failed attempt in comment 32.

After confirmed with fabian by IRC,   this bug seem has been fixed and the final gold signed NGN squashfs and NGN iso is planned for tonight. so no need the log to reproduce the bug anymore. cancel the needinfo.

Thanks.

Comment 39 cshao 2016-06-16 07:59:58 UTC
Test version:
ovirt-node-ng-installer-master-2016061511.iso

Test steps:
1. Anaconda install NGN 4.0 in UEFI machine.
2. Try to finish the installation(but met error like screen-shot"uefi" show).
3. Try to ignore the error and continue install.
4. Still got failed.


Test result:
Still got failed.
(There was an error running the ks at line 6. This is fatal error and installation will be aborted.)

I can't provide more log info due to press ctrl+alt+F2~5 can't enter into shell mode, but I will try to obtain the log via another way asap.


Change bug status to ASSIGNED.

Comment 40 Red Hat Bugzilla Rules Engine 2016-06-16 08:00:04 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 41 cshao 2016-06-16 08:00:48 UTC
Created attachment 1168587 [details]
june-16-uefi1

Comment 42 cshao 2016-06-16 08:01:39 UTC
Created attachment 1168588 [details]
june-16-uefi2

Comment 43 cshao 2016-06-16 08:02:16 UTC
Created attachment 1168589 [details]
june-16-uefi3

Comment 44 Fabian Deutsch 2016-06-16 09:41:03 UTC
The lgos from the installation are needed, a screenshot is not sufficient to debug the issue.

Comment 45 Ryan Barry 2016-06-16 12:14:32 UTC
(In reply to Fabian Deutsch from comment #44)
> The lgos from the installation are needed, a screenshot is not sufficient to
> debug the issue.

Actually, the logs aren't necessary either.

This will work downstream, but the problem upstream (or with CentOS images) is that the installclass sets efi_dir to "redhat" (adding "oVirt Node" was a relatively quick fix, and efi_dir was not set)

This will work tomorrow:

https://github.com/evol262/anaconda/commit/2e879667a618d4c6f37def1f477f26acac16b780

Comment 46 Fabian Deutsch 2016-06-16 12:22:16 UTC
Good catch, tho I have a comment on the implementation.

Comment 47 cshao 2016-06-16 14:02:59 UTC
(In reply to Ryan Barry from comment #45)
> (In reply to Fabian Deutsch from comment #44)
> > The lgos from the installation are needed, a screenshot is not sufficient to
> > debug the issue.
> 
> Actually, the logs aren't necessary either.
> 
> This will work downstream, but the problem upstream (or with CentOS images)
> is that the installclass sets efi_dir to "redhat" (adding "oVirt Node" was a
> relatively quick fix, and efi_dir was not set)
> 
> This will work tomorrow:
> 
> https://github.com/evol262/anaconda/commit/
> 2e879667a618d4c6f37def1f477f26acac16b780

Thanks Ryan,
Cancel the needinfo due to the logs aren't necessary.

Comment 48 cshao 2016-06-20 07:53:20 UTC
(In reply to Ryan Barry from comment #45)
> (In reply to Fabian Deutsch from comment #44)
> > The lgos from the installation are needed, a screenshot is not sufficient to
> > debug the issue.
> 
> Actually, the logs aren't necessary either.
> 
> This will work downstream, but the problem upstream (or with CentOS images)
> is that the installclass sets efi_dir to "redhat" (adding "oVirt Node" was a
> relatively quick fix, and efi_dir was not set)
> 
> This will work tomorrow:
> 
> https://github.com/evol262/anaconda/commit/
> 2e879667a618d4c6f37def1f477f26acac16b780


Test version:
rhev-hypervisor7-ng-4.0-20160616.0.x86_64
imgbased-0.7.0-0.1.el7ev.noarch
redhat-release-rhev-hypervisor-4.0-0.7.el7.x86_64

Test result:
Still got failed with above downstream build.

details info please refer "uefi" log.

Comment 49 cshao 2016-06-20 08:00:38 UTC
Created attachment 1169744 [details]
uefi.log

Comment 50 cshao 2016-06-20 08:01:13 UTC
Created attachment 1169745 [details]
uefi.png

Comment 51 Fabian Deutsch 2016-06-20 09:39:59 UTC
The screenshot in comment 50 looks like a different bug,

Please provide the anaconda logs.

Comment 52 cshao 2016-06-20 09:48:25 UTC
Created attachment 1169804 [details]
/tmp/all_log

Comment 53 Fabian Deutsch 2016-06-20 10:22:18 UTC
From the logs:

RuntimeError: An existing imgbase was found with tags, but imgbase was called with --init. If this wasintentional, please untag the existing volumes and try again.

Did you ensure to remove the existing VG from teh disk, i.e. delete everything which was on disk?

Comment 54 cshao 2016-06-21 06:37:45 UTC
(In reply to Fabian Deutsch from comment #53)
> From the logs:
> 
> RuntimeError: An existing imgbase was found with tags, but imgbase was
> called with --init. If this wasintentional, please untag the existing
> volumes and try again.
> 
> Did you ensure to remove the existing VG from teh disk, i.e. delete
> everything which was on disk?

After remove the existing VG on uefi machine, the issue from #c48 will gone, so please ignore #c48.  but still met the original bug.

Comment 55 Fabian Deutsch 2016-06-21 10:05:50 UTC
Ryan,
I suppose this is because the install class patch was reverted, maybe we can change the title again to make sure EFi is working.

Comment 56 cshao 2016-06-21 10:48:23 UTC
Created attachment 1170204 [details]
uefi_all_log

Comment 57 Ryan Barry 2016-06-21 13:08:03 UTC
(In reply to Fabian Deutsch from comment #55)
> Ryan,
> I suppose this is because the install class patch was reverted, maybe we can
> change the title again to make sure EFi is working.

The situation here is confusing, this this bug is being tested both against upstream and downstream.

Yes, the patch was reverted downstream (though it's re-applied as of last week, and will be in the next engineering distill).

However, it's present upstream.

To verify this downstream, we can have jboggs distill a new beta 1 ISO, or wait for a distilled beta 2 image (or whatever included anaconda-21.48.22.56-5.el7ev).

But this should be able to be verified right now upstream.

Comment 58 cshao 2016-06-22 16:06:28 UTC
Test version:
RHEV-H-7.2-20160621.4-RHVH-x86_64-dvd1.iso
imgbased-0.7.0-0.1.el7ev.noarch
redhat-release-rhev-hypervisor-4.0-0.7.el7.x86_64

Test machine:
Dell R210  UEFI 

Test steps:
1. Boot from iSO.
2. Anaconda interactive install NGN 4.0 in UEFI mode.
3. Install NGN on local storage.
4. Reboot NGN 
5. Login NGN in UEFI mode.

Test result:
After step3, no error occurred during the installation process.
After step5, login NGN in UEFI mode can successful.

Comment 59 cshao 2016-07-28 07:31:09 UTC
Test version:
redhat-virtualization-host-4.0-20160714.3
imgbased-0.7.2-0.1.el7ev.noarch
redhat-release-virtualization-host-4.0-0.20.el7.x86_64

Test steps:
1. Anaconda install RHVH 4.0 in UEFI machine.
2. Finish the installation with correct steps.
3. Reboot RHVH.
4. Boot RHVH on UEFI mode.

Test result:
RHVH can be bootable in UEFI machine.

So the bug is fixed, change bug status to VERIFIED.