Project

General

Profile

Bug #17900

Installer falls over in current 10-Nightly buld

Added by Sean Fagan about 3 years ago. Updated about 2 years ago.

Status:
Resolved
Priority:
Critical
Assignee:
Vaibhav Chauhan
Category:
OS
Target version:
Seen in:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

See attached screenshot. This was an ISO redirect erase install using the BIOS boot method on z35ref-a.sjlab1.ixsystems.com (IPMI is on https://10.212.3.4).

z35ref is sadly dead in the water until we fix this.
--------

Need to make the same change to FN9.

Associated revisions

Revision c492ebdb (diff)
Added by Sean Fagan about 3 years ago

If we've not chosen EFI, then set BOOTMODE to BIOS. Else wackiness. Ticket: #17900

Revision aa4b935e (diff)
Added by Sean Fagan about 3 years ago

Fix a late-night-o'd. THANK YOU WILLIAMD AND KRIS. Ticket: #17900

History

#2 Updated by Sean Fagan about 3 years ago

  • Status changed from Unscreened to Needs Developer Review

Oops, I should have asked for review before I checked it into master. Kris can you review?

Branch FIX-17900, but in master as c492ebdbb80ce16fa255955d17dca15f60543289

I think this needs to go into the next update, since it'll make certain systems non-installable.

#3 Updated by William Grzybowski about 3 years ago

I am not kris, but this looks strange

ask boot method, and then set BOOTMODE to efi in both branches of the if statement?

         if ask_boot_method
         then
           BOOTMODE="efi" 
+       else
+         BOOTMODE="efi" 
         fi

#4 Avatar?id=14398&size=24x24 Updated by Kris Moore about 3 years ago

Yea, that does look wack. Shouldn't it be :

if ask_boot_method
then
BOOTMODE="efi"
+ else
+ BOOTMODE="bios"
fi

#5 Avatar?id=14398&size=24x24 Updated by Kris Moore about 3 years ago

  • Target version changed from 9.10.1-U2 to 9.10.2
  • Seen in changed from 9.10.1-U1 to N/A

The underlying changes hadn't made it into 9.10.1, slated for 9.10.2 BTW

#6 Updated by Sean Fagan about 3 years ago

Gah, thanks, you guys are awesome.

It was late. :)

I've updated the branch and master.

#7 Updated by Sean Fagan about 3 years ago

  • Status changed from Needs Developer Review to Reviewed
  • Assignee changed from Sean Fagan to Vaibhav Chauhan

This has been reviewed I think, and in build testing for a while. Over to VB.

#8 Updated by Vaibhav Chauhan about 3 years ago

  • Status changed from Reviewed to Ready For Release

#9 Updated by Dru Lavigne about 2 years ago

  • Status changed from Ready For Release to Resolved

Also available in: Atom PDF