-
Notifications
You must be signed in to change notification settings - Fork 1.4k
OCPBUGS-58216: allow to deploy single nodes with UPI #9763
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
Conversation
// Single node OpenShift installations supported without `bootstrapInPlace` | ||
return true | ||
case nonetypes.Name, externaltypes.Name: | ||
case externaltypes.Name: |
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.
Isn't external
platform basically none
platform?
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.
if that was the case, the current code would be redundant right?
I think external means the oracle integration used with assisted installer
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.
Right, that's one of the use cases (but it could be other external providers like digital ocean, alibaba).
But no idea why I put this non-sense question here, sorry (must be a long friday, please ignore this 😅 ) Do you have any e2e tests in mind to run to check this change?
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.
/cc @sadasu
/retest |
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.
/lgtm
The original change is from #9048. This has been backported to 4.18. Do you plan to backport this (might need a chain of OCPBUGS jira ticket)?
/cc @patrickdillon @patrickdillon or @sadasu can provide a more meaningful approval here. |
@sadasu can you review this? You are much more familiar with the SNO landscape than I am. |
Hi, thanks for opening this PR. We're encountering the same issue while deploying a Single Node OpenShift (SNO) cluster on bare metal using the UPI approach. The installer fails with the following error:
This deployment method—using Here's the relevant apiVersion: v1
baseDomain: ${CLUSTER_BASE_DOMAIN}
fips: ${FIPS_ENABLED}
metadata:
name: ${CLUSTER_NAME}
controlPlane:
architecture: ppc64le
hyperthreading: Enabled
name: master
replicas: 1
networking:
clusterNetwork:
- cidr: ${CLUSTER_NETWORK}/14
hostPrefix: 23
machineNetwork:
- cidr: ${MACHINE_NETWORK}/24
networkType: ${CLUSTER_NETWORK_TYPE}
serviceNetwork:
- 172.30.0.0/16
compute:
- architecture: ${ARCH}
hyperthreading: Enabled
name: worker
replicas: 0
platform:
none: {}
pullSecret: ${PULL_SECRET}
sshKey: ${SSH_PUB_KEY} It seems the recent change is unintentionally breaking support for this long-standing approach. We'd really appreciate this fix being merged—happy to help validate or provide additional details if needed. |
Hello. We are now having this same issue in our Pipelines once we have moved from 4.20.0-ec.2 to 4.20.0-ec.3
We'd really appreciate/need this fix being merged soon. |
This is a workaround until the openshift/installer#9763 PR is merged. Then, revert this change. Signed-off-by: Carlos Cardenosa <[email protected]>
This is a workaround until the openshift/installer#9763 PR is merged. Then, revert this change. Signed-off-by: Carlos Cardenosa <[email protected]>
This is a workaround until the openshift/installer#9763 PR is merged. [Forgot to remove PR condition in the previous commit] Then, revert this change. Signed-off-by: Carlos Cardenosa <[email protected]>
This is a workaround until the openshift/installer#9763 PR is merged. [Forgot to remove PR condition in the previous commit] Then, revert this change. Signed-off-by: Carlos Cardenosa <[email protected]>
This is a workaround until the openshift/installer#9763 PR is merged. [Forgot to remove PR condition in the previous commit] Then, revert this change. Signed-off-by: Carlos Cardenosa <[email protected]>
/retitle OCPBUGS-58216: allow to deploy single nodes with UPI |
@karmab: This pull request references Jira Issue OCPBUGS-58216, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
Requesting review from QA contact: The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
@karmab and @Vaibhav-Nazare, it was my understanding that platform: none was always being used with @cybertron @mkowalski making sure this is fine w.r.t on-prem networking. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sadasu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
UPI, platform none -- would be very inappropriate for on-prem net team to have anything to block here :) |
Thanks @mkowalski . Noted. |
/cherry-pick release-4.19 |
@sadasu: once the present PR merges, I will cherry-pick it on top of In response to this:
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-sigs/prow repository. |
@karmab: The following tests failed, say
Full PR test history. Your PR dashboard. 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-sigs/prow repository. I understand the commands that are listed here. |
2936edc
into
openshift:main
@karmab: Jira Issue OCPBUGS-58216: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-58216 has been moved to the MODIFIED state. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
@sadasu: new pull request created: #9812 In response to this:
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-sigs/prow repository. |
This PR allows back deployment of single node openshift using UPI with platform None and without bootstrap in place, since it has worked fine for ever and is used at least internally