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

Revert "add support for UEFI HTTP Boot" #227

Merged
merged 1 commit into from
Dec 1, 2021

Conversation

mmlb
Copy link
Contributor

@mmlb mmlb commented Dec 1, 2021

Description

This reverts commit ceb470a.

Why is this needed

Something in #216 broke our c2.large.arm provisions. Reverting
this to unblock things that came after while I look into debugging.

Some more info in #225.

How Has This Been Tested?

Tested via sandbox against EM c2.large.arm

This reverts commit ceb470a.

Something in tinkerbell#216 broke our c2.large.arm provisions. Reverting
this to unblock things that came after while I look into debugging.

Some more info in [tinkerbell#225].

[tinkerbell#225]: tinkerbell#225

Signed-off-by: Manuel Mendez <mmendez@equinix.com>
@codecov
Copy link

codecov bot commented Dec 1, 2021

Codecov Report

Merging #227 (8a893a6) into main (0728d55) will increase coverage by 0.07%.
The diff coverage is 58.82%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #227      +/-   ##
==========================================
+ Coverage   38.87%   38.95%   +0.07%     
==========================================
  Files          41       41              
  Lines        2765     2765              
==========================================
+ Hits         1075     1077       +2     
+ Misses       1604     1602       -2     
  Partials       86       86              
Impacted Files Coverage Δ
cmd/boots/tftp.go 14.40% <0.00%> (+0.24%) ⬆️
job/http.go 0.00% <0.00%> (ø)
job/job.go 35.05% <ø> (+1.38%) ⬆️
job/logging.go 33.33% <0.00%> (-16.67%) ⬇️
job/dhcp.go 57.95% <90.90%> (+0.97%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0728d55...8a893a6. Read the comment docs.

@mmlb mmlb marked this pull request as ready for review December 1, 2021 21:03
@mmlb mmlb requested a review from jacobweinstock December 1, 2021 21:03
@mmlb mmlb added the ready-to-merge Signal to Mergify to merge the PR. label Dec 1, 2021
@mergify mergify bot merged commit e94a4c8 into tinkerbell:main Dec 1, 2021
@mmlb mmlb deleted the revert-pr216 branch December 1, 2021 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants