Skip to content

Commit

Permalink
ci: ovn-kubernetes: Pin ovn-k8s version.
Browse files Browse the repository at this point in the history
Recently, ovn-k8s started using the new OVN Northbound database table
Chassis_Template_Var that was added in v22.12.0.  Unfortunately,
ovn-k8s (libovsdb) became incompatible with OVN 22.09 as it doesn't
support connecting to servers that don't contain this table in their
schema.

It's still important to check that we don't introduce regressions on
stable branches so, instead of completely removing the ovn-k8s CI jobs,
we now pin ovn-k8s to a version that's known to work with OVN 22.09.

NOTE: This is not really a permanent solution, it's just a way to
unblock and still run some of the ovn-k8s CI on branches older than
22.09.  The ideal solution would be to ensure that ovn-k8s can work with
older OVN versions (maybe it's enough to ensure compatibility with LTS).
Until then we will have to keep pinning ovn-kubernetes on stable
branches every time a new table is used by the CMS.

Signed-off-by: Dumitru Ceara <dceara@redhat.com>
Acked-by: Ilya Maximets <i.maximets@ovn.org>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
(cherry picked from commit e16d415)
  • Loading branch information
dceara committed Apr 11, 2023
1 parent df6d579 commit 55e08b6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .ci/ovn-kubernetes/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
ARG OVNKUBE_COMMIT=master
ARG OVNKUBE_COMMIT=b79d290a7ee7
ARG LIBOVSDB_COMMIT=a6a173993830

FROM fedora:35 AS ovnbuilder
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ovn-kubernetes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ on:
env:
GO_VERSION: "1.18.4"
K8S_VERSION: v1.24.0
OVNKUBE_COMMIT: "master"
OVNKUBE_COMMIT: "b79d290a7ee7"
LIBOVSDB_COMMIT: "a6a173993830"
KIND_CLUSTER_NAME: ovn
KIND_INSTALL_INGRESS: true
Expand Down

0 comments on commit 55e08b6

Please sign in to comment.