Skip to content

OCPBUGS-55123: Fix apiVersion in compute machine set specs #92348

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

@openshift-ci-robot
Copy link

@jeana-redhat: This pull request references Jira Issue OCPBUGS-55123, 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.19.0) matches configured target version for branch (4.19.0)
  • bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @huali9

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

In response to this:

Version(s):
4.12+

Issue:
OCPBUGS-55123

Link to docs preview:
🚧

QE review:

  • QE has approved this change.

Additional information:

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 huali9 April 17, 2025 17:53
@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Apr 17, 2025
Copy link

openshift-ci bot commented Apr 17, 2025

@jeana-redhat: all tests passed!

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

@jeana-redhat: This pull request references Jira Issue OCPBUGS-55123, which is valid.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.19.0) matches configured target version for branch (4.19.0)
  • bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @huali9

In response to this:

Version(s):
4.12+

Issue:
OCPBUGS-55123

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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.

@jeana-redhat
Copy link
Contributor Author

@huali9 a global search to fix these turned up some instances in WinC and Multiarch. If those cases are outside your scope, let me know 🙂

@huali9
Copy link

huali9 commented Apr 18, 2025

Hi @jeana-redhat
Although I think WinC and Multiarch instances should be the same with normal instances in that respect(apiVersion), because in our CI I can see that they are all copied from a default worker machineset and then update some specific settings. For example: https://github.com/openshift/release/blob/master/ci-operator/step-registry/ipi/conf/vsphere/windows-machineset/ipi-conf-vsphere-windows-machineset-commands.sh#L23-L41
https://github.com/openshift/release/blob/master/ci-operator/step-registry/ipi/install/heterogeneous/ipi-install-heterogeneous-commands.sh#L195-L202 and https://github.com/openshift/release/blob/master/ci-operator/step-registry/ipi/install/heterogeneous/ipi-install-heterogeneous-commands.sh#L315-L317

But I think we'd better not touch WinC and Multiarch part, because we don't create WinC and Multiarch machines by default, and both apiVersion work. In addition, if we change it, I see another inconsistency in WinC and Multiarch part, like https://drive.google.com/file/d/1LqJUE8aOPAc4cwU4WE2I4o2XUsc_nWqs/view?usp=sharing and https://drive.google.com/file/d/17B_vep3X3r3yYqlrMT0HCspLy1y7yRua/view?usp=sharing
and I'm not sure if there are other inconsistencies.

Version(s):
4.12+

For versions, I am not sure from which OCP version the apiVersion was changed, but I'm sure it's machine.openshift.io/v1beta1 since 4.16, so I suggest 4.16+, what do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.12 branch/enterprise-4.13 branch/enterprise-4.14 branch/enterprise-4.15 branch/enterprise-4.16 branch/enterprise-4.17 branch/enterprise-4.18 branch/enterprise-4.19 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. 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.

4 participants