-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
BZ190696: Add warning about protracted boot time with no feedback #37233
Conversation
✔️ Deploy Preview for osdocs ready! 🔨 Explore the source changes: f26acf0 🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/6166f4a97120b5000892daec 😎 Browse the preview: https://deploy-preview-37233--osdocs.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal_ipi/ipi-install-prerequisites |
2bb7c0a
to
1d146ea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good @tmulquee. A few suggestions below.
But first:
- please edit your title to fit the format "BZ999999: descriptive title"
- does this really apply only to
enterprise-4.8
? If so, you'll need to base your feature branch on 4.8 and merge only to 4.8
|
||
[WARNING] | ||
==== | ||
Boot times can take up to 10 minutes without any indication of progress. Do not assume the boot is stuck, or abort or reboot during this installation, as this could require troubleshooting on the next installation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boot times can take up to 10 minutes without any indication of progress. Do not assume the boot is stuck, or abort or reboot during this installation, as this could require troubleshooting on the next installation. | |
Boot times can take up to 10 minutes without any indication of progress. Do not cancel the installation or restart during this period because this could require troubleshooting on the next installation. |
A suggested rewrite, for clarity and to remove terms 'reboot', 'abort', 'as' (see ISG glossary).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the feedback, Chris, update applied.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR description updated as requested.
More specific destails on BZ190696 Minor edits for BZ190696
1d146ea
to
f26acf0
Compare
https://bugzilla.redhat.com/show_bug.cgi?id=1960696
Added Warning: Boot times are protracted without any indication of progress. Do not assume the boot is stuck, or abort or reboot during this installation.
Merge to enterprise-4.8.
Browse to preview: https://deploy-preview-37233--osdocs.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal_ipi/ipi-install-prerequisites?utm_source=github&utm_campaign=bot_dp#ipi-install-firmware-requirements-for-installing-with-virtual-media_ipi-install-prerequisites
SME feedback: The new wording is better IMHO. This way the deploying person(s) won't way for hours/days to start debugging.
Thanks! (Full details in the BZ)