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
Bump OVN to 22.03 #1033
Bump OVN to 22.03 #1033
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dcbw 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 |
/test images |
2 similar comments
/test images |
/test images |
/retest |
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
libovsdb List API worked with a pointer to a slice of structs, but now it also supports a pointer to a slice of Models, with Models defined as pointers to structs. This is more aligned with the rest of the API and prevents conversions between the two forms. Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
For ACLs: * remove ModelClient usage outside of libovsdbops * refactor libovsdbops to use ModelCLient Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Attempted mutations with nil or empty values are ignored and not sent to the server as they would be an actual noop. Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
Signed-off-by: Tim Rozet <trozet@redhat.com>
Fixes adding nohostsubnets nodes (i.e. windows)
AWS issues:
/test 4.11-upgrade-from-stable-4.10-local-gateway-e2e-aws-ovn-upgrade |
AWS issues:
/test e2e-aws-ovn-local-to-shared-gateway-mode-migration |
Azure issues
/test e2e-azure-ovn |
AWS issues:
/test e2e-ovn-hybrid-step-registry |
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 Do you think that NBDB will start without errors with the updated schema and multiple Copps with duplicate names (""
)?
oops I got the wrong PR |
libovsdb: cleanup ModelClient usage
Requires a bump of the libovsdb modelgen because our gen files use the -extended option which landed in libovsdb in January. Signed-off-by: Dan Williams <dcbw@redhat.com>
The COPP table is now indexed by name (which was actually requested by us/ovn-kubernetes) but this means we need to set the name of previously added COPP rows to prevent database consistency errors. Signed-off-by: Dan Williams <dcbw@redhat.com>
@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. |
@trozet @dceara