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

[1.2] Add a task to pull quanta fw update tools in microkernel #221

Merged

Conversation

iceiilin
Copy link
Member

@iceiilin iceiilin commented May 26, 2016

To address ODR 726 about BIOS update workflow stuck when issuing the second time.

From the vendor, BIOS settings needs to be cleared when updating, and the default network setting is UEFI. In submitter's system, under this condition, the specific microkernel 3.13 for updating FW kernel panic when loading, while microkernel 3.16 for discovery works well.

In this PR, I changed to use 3.16 microkernel for FW update, and add a task to pull the FW update tools after entering microkernel.

Another change in this PR. BMC network setting got lost after BMC FW update. The default setting is DHCP, and it didn't impact machines whose BMC is in RackHD 172 subnet. But for BMC with static IP previously, FW update would change BMC's IP. The task changes from a list of update commands to a script, saving the setting and restoring after flashing.

Related PR:
RackHD/on-http#316
RackHD/on-taskgraph#110

@coveralls
Copy link

coveralls commented May 26, 2016

Coverage Status

Coverage increased (+0.002%) to 87.437% when pulling 39ace39 on iceiilin:bugfix/1.2.0-odr726-bios into c3da1c8 on RackHD:release/1.2.0-branch.

@yyscamper yyscamper changed the title Add a task to pull quanta fw update tools in microkernel [1.2] Add a task to pull quanta fw update tools in microkernel May 26, 2016
@yyscamper
Copy link
Contributor

👍

@JenkinsRHD
Copy link
Contributor

*** BUILD #880 ***

@johren
Copy link
Contributor

johren commented May 26, 2016

👎 We do not want this in the 1.2 branch right now. We are on the verge of cutting this release.

@coveralls
Copy link

coveralls commented May 27, 2016

Coverage Status

Coverage increased (+0.002%) to 87.437% when pulling f51d718 on iceiilin:bugfix/1.2.0-odr726-bios into c3da1c8 on RackHD:release/1.2.0-branch.

@JenkinsRHD
Copy link
Contributor

*** BUILD #882 ***

@anhou
Copy link
Member

anhou commented Jun 1, 2016

@johren Just want to confirm does this PR could be merged to 1.2 branch now?

@johren
Copy link
Contributor

johren commented Jun 2, 2016

👍
FYI @frostr3 @derrickostertag

@anhou
Copy link
Member

anhou commented Jun 2, 2016

Hold on merging, Just heard from @iceiilin she needs some refinements for this PR.

@anhou anhou merged commit 8410319 into RackHD:release/1.2.0-branch Jun 6, 2016
kellylu2sym pushed a commit to kellylu2sym/on-tasks that referenced this pull request Aug 8, 2017
Replace bootstrap ubuntu with rancherOS in discovery and Micro-docker
@iceiilin iceiilin deleted the bugfix/1.2.0-odr726-bios branch September 4, 2017 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants