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

Need updated pxegrub file to address OS-3634 and OS-3635 #1

Open
nshalman opened this issue Feb 4, 2015 · 5 comments
Open

Need updated pxegrub file to address OS-3634 and OS-3635 #1

nshalman opened this issue Feb 4, 2015 · 5 comments

Comments

@nshalman
Copy link

nshalman commented Feb 4, 2015

OS-3634 and OS-3635 improve how grub deals with serial ports.
Someone on IRC ran into a problem that was addressed by replacing the current tftpboot/pxegrub file with a fresh one from the proto dir of a platform build.

Quick fix is to grab a fresh one and check it in.
A more thorough though more complicated fix is to package up pxegrub in the boot tarball generated by the build process (tweak a manifest in illumos-joyent e.g. nshalman/illumos-joyent@c772887) and then have sdc-booter somehow grab pxegrub out of that as a build step.

@trentm
Copy link
Contributor

trentm commented Feb 4, 2015

@rgulewich ping, thoughts?

@nshalman nshalman mentioned this issue Feb 6, 2015
@sperreault
Copy link

Could we just use a quick fix here and copy a pxegrub file that works (i'm doing this manually right now)

@nshalman
Copy link
Author

I agree. One of these two tickets (this one and #2) should be fixed by checking in a working binary. The other could be left open for the possibility of the fancier fix.

@nshalman
Copy link
Author

nshalman commented Apr 9, 2015

RELENG-608 (TritonDataCenter/illumos-joyent@62890e2) provides the raw materials for both the quick fix and the better fix.

@nshalman
Copy link
Author

nshalman commented Aug 2, 2016

@jclulow I just noticed https://smartos.org/bugview/HEAD-2315

While you're doing GRUB related stuff, perhaps you could take a look at this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants