api icon indicating copy to clipboard operation
api copied to clipboard

Promote CPMSMachineNamePrefix feature gate to default feature set

Open chiragkyal opened this issue 7 months ago • 3 comments

/hold

chiragkyal avatar Mar 31 '25 08:03 chiragkyal

Hello @chiragkyal! Some important instructions when contributing to openshift/api: API design plays an important part in the user experience of OpenShift and as such API PRs are subject to a high level of scrutiny to ensure they follow our best practices. If you haven't already done so, please review the OpenShift API Conventions and ensure that your proposed changes are compliant. Following these conventions will help expedite the api review process for your PR.

openshift-ci[bot] avatar Mar 31 '25 08:03 openshift-ci[bot]

@chiragkyal: This pull request references OAPE-148 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set.

In response to this:

/hold

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-robot avatar Apr 02 '25 06:04 openshift-ci-robot

/test verify-feature-promotion

chiragkyal avatar Apr 10 '25 18:04 chiragkyal

/retest

JoelSpeed avatar Apr 14 '25 09:04 JoelSpeed

error: only 0 tests found, need at least 5 for "CPMSMachineNamePrefix" on {vsphere amd64 ha }
error: only 0 tests found, need at least 5 for "CPMSMachineNamePrefix" on {metal amd64 ha ipv4}
error: only 0 tests found, need at least 5 for "CPMSMachineNamePrefix" on {metal amd64 ha ipv6}
error: only 0 tests found, need at least 5 for "CPMSMachineNamePrefix" on {metal amd64 ha dual}
error: only 0 tests found, need at least 5 for "CPMSMachineNamePrefix" on {aws amd64 ha }
error: only 0 tests found, need at least 5 for "CPMSMachineNamePrefix" on {azure amd64 ha }

It's not able to fetch the jobs, even though they exist. https://sippy.dptools.openshift.org/sippy-ng/tests/4.19/details?filters=%7B%22items%22%3A%5B%7B%22columnField%22%3A%22name%22%2C%22operatorValue%22%3A%22contains%22%2C%22value%22%3A%22FeatureGate%3ACPMSMachineNamePrefix%5D%22%7D%5D%7D

chiragkyal avatar Apr 14 '25 10:04 chiragkyal

There is an issue with the verify feature promotion at the moment, I'm investigating

JoelSpeed avatar Apr 14 '25 10:04 JoelSpeed

/test verify-feature-promotion

JoelSpeed avatar Apr 14 '25 16:04 JoelSpeed

Looking at the current feature promotion data, we don't have enough runs to be able to pass this right now. Could you please trigger 10 runs using gangway on each of AWS, Azure, GCP and vSphere.

This feature doesn't work on metal platforms so we will have to except the metal platform requirements here

JoelSpeed avatar Apr 14 '25 16:04 JoelSpeed

Looking at the current feature promotion data, we don't have enough runs to be able to pass this right now. Could you please trigger 10 runs using gangway on each of AWS, Azure, GCP and vSphere.

I had triggered 3-5 jobs in all these platforms. Might need to trigger more.

chiragkyal avatar Apr 14 '25 16:04 chiragkyal

/retest

chiragkyal avatar Apr 15 '25 09:04 chiragkyal

/test verify-feature-promotion

chiragkyal avatar Apr 15 '25 11:04 chiragkyal

@chiragkyal: This pull request references OAPE-148 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set.

In response to this:

Promote CPMSMachineNamePrefix feature gate to default feature set

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-robot avatar Apr 15 '25 14:04 openshift-ci-robot

/test verify-feature-promotion

chiragkyal avatar Apr 15 '25 14:04 chiragkyal

/hold cancel

chiragkyal avatar Apr 15 '25 14:04 chiragkyal

@JoelSpeed I think we have sufficient runs now, verify-feature-promotion is failing due to the metal platform only.

chiragkyal avatar Apr 15 '25 14:04 chiragkyal

/lgtm

/override ci/prow/verify-feature-promotion

This feature does not support metal platforms, all others have passed

JoelSpeed avatar Apr 15 '25 15:04 JoelSpeed

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chiragkyal, JoelSpeed

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[bot] avatar Apr 15 '25 15:04 openshift-ci[bot]

@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/verify-feature-promotion

In response to this:

/lgtm

/override ci/prow/verify-feature-promotion

This feature does not support metal platforms, all others have passed

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.

openshift-ci[bot] avatar Apr 15 '25 15:04 openshift-ci[bot]

/retest-required

chiragkyal avatar Apr 15 '25 15:04 chiragkyal

/retest-required

Remaining retests: 0 against base HEAD b7680e188c5efa5780f93cb89dfa73d728ed5f12 and 2 for PR HEAD cf5f806f84bc587dcd1dc60bb653c5c501aa8cd6 in total

openshift-ci-robot avatar Apr 15 '25 21:04 openshift-ci-robot

/retest-required

Remaining retests: 0 against base HEAD b7680e188c5efa5780f93cb89dfa73d728ed5f12 and 2 for PR HEAD cf5f806f84bc587dcd1dc60bb653c5c501aa8cd6 in total

openshift-ci-robot avatar Apr 16 '25 01:04 openshift-ci-robot

/retest-required

chiragkyal avatar Apr 16 '25 08:04 chiragkyal

e2e-aws-serial-techpreview seems to be unrelated to our changes.

chiragkyal avatar Apr 16 '25 08:04 chiragkyal

/retest-required

Remaining retests: 0 against base HEAD b7680e188c5efa5780f93cb89dfa73d728ed5f12 and 2 for PR HEAD cf5f806f84bc587dcd1dc60bb653c5c501aa8cd6 in total

openshift-ci-robot avatar Apr 16 '25 13:04 openshift-ci-robot

/retest-required

JoelSpeed avatar Apr 16 '25 22:04 JoelSpeed

/retest-required

chiragkyal avatar Apr 17 '25 06:04 chiragkyal

/test e2e-aws-serial-techpreview

chiragkyal avatar Apr 17 '25 07:04 chiragkyal

CI outage is causing a lot of issues. Slack thread

{  failed to wait for importing imagestreamtags [cluster-version-operator, cli] on ci-op-3rg0nbvt/stable: failed to import tag(s) [cli,cluster-version-operator] on image stream ci-op-3rg0nbvt/stable because of missing definition in the spec}

chiragkyal avatar Apr 17 '25 07:04 chiragkyal

/retest

JoelSpeed avatar Apr 17 '25 08:04 JoelSpeed

/retest-required

Remaining retests: 0 against base HEAD b7680e188c5efa5780f93cb89dfa73d728ed5f12 and 2 for PR HEAD cf5f806f84bc587dcd1dc60bb653c5c501aa8cd6 in total

openshift-ci-robot avatar Apr 17 '25 19:04 openshift-ci-robot