Skip to content
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

OCPBUGS-24363,SDN-4403: Downstream merge 20240415 #2124

Merged

Conversation

ricky-rav
Copy link
Contributor

girishmg and others added 13 commits April 11, 2024 08:15
control file is used to communicate with the running daemon to modify
its properties, whereas sock file is AF_UNIX transport to run OVSDB
protocol over

Signed-off-by: Girish Moodalbail <gmoodalbail@nvidia.com>
Bump to k8s 1.29.2 in kind, and bumps to go 1.21

Signed-off-by: Benjamin Pickard <bpickard@redhat.com>
serviceCidr is a new v1alpha feature that we do not support yet, so we
should skip the tests

Signed-off-by: Benjamin Pickard <bpickard@redhat.com>
go mod tidy && go mod vendor && go mod verify

framework.expectEqual() was deprecated in favor of
gomega.Expect(foo).To(gomega.equal(foo))

dropped vendor dir this time

Signed-off-by: Benjamin Pickard <bpickard@redhat.com>
Bumps google.golang.org/protobuf from 1.31.0 to 1.33.0.

---
updated-dependencies:
- dependency-name: google.golang.org/protobuf
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
control file is not same as sock file
…google.golang.org/protobuf-1.33.0

Bump google.golang.org/protobuf from 1.31.0 to 1.33.0 in /test/e2e
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.16.0 to 0.17.0.
- [Commits](golang/crypto@v0.16.0...v0.17.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
…golang.org/x/crypto-0.17.0

Bump golang.org/x/crypto from 0.16.0 to 0.17.0 in /test/e2e
Fix the case for "all endpoints terminating on a node when traffic policy is local":
"When the traffic policy is "Local" and all endpoints are terminating within a single node, then traffic should be routed to any terminating endpoint that is ready on that node."
https://github.com/kubernetes/enhancements/blob/master/keps/sig-network/1669-proxy-terminating-endpoints/README.md#example-all-endpoints-terminating-on-a-node-when-traffic-policy-is-local

The endpoint selection logic in the services controller is now entirely implemented in getEndpointsForService, which computes for a given service and each service port all its cluster-wide endpoints and per-node local endpoints. We first apply a cluster-wide vs local endpoint selection and only then we apply readiness-based filtering with getEligibleEndpointAddresses.

Added unit tests for the new logic.

Signed-off-by: Riccardo Ravaioli <rravaiol@redhat.com>
this test is flaking and needs to be fixed

Signed-off-by: Jacob Tanenbaum <jtanenba@redhat.com>
skip ipv6 'should listen on each host address test'
@ricky-rav ricky-rav requested a review from dcbw as a code owner April 15, 2024 18:31
@openshift-ci openshift-ci bot requested a review from abhat April 15, 2024 18:33
Copy link
Contributor

openshift-ci bot commented Apr 15, 2024

@ricky-rav: 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/e2e-aws-ovn-kubevirt bc38bfd link false /test e2e-aws-ovn-kubevirt
ci/prow/security bc38bfd link false /test security
ci/prow/e2e-metal-ipi-ovn-dualstack-local-gateway bc38bfd link false /test e2e-metal-ipi-ovn-dualstack-local-gateway
ci/prow/e2e-metal-ipi-ovn-ipv6 bc38bfd link true /test e2e-metal-ipi-ovn-ipv6
ci/prow/okd-e2e-gcp-ovn bc38bfd link false /test okd-e2e-gcp-ovn

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/test-infra repository. I understand the commands that are listed here.

@tssurya
Copy link
Contributor

tssurya commented Apr 16, 2024

/retest

@tssurya
Copy link
Contributor

tssurya commented Apr 16, 2024

/assign @tssurya

@tssurya
Copy link
Contributor

tssurya commented Apr 16, 2024

@ricky-rav : do we need to run payload on this one for disruption signals? (I see words services, etp=local and then immediately go payload :) better safe than sorry?)

Copy link
Contributor

openshift-ci bot commented Apr 16, 2024

@ricky-rav: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

1 similar comment
Copy link
Contributor

openshift-ci bot commented Apr 16, 2024

@ricky-rav: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

@ricky-rav
Copy link
Contributor Author

/payload 4.16 ci blocking
/payload 4.16 nightly blocking

Copy link
Contributor

openshift-ci bot commented Apr 16, 2024

@ricky-rav: trigger 5 job(s) of type blocking for the ci release of OCP 4.16

  • periodic-ci-openshift-release-master-ci-4.16-upgrade-from-stable-4.15-e2e-aws-ovn-upgrade
  • periodic-ci-openshift-release-master-ci-4.16-upgrade-from-stable-4.15-e2e-azure-sdn-upgrade
  • periodic-ci-openshift-release-master-ci-4.16-e2e-gcp-ovn-upgrade
  • periodic-ci-openshift-release-master-ci-4.16-e2e-aws-sdn-serial
  • periodic-ci-openshift-hypershift-release-4.16-periodics-e2e-aws-ovn

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/fae2c1a0-fbd3-11ee-9d4b-d9670853cee6-0

trigger 8 job(s) of type blocking for the nightly release of OCP 4.16

  • periodic-ci-openshift-release-master-nightly-4.16-e2e-aws-sdn-upgrade
  • periodic-ci-openshift-release-master-ci-4.16-e2e-azure-ovn-upgrade
  • periodic-ci-openshift-release-master-ci-4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade
  • periodic-ci-openshift-hypershift-release-4.16-periodics-e2e-aws-ovn-conformance
  • periodic-ci-openshift-release-master-nightly-4.16-e2e-aws-ovn-serial
  • periodic-ci-openshift-release-master-ci-4.16-e2e-aws-ovn-upgrade
  • periodic-ci-openshift-release-master-nightly-4.16-e2e-metal-ipi-ovn-ipv6
  • periodic-ci-openshift-release-master-nightly-4.16-e2e-metal-ipi-sdn-bm

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/fae2c1a0-fbd3-11ee-9d4b-d9670853cee6-1

@bpickard22
Copy link
Contributor

/lgtm thanks for taking care of this ricky

@tssurya
Copy link
Contributor

tssurya commented Apr 16, 2024

/approve

@tssurya
Copy link
Contributor

tssurya commented Apr 16, 2024

/tide refresh

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 16, 2024
@tssurya
Copy link
Contributor

tssurya commented Apr 16, 2024

/retitle SDN-4403: Downstream merge 20240415

@tssurya
Copy link
Contributor

tssurya commented Apr 16, 2024

/tide refresh

@tssurya
Copy link
Contributor

tssurya commented Apr 16, 2024

/jira refresh

@openshift-ci-robot
Copy link
Contributor

@tssurya: No Jira issue is referenced in the title of this pull request.
To reference a jira issue, add 'XYZ-NNN:' to the title of this pull request and request another refresh with /jira refresh.

In response to this:

/jira refresh

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 changed the title Downstream merge 20240415 SDN-4403: Downstream merge 20240415 Apr 16, 2024
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Apr 16, 2024
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Apr 16, 2024

@ricky-rav: This pull request references SDN-4403 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.16.0" version, but no target version was set.

In response to this:

@bpickard22 @JacobTanenbaum PTAL

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.

@tssurya
Copy link
Contributor

tssurya commented Apr 16, 2024

/tide refresh

@bpickard22
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 16, 2024
Copy link
Contributor

openshift-ci bot commented Apr 16, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bpickard22, ricky-rav, tssurya

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-merge-bot openshift-merge-bot bot merged commit 50306b3 into openshift:master Apr 16, 2024
30 of 33 checks passed
@ricky-rav ricky-rav changed the title SDN-4403: Downstream merge 20240415 OCPBUGS-24363,SDN-4403: Downstream merge 20240415 Apr 16, 2024
@openshift-ci-robot
Copy link
Contributor

@ricky-rav: Jira Issue OCPBUGS-24363 is in an unrecognized state (Verified) and will not be moved to the MODIFIED state.

In response to this:

@bpickard22 @JacobTanenbaum PTAL

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.

@ricky-rav
Copy link
Contributor Author

/jira refresh

@openshift-ci-robot
Copy link
Contributor

@ricky-rav: Jira Issue OCPBUGS-24363: All pull requests linked via external trackers have merged:

Jira Issue OCPBUGS-24363 has been moved to the MODIFIED state.

In response to this:

/jira refresh

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-bot
Copy link
Contributor

[ART PR BUILD NOTIFIER]

This PR has been included in build ose-ovn-kubernetes-base-container-v4.16.0-202404162215.p0.g50306b3.assembly.stream.el9 for distgit ovn-kubernetes-base.
All builds following this will include this PR.

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-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.

None yet

7 participants