Skip to content

installation: provide rd.neednet=1 when using Tang disk encryption#21177

Merged
kalexand-rh merged 1 commit intoopenshift:masterfrom
miabbott:tang_fix_neednet
Oct 12, 2020
Merged

installation: provide rd.neednet=1 when using Tang disk encryption#21177
kalexand-rh merged 1 commit intoopenshift:masterfrom
miabbott:tang_fix_neednet

Conversation

@miabbott
Copy link
Member

In BZ#1819215[1] an error was reported that the RHCOS nodes did not
successfully reboot when Tang disk encryption was used. The root
cause was the missing kernel argument rd.neednet=1.

This updates the docs to call out the requirement on that kernel
argument when using Tang for disk encryption.

@miabbott miabbott force-pushed the tang_fix_neednet branch 2 times, most recently from 634c00c to 2ad814f Compare April 15, 2020 14:50
@miabbott miabbott changed the title installtion: provide rd.neednet=1 when using Tang disk encryption installation: provide rd.neednet=1 when using Tang disk encryption Apr 15, 2020
Copy link
Contributor

Choose a reason for hiding this comment

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

@chrisnegus, will you PTAL?

Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't know this. we should confirm with @darkmuggle

Copy link
Member

Choose a reason for hiding this comment

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

@darkmuggle can you confirm for @chrisnegus?

Copy link
Member Author

Choose a reason for hiding this comment

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

This addition is implicitly confirmed by @darkmuggle in this comment on the linked BZ - https://bugzilla.redhat.com/show_bug.cgi?id=1819215#c5

@darkmuggle
Copy link

LGTM

@openshift-bot
Copy link

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci-robot openshift-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 10, 2020
@miabbott
Copy link
Member Author

/remove-lifecycle stale

This change is still useful/required for customers deploying older versions of RHCOS/OCP

@openshift-ci-robot openshift-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 12, 2020
@kalexand-rh
Copy link
Contributor

@miabbott, if you rebase to fix that error and squash, I'll merge this change.

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 12, 2020
@miabbott
Copy link
Member Author

@kalexand-rh oops, missed that conflict initially...all fixed now ⬆️

Copy link
Contributor

@kalexand-rh kalexand-rh left a comment

Choose a reason for hiding this comment

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

@miabbott, will you please make one more update and squash?

In BZ#1819215[1] an error was reported that the RHCOS nodes did not
successfully reboot when Tang disk encryption was used.  The root
cause was the missing kernel argument `rd.neednet=1`.

This updates the docs to call out the requirement on that kernel
argument when using Tang for disk encryption.

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1819215
@kalexand-rh kalexand-rh merged commit 49b302c into openshift:master Oct 12, 2020
@kalexand-rh
Copy link
Contributor

kalexand-rh commented Oct 12, 2020

/cherrypick enterprise-4.6

@kalexand-rh
Copy link
Contributor

kalexand-rh commented Oct 12, 2020

/cherrypick enterprise-4.5

@openshift-cherrypick-robot
Copy link

openshift-cherrypick-robot commented Oct 12, 2020

@kalexand-rh: new pull request created: #26342

Details

In response to this:

/cherrypick enterprise-4.6

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.

@openshift-cherrypick-robot
Copy link

openshift-cherrypick-robot commented Oct 12, 2020

@kalexand-rh: new pull request created: #26343

Details

In response to this:

/cherrypick enterprise-4.5

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.

@kalexand-rh
Copy link
Contributor

kalexand-rh commented Oct 12, 2020

/cherrypick enterprise-4.4

@openshift-cherrypick-robot
Copy link

openshift-cherrypick-robot commented Oct 12, 2020

@kalexand-rh: new pull request created: #26345

Details

In response to this:

/cherrypick enterprise-4.4

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.

@kalexand-rh
Copy link
Contributor

kalexand-rh commented Oct 12, 2020

/cherrypick enterprise-4.3

@openshift-cherrypick-robot
Copy link

openshift-cherrypick-robot commented Oct 12, 2020

@kalexand-rh: new pull request created: #26346

Details

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.

@miabbott
Copy link
Member Author

Thanks @kalexand-rh 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.3 branch/enterprise-4.4 branch/enterprise-4.5 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.

8 participants