-
Notifications
You must be signed in to change notification settings - Fork 562
NE-2009: Re-enable GatewayAPI featuregate in Default featureset #2281
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
NE-2009: Re-enable GatewayAPI featuregate in Default featureset #2281
Conversation
…ote-ga" This reverts commit 088accf.
@alebedev87: This pull request references NE-2009 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: 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. |
Hello @alebedev87! Some important instructions when contributing to openshift/api: |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alebedev87, 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 |
@alebedev87: This pull request references NE-2009 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:
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. |
Failed HyperShift conformance test: link. Error from the logs:
PR to add a dedicated VAP for HyperShift: openshift/cluster-ingress-operator#1221. |
/retest |
/retest-required |
1 similar comment
/retest-required |
/tide refresh |
/test e2e-upgrade-out-of-change |
/test e2e-aws-ovn-hypershift |
/test e2e-aws-serial-techpreview |
Requested Core Networking team look at https://issues.redhat.com/browse/OCPBUGS-53279 for the e2e-upgrade-out-of-change issue, which is consistently having network issues that impact etcd and seem caused by the error |
/retest-required |
As a matter of fact all the e2e tests passed:
However some CI job steps fail due to CI image registry problems:
|
/test e2e-aws-serial-techpreview |
/retest |
Still seem to be seeing payload building failures in CI here :/ |
Yes, the appci incident is ongoing. |
/test e2e-aws-ovn-hypershift-conformance |
Nice try but no, the CI is still missing some images:
|
Can't hurt to try. |
/retest-required |
Let's wait to try re-running e2e-aws-ovn-hypershift until after openshift/origin#29683 merges. The hypershift tests were failing due to crashing pods:
cc @JoelSpeed |
But it was running too long (>4h) which seems to be the case for some other runs. An increase of the timeout needs to be considered. |
/retest-required
|
/unhold The hypershift conformance is green as requested by Joel. |
/test e2e-aws-serial-techpreview (install failed) |
install failed due to quota limit exceeded
|
|
AWS IAM role quota issue was reported in the test-platform forum: Slack thread. |
/retest-required |
@alebedev87: The following test 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. |
/override ci/prow/e2e-aws-serial-techpreview This change should not effect tech preview as the feature is already available in tech preview. Seems there are permanent errors with infrastructure provisioning at the moment. /override ci/prow/e2e-upgrade-out-of-change This has previously passed, but is now facing infrastructure issues. Since this was already promoted, and then reverted, I'm fairly confident that this will be safe to override |
@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/e2e-aws-serial-techpreview, ci/prow/e2e-upgrade-out-of-change 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. |
f636181
into
openshift:master
[ART PR BUILD NOTIFIER] Distgit: ose-cluster-config-api |
This PR re-applies #2261 which was reverted #2277.
PR which fixed the missing
GatewayAPI
featuregate inDefault
on Hypershift: openshift/hypershift#6035.PR which added HyperShift conformance tests to the api presubmites: openshift/release#63810.