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

Bug 1741922, fixed initrd argument #17561

Merged
merged 1 commit into from
Dec 3, 2019

Conversation

ahardin-rh
Copy link
Contributor

@ahardin-rh ahardin-rh added this to the Next Release milestone Oct 23, 2019
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 23, 2019
@openshift-docs-preview-bot

The preview will be available shortly at:

@@ -63,7 +63,7 @@ TIMEOUT 20
PROMPT 0
LABEL pxeboot
KERNEL http://<HTTP_server>/rhcos-<version>-<architecture>-installer-kernel <1>
APPEND ip=dhcp rd.neednet=1 initrd=http://<HTTP_server>/rhcos-<version>-<architecture>-installer-initramfs.img console=tty0 console=ttyS0 coreos.inst=yes coreos.inst.install_dev=sda coreos.inst.image_url=http://<HTTP_server>/rhcos-<version>-<architecture>-metal-bios.raw.gz coreos.inst.ignition_url=http://<HTTP_server>/bootstrap.ign <2> <3>
APPEND ip=dhcp rd.neednet=1 initrd=rhcos-<version>-<architecture>-installer-initramfs.img console=tty0 console=ttyS0 coreos.inst=yes coreos.inst.install_dev=sda coreos.inst.image_url=http://<HTTP_server>/rhcos-<version>-<architecture>-metal-bios.raw.gz coreos.inst.ignition_url=http://<HTTP_server>/bootstrap.ign <2> <3>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this change should be at line 81 instead, unless it applies to both iPXE and PXE.

@morenod, will you PTAL?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

both PXE and iPXE can use http locations for kerner, initrd and image files

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@morenod Thanks. I'm looking into your comment in the BZ.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rvanderp3 See the above comment. Per the BZ, could the problem be that http:\the-server:8080\rhcos-4.1.0-x86_64-installer-initramfs.img should be http:// instead of http:\?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ahardin-rh ahardin-rh self-assigned this Oct 23, 2019
@jianlinliu
Copy link

@morenod could you help review this.

@morenod
Copy link

morenod commented Dec 2, 2019

lgtm

@ahardin-rh ahardin-rh added the peer-review-done Signifies that the peer review team has reviewed this PR label Dec 3, 2019
@ahardin-rh ahardin-rh merged commit 457698e into openshift:master Dec 3, 2019
@ahardin-rh
Copy link
Contributor Author

/cherrypick enterprise-4.2

@openshift-cherrypick-robot

@ahardin-rh: new pull request created: #18406

In response to this:

/cherrypick enterprise-4.2

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ahardin-rh
Copy link
Contributor Author

/cherrypick enterprise-4.3

@openshift-cherrypick-robot

@ahardin-rh: new pull request created: #18407

In response to this:

/cherrypick enterprise-4.3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.2 branch/enterprise-4.3 peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants