Skip to content

[WIP] BZ1975701: Add Tang workaround for vSphere to rel notes #37356

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

Closed
wants to merge 1 commit into from

Conversation

bobfuru
Copy link
Contributor

@bobfuru bobfuru commented Oct 11, 2021

BZ1975701
As mentioned in #33497 (comment), this adds a known issue and documented workaround to be added to the OCP 4.9 release notes. I will look at adding the workaround to the Encrypting and mirroring disks during installation doc in a separate PR.

Preview link (last bullet): https://deploy-preview-37356--osdocs.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-9-release-notes?utm_source=github&utm_campaign=bot_dp#ocp-4-9-known-issues

@bobfuru bobfuru added this to the Future Release milestone Oct 11, 2021
@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 11, 2021
@netlify
Copy link

netlify bot commented Oct 11, 2021

✔️ Deploy Preview for osdocs ready!

🔨 Explore the source changes: fe5ac3f

🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/616483bd6517fc00080f8b5c

😎 Browse the preview: https://deploy-preview-37356--osdocs.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-9-release-notes

@bobfuru
Copy link
Contributor Author

bobfuru commented Oct 11, 2021

@jinyunma PTAL for QE ack.
@dustymabe PTAL for Eng ack.
🙏

nameserver='8.8.8.8'
echo "ip=${ip}::${gateway}:${netmask}:${hostname}:${interface}:none:${nameserver}"
----

Choose a reason for hiding this comment

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

The workaround listed here is not the right one I applied and verified, because this is node-specific config and need to be applied on each master/worker node ( IP is different on each node), I use the workaround provided in doc https://docs.google.com/document/d/1rnAutjuxzUAauqOZwc8Iccl7SSZVWGJXH5VJL08YvTU/edit#, which also mentioned in https://bugzilla.redhat.com/show_bug.cgi?id=1975701#c12

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, @jinyunma - I'll update this to address your feedback so this can make it in the 4.9 release notes.

@bobfuru bobfuru changed the title BZ1975701: Add Tang workaround for vSphere to rel notes [WIP] BZ1975701: Add Tang workaround for vSphere to rel notes Oct 12, 2021
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 12, 2021
@bobfuru
Copy link
Contributor Author

bobfuru commented Oct 13, 2021

We've had a high volume of added known issues to this file, which has caused a number of conflicts. As a result, I'm closing this PR and have started over with #37500

@bobfuru bobfuru closed this Oct 13, 2021
@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 13, 2021
@openshift-ci
Copy link

openshift-ci bot commented Oct 13, 2021

@bobfuru: PR needs rebase.

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.

@vikram-redhat vikram-redhat removed this from the Future Release milestone Oct 18, 2021
@bobfuru bobfuru deleted the BZ1975701-rn branch November 16, 2021 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants