Skip to content

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

Merged
merged 1 commit into from
Jul 1, 2025

Conversation

karmab
Copy link
Contributor

@karmab karmab commented May 30, 2025

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

@openshift-ci openshift-ci bot requested review from barbacbd and tthvo May 30, 2025 12:25
// Single node OpenShift installations supported without `bootstrapInPlace`
return true
case nonetypes.Name, externaltypes.Name:
case externaltypes.Name:
Copy link
Member

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?

Copy link
Contributor Author

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

Copy link
Member

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?

Copy link
Member

@tthvo tthvo left a comment

Choose a reason for hiding this comment

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

/cc @sadasu

@openshift-ci openshift-ci bot requested a review from sadasu May 30, 2025 17:57
@karmab
Copy link
Contributor Author

karmab commented Jun 2, 2025

/retest

Copy link
Member

@tthvo tthvo left a 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)?

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 2, 2025
@barbacbd
Copy link
Contributor

barbacbd commented Jun 2, 2025

/cc @patrickdillon

@patrickdillon or @sadasu can provide a more meaningful approval here.

@openshift-ci openshift-ci bot requested a review from patrickdillon June 2, 2025 19:55
@patrickdillon
Copy link
Contributor

@sadasu can you review this? You are much more familiar with the SNO landscape than I am.

@Vaibhav-Nazare
Copy link

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:

level=fatal msg=failed to fetch Bootstrap Ignition Config: failed to fetch dependency of "Bootstrap Ignition Config": failed to generate asset "Master Machines": this install method does not support Single Node installation on platform none

This deployment method—using platform: none with controlPlane.replicas: 1 and no worker nodes—has been working for a long time and is a common setup for SNO on bare metal with UPI.

Here's the relevant install-config.yaml snippet:

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.

@ccardenosa
Copy link

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

Using installer version 4.20.0-ec.3
Using image rhcos-9.6.20250523-0-openstack.x86_64.qcow2
FATAL failed to fetch Master Machines: failed to generate asset "Master Machines": this install method does not support Single Node installation on platform none

We'd really appreciate/need this fix being merged soon.

ccardenosa added a commit to ccardenosa/release that referenced this pull request Jun 26, 2025
This is a workaround until the
openshift/installer#9763
PR is merged.

Then, revert this change.

Signed-off-by: Carlos Cardenosa <[email protected]>
openshift-merge-bot bot pushed a commit to openshift/release that referenced this pull request Jun 26, 2025
This is a workaround until the
openshift/installer#9763
PR is merged.

Then, revert this change.

Signed-off-by: Carlos Cardenosa <[email protected]>
ccardenosa added a commit to ccardenosa/release that referenced this pull request Jun 27, 2025
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]>
ccardenosa added a commit to ccardenosa/release that referenced this pull request Jun 27, 2025
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]>
openshift-merge-bot bot pushed a commit to openshift/release that referenced this pull request Jun 27, 2025
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]>
@tthvo
Copy link
Member

tthvo commented Jun 27, 2025

/retitle OCPBUGS-58216: allow to deploy single nodes with UPI

@openshift-ci openshift-ci bot changed the title Allow to deploy single nodes with UPI OCPBUGS-58216: allow to deploy single nodes with UPI Jun 27, 2025
@openshift-ci-robot openshift-ci-robot added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. labels Jun 27, 2025
@openshift-ci-robot
Copy link
Contributor

@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
  • bug is open, matching expected state (open)
  • bug target version (4.20.0) matches configured target version for branch (4.20.0)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @gpei

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

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

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.

@openshift-ci openshift-ci bot requested a review from gpei June 27, 2025 22:23
@sadasu
Copy link
Contributor

sadasu commented Jun 30, 2025

@karmab and @Vaibhav-Nazare, it was my understanding that platform: none was always being used with bootstrapInplace.

@cybertron @mkowalski making sure this is fine w.r.t on-prem networking.

/approve

Copy link
Contributor

openshift-ci bot commented Jun 30, 2025

[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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 30, 2025
@mkowalski
Copy link
Contributor

UPI, platform none -- would be very inappropriate for on-prem net team to have anything to block here :)

@sadasu
Copy link
Contributor

sadasu commented Jun 30, 2025

UPI, platform none -- would be very inappropriate for on-prem net team to have anything to block here :)

Thanks @mkowalski . Noted.

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD e6e1537 and 2 for PR HEAD 34c9fdd in total

@sadasu
Copy link
Contributor

sadasu commented Jun 30, 2025

/cherry-pick release-4.19

@openshift-cherrypick-robot

@sadasu: once the present PR merges, I will cherry-pick it on top of release-4.19 in a new PR and assign it to you.

In response to this:

/cherry-pick release-4.19

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.

Copy link
Contributor

openshift-ci bot commented Jun 30, 2025

@karmab: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/okd-scos-e2e-aws-ovn 34c9fdd link false /test okd-scos-e2e-aws-ovn
ci/prow/e2e-vsphere-ovn-multi-network 34c9fdd link false /test e2e-vsphere-ovn-multi-network

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.

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD e6e1537 and 2 for PR HEAD 34c9fdd in total

@openshift-merge-bot openshift-merge-bot bot merged commit 2936edc into openshift:main Jul 1, 2025
16 of 18 checks passed
@openshift-ci-robot
Copy link
Contributor

@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:

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

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.

@openshift-cherrypick-robot

@sadasu: new pull request created: #9812

In response to this:

/cherry-pick release-4.19

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants