-
Notifications
You must be signed in to change notification settings - Fork 136
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
Bug 1999852: bump OVN to ovn21.09-21.09.0-18.el8fdp #704
Bug 1999852: bump OVN to ovn21.09-21.09.0-18.el8fdp #704
Conversation
@dcbw: This pull request references Bugzilla bug 1999852, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (anusaxen@redhat.com), skipping review request. 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/test-infra repository. |
@dcbw: An error was encountered querying GitHub for users with public email (anusaxen@redhat.com) for bug 1999852 on the Bugzilla server at https://bugzilla.redhat.com. No known errors were detected, please see the full error message for details. Full error message.
non-200 OK status code: 403 Forbidden body: "{\n \"documentation_url\": \"https://docs.github.com/en/free-pro-team@latest/rest/overview/resources-in-the-rest-api#secondary-rate-limits\",\n \"message\": \"You have exceeded a secondary rate limit. Please wait a few minutes before you try again.\"\n}\n"
Please contact an administrator to resolve this issue, then request a bug refresh with 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/test-infra repository. |
/retest |
5 similar comments
/retest |
/retest |
/retest |
/retest |
/retest |
/hold |
we do not want the revert for zero snat @flavio-fernandes |
@dceara FYI |
Correct. Let's undo the (partially?) |
@flavio-fernandes I'm not sure it's that straightforward. We need to see how to do it without reintroducing https://bugzilla.redhat.com/show_bug.cgi?id=1992705 I'll try to prepare an u/s OVN patch soon. |
@numansiddique FYI |
@dcbw: This pull request references Bugzilla bug 1999852, which is valid. 3 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (dbrahane@redhat.com), skipping review request. 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/test-infra repository. |
aeb1b63
to
4b0994f
Compare
@@ -33,7 +33,7 @@ RUN yum install -y \ | |||
yum clean all | |||
|
|||
ARG ovsver=2.15.0-28.el8fdp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dcbw won't we also need a new OVS to satisfy the call to the recently introduced 'ovsrec_server_has_datapath_table' symbol?
Ref: https://patchwork.ozlabs.org/project/ovn/patch/20210901191304.532.97471.stgit@dceara.remote.csb/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@flavio-fernandes Not really, that's just at compile time and the OVN rpm was built with the correct OVS libs. OVN links statically with OVS libs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah. TY. Learning something new every day! ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how to do that?
i do like this ./configure --with-ovs-source=/root/ovs/openvswitch-2.16.0 --with-ovs-build= /root/ovs/openvswitch-2.6.0
but still has error: undefined reference ""ovsrec_server_has_datapath_table
Includes fixes for: optimizations to loadbalancer logical flow creation in northd (reduced latency and iteration time) optimized hashing in northd (reduced latency and iteration time) additional northd stopwatches for debugging iteration times two northd perf optimizations to avoid multiple identical datapath groups optimization to JSON serialization improving all inter-component communication
/hold cancel |
4b0994f
to
8bda565
Compare
@dcbw: This pull request references Bugzilla bug 1999852, which is valid. 3 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (dbrahane@redhat.com), skipping review request. 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/test-infra repository. |
/retest |
3 similar comments
/retest |
/retest |
/retest |
/override ci/prow/e2e-gcp-ovn |
@abhat: Overrode contexts on behalf of abhat: ci/prow/e2e-gcp-ovn 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/test-infra repository. |
/retest |
@dcbw: The following tests 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/test-infra repository. I understand the commands that are listed here. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dcbw, mccv1r0 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 |
@dcbw: All pull requests linked via external trackers have merged: Bugzilla bug 1999852 has been moved to the MODIFIED state. 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/test-infra repository. |
Includes fixes for: