-
Notifications
You must be signed in to change notification settings - Fork 233
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
Update proc_starting-the-virtual-machine.adoc #871
Conversation
added Note to setup instructions to account for problem such as "Bundle 'crc_hyperkit_4.2.8.crcbundle' was requested, but the existing VM is using 'crc_hyperkit_4.2.2.crcbundle'"
Can one of the admins verify this patch? |
docs/source/getting-started/content/proc_starting-the-virtual-machine.adoc
Outdated
Show resolved
Hide resolved
Revised note to generalize for dependency mismatch
docs/source/getting-started/content/proc_starting-the-virtual-machine.adoc
Outdated
Show resolved
Hide resolved
created specific troubleshooting section for bundle version error
added new troubleshooting topic on bundle version mismatch
Revised note on error on start.
Added note on the need to capture any changes made to the instances
docs/source/getting-started/content/proc_starting-the-virtual-machine.adoc
Outdated
Show resolved
Hide resolved
[id="troubleshooting-bundle-version-mismatch_{context}"] | ||
= Troubleshooting bundle version mismatch | ||
|
||
As {prod} releases are setup and started, instances are created with bundle information |
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.
"set up" instead of "setup". This applies generally.
Also, could you please keep each sentence on its own line? We don't particularly care about the line length, but want to keep each sentence isolated so that diffs are cleaner. :)
docs/source/getting-started/content/proc_troubleshooting-bundle-version-mismatch.adoc
Show resolved
Hide resolved
docs/source/getting-started/content/proc_troubleshooting-bundle-version-mismatch.adoc
Outdated
Show resolved
Hide resolved
https://github.com/cfergeau/crc/commits/pr871 should have the changes requested by @robin-owen |
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.
In my opinion, this will be ready to merge once we have each sentence placed on its own line.
docs/source/getting-started/content/proc_troubleshooting-bundle-version-mismatch.adoc
Outdated
Show resolved
Hide resolved
docs/source/getting-started/content/proc_troubleshooting-bundle-version-mismatch.adoc
Outdated
Show resolved
Hide resolved
separated sentences into separate lines
Fixes #870
added Note to setup instructions to account for problem such as "Bundle 'crc_hyperkit_4.2.8.crcbundle' was requested, but the existing VM is using 'crc_hyperkit_4.2.2.crcbundle'"