Project

General

Profile

Bug #17900

Installer falls over in current 10-Nightly buld

Added by Sean Fagan almost 3 years ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Critical
Assignee:
Vaibhav Chauhan
Category:
OS
Target version:
9.10.2
Seen in:
N/A
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.

Screen Shot 2016-09-29 at 8.15.20 PM.png (178 KB) Screen Shot 2016-09-29 at 8.15.20 PM.png Jordan Hubbard, 09/29/2016 08:17 PM
7297

Associated revisions

Revision c492ebdb (diff)
Added by Sean Fagan almost 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 almost 3 years ago

Fix a late-night-o'd. THANK YOU WILLIAMD AND KRIS.

Ticket: #17900

History

#2 Updated by Sean Fagan almost 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 almost 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 almost 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 almost 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 almost 3 years ago

Gah, thanks, you guys are awesome.

It was late. :)

I've updated the branch and master.

#7 Updated by Sean Fagan almost 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 over 2 years ago

  • Status changed from Reviewed to Ready For Release

#9 Updated by Dru Lavigne over 1 year ago

  • Status changed from Ready For Release to Resolved

Also available in: Atom PDF