diff --git a/go.mod b/go.mod index 5f853a75f..bb81eb9f5 100644 --- a/go.mod +++ b/go.mod @@ -97,3 +97,7 @@ replace github.com/openshift/api => github.com/openshift/api v0.0.0-202304141430 // needed to to cert-manager v1.11.4 see https://github.com/cert-manager/cert-manager/blob/v1.11.4/go.mod#L263C1-L264C104 // remove this once we bump to cert-manager v1.12.x replace github.com/Venafi/vcert/v4 => github.com/jetstack/vcert/v4 v4.9.6-0.20230519122548-219f317ae107 //allow-merging + +replace github.com/openstack-k8s-operators/mariadb-operator/api => github.com/gibizer/mariadb-operator/api v0.0.0-20230927125835-d928d57aebb5 + +replace github.com/openstack-k8s-operators/lib-common/modules/test => github.com/gibizer/lib-common/modules/test v0.0.0-20230928090455-81c3333b6889 diff --git a/go.sum b/go.sum index ed3f95a7a..9bc8f7433 100644 --- a/go.sum +++ b/go.sum @@ -82,6 +82,10 @@ github.com/flowstack/go-jsonschema v0.1.1/go.mod h1:yL7fNggx1o8rm9RlgXv7hTBWxdBM github.com/fsnotify/fsnotify v1.6.0 h1:n+5WquG0fcWoWp6xPWfHdbskMCQaFnG6PfBrh1Ky4HY= github.com/fsnotify/fsnotify v1.6.0/go.mod h1:sl3t1tCWJFWoRz9R8WJCbQihKKwmorjAbSClcnxKAGw= github.com/ghodss/yaml v1.0.0/go.mod h1:4dBDuWmgqj2HViK6kFavaiC9ZROes6MMH2rRYeMEF04= +github.com/gibizer/lib-common/modules/test v0.0.0-20230928090455-81c3333b6889 h1:9mfkwtby6sN7MQUOPK3bNyvMRP/qIpetnQZYgXPeWqQ= +github.com/gibizer/lib-common/modules/test v0.0.0-20230928090455-81c3333b6889/go.mod h1:i+7ijhNyqcPBBzYf4BvnNBzYrYvu9B2BHSEoQOR+FMk= +github.com/gibizer/mariadb-operator/api v0.0.0-20230927125835-d928d57aebb5 h1:Y7jf9PMuPk7omjPH5NINR3vlxdAJwsXqzCbEc6Qpy4k= +github.com/gibizer/mariadb-operator/api v0.0.0-20230927125835-d928d57aebb5/go.mod h1:xhiz5wFdKWwVM7BF/VYon4TT3NuUPXp/Pyn2hWcp0CE= github.com/go-gl/glfw v0.0.0-20190409004039-e6da0acd62b1/go.mod h1:vR7hzQXu2zJy9AVAgeJqvqgH9Q5CA+iKCZ2gyEVpxRU= github.com/go-gl/glfw/v3.3/glfw v0.0.0-20191125211704-12ad95a8df72/go.mod h1:tQ2UAYgL5IevRw8kRxooKSPJfGvJ9fJQFa0TUsXzTg8= github.com/go-gl/glfw/v3.3/glfw v0.0.0-20200222043503-6f7a984d4dc4/go.mod h1:tQ2UAYgL5IevRw8kRxooKSPJfGvJ9fJQFa0TUsXzTg8= @@ -250,10 +254,6 @@ github.com/openstack-k8s-operators/lib-common/modules/openstack v0.1.1-0.2023092 github.com/openstack-k8s-operators/lib-common/modules/openstack v0.1.1-0.20230927082538-4f614f333d17/go.mod h1:+iJZo5alCeOGD/524hWWdlINA6zqY+MjfWT7cDcbvBE= github.com/openstack-k8s-operators/lib-common/modules/storage v0.1.1-0.20230927082538-4f614f333d17 h1:epXz35gpdGp/CbYzAT926971T9kpO6r1v2GX8I1HHGM= github.com/openstack-k8s-operators/lib-common/modules/storage v0.1.1-0.20230927082538-4f614f333d17/go.mod h1:kKFAr7wZw3mX83hlQVbf2hV7TGhNNxVgMSNt9YtUPzI= -github.com/openstack-k8s-operators/lib-common/modules/test v0.1.2-0.20230927082538-4f614f333d17 h1:zJguNin+9IwRnGKy1A7ranxASKO1vTvWxoXwkCz8MWw= -github.com/openstack-k8s-operators/lib-common/modules/test v0.1.2-0.20230927082538-4f614f333d17/go.mod h1:YOFHrNK/QqCvZUPlDJYmDyaCkbKIB98V04uyofiC9a8= -github.com/openstack-k8s-operators/mariadb-operator/api v0.1.1-0.20230927094006-890f79149df4 h1:NVvdjUKCVdwf/rboYM7mqZaBX7g65q1yw7tOJTxfT7g= -github.com/openstack-k8s-operators/mariadb-operator/api v0.1.1-0.20230927094006-890f79149df4/go.mod h1:xXHF/R/L0XamRHR/UkzlgzSTocBQ6GSQ2U16Q9Mf/bA= github.com/pkg/errors v0.8.0/go.mod h1:bwawxfHBFNV+L2hUp1rHADufV3IMtnDRdf1r5NINEl0= github.com/pkg/errors v0.8.1/go.mod h1:bwawxfHBFNV+L2hUp1rHADufV3IMtnDRdf1r5NINEl0= github.com/pkg/errors v0.9.1 h1:FEBLx1zS214owpjy7qsBeixbURkuhQAwrK5UwLGTwt4= diff --git a/test/functional/glance_controller_test.go b/test/functional/glance_controller_test.go index ec34816e7..57a6dd691 100644 --- a/test/functional/glance_controller_test.go +++ b/test/functional/glance_controller_test.go @@ -125,8 +125,8 @@ var _ = Describe("Glance controller", func() { BeforeEach(func() { DeferCleanup(th.DeleteInstance, CreateGlance(glanceTest.Instance, GetGlanceDefaultSpec())) DeferCleanup( - th.DeleteDBService, - th.CreateDBService( + mariadb.DeleteDBService, + mariadb.CreateDBService( glanceName.Namespace, GetGlance(glanceTest.Instance).Spec.DatabaseInstance, corev1.ServiceSpec{ @@ -136,7 +136,7 @@ var _ = Describe("Glance controller", func() { ) }) It("Should set DBReady Condition and set DatabaseHostname Status when DB is Created", func() { - th.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) + mariadb.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) th.SimulateJobSuccess(glanceTest.GlanceDBSync) Glance := GetGlance(glanceTest.Instance) Expect(Glance.Status.DatabaseHostname).To(Equal("hostname-for-openstack")) @@ -155,7 +155,7 @@ var _ = Describe("Glance controller", func() { }) It("Should fail if db-sync job fails when DB is Created", func() { - th.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) + mariadb.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) th.SimulateJobFailure(glanceTest.GlanceDBSync) th.ExpectCondition( glanceTest.Instance, @@ -179,8 +179,8 @@ var _ = Describe("Glance controller", func() { BeforeEach(func() { DeferCleanup(th.DeleteInstance, CreateGlance(glanceTest.Instance, GetGlanceDefaultSpec())) DeferCleanup( - th.DeleteDBService, - th.CreateDBService( + mariadb.DeleteDBService, + mariadb.CreateDBService( glanceName.Namespace, GetGlance(glanceTest.Instance).Spec.DatabaseInstance, corev1.ServiceSpec{ @@ -188,7 +188,7 @@ var _ = Describe("Glance controller", func() { }, ), ) - th.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) + mariadb.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) th.SimulateJobSuccess(glanceTest.GlanceDBSync) keystoneAPI := keystone.CreateKeystoneAPI(glanceTest.Instance.Namespace) DeferCleanup(keystone.DeleteKeystoneAPI, keystoneAPI) @@ -230,8 +230,8 @@ var _ = Describe("Glance controller", func() { DeferCleanup(th.DeleteInstance, CreateGlance(glanceTest.Instance, GetGlanceDefaultSpec())) // Get Default GlanceAPI (internal/external) DeferCleanup( - th.DeleteDBService, - th.CreateDBService( + mariadb.DeleteDBService, + mariadb.CreateDBService( glanceTest.Instance.Namespace, GetGlance(glanceName).Spec.DatabaseInstance, corev1.ServiceSpec{ @@ -240,7 +240,7 @@ var _ = Describe("Glance controller", func() { ), ) DeferCleanup(keystone.DeleteKeystoneAPI, keystone.CreateKeystoneAPI(glanceTest.Instance.Namespace)) - th.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) + mariadb.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) th.SimulateJobSuccess(glanceTest.GlanceDBSync) keystone.SimulateKeystoneServiceReady(glanceTest.Instance) keystone.SimulateKeystoneEndpointReady(glanceTest.GlanceInternalRoute) @@ -259,8 +259,8 @@ var _ = Describe("Glance controller", func() { DeferCleanup(th.DeleteInstance, CreateGlance(glanceTest.Instance, GetGlanceDefaultSpec())) // Get Default GlanceAPI (internal/external) DeferCleanup( - th.DeleteDBService, - th.CreateDBService( + mariadb.DeleteDBService, + mariadb.CreateDBService( glanceTest.Instance.Namespace, GetGlance(glanceName).Spec.DatabaseInstance, corev1.ServiceSpec{ @@ -269,13 +269,13 @@ var _ = Describe("Glance controller", func() { ), ) DeferCleanup(keystone.DeleteKeystoneAPI, keystone.CreateKeystoneAPI(glanceTest.Instance.Namespace)) - th.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) + mariadb.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) th.SimulateJobSuccess(glanceTest.GlanceDBSync) keystone.SimulateKeystoneServiceReady(glanceTest.Instance) keystone.SimulateKeystoneEndpointReady(glanceTest.GlanceInternalRoute) }) It("removes the finalizers from the Glance DB", func() { - mDB := th.GetMariaDBDatabase(glanceTest.Instance) + mDB := mariadb.GetMariaDBDatabase(glanceTest.Instance) Expect(mDB.Finalizers).To(ContainElement("Glance")) th.DeleteInstance(GetGlance(glanceTest.Instance)) @@ -309,8 +309,8 @@ var _ = Describe("Glance controller", func() { } DeferCleanup(th.DeleteInstance, CreateGlance(glanceTest.Instance, rawSpec)) DeferCleanup( - th.DeleteDBService, - th.CreateDBService( + mariadb.DeleteDBService, + mariadb.CreateDBService( glanceTest.Instance.Namespace, GetGlance(glanceName).Spec.DatabaseInstance, corev1.ServiceSpec{ @@ -318,7 +318,7 @@ var _ = Describe("Glance controller", func() { }, ), ) - th.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) + mariadb.SimulateMariaDBDatabaseCompleted(glanceTest.Instance) th.SimulateJobSuccess(glanceTest.GlanceDBSync) keystoneAPI := keystone.CreateKeystoneAPI(glanceTest.Instance.Namespace) DeferCleanup(keystone.DeleteKeystoneAPI, keystoneAPI) diff --git a/test/functional/suite_test.go b/test/functional/suite_test.go index 066a4e226..47bcb670d 100644 --- a/test/functional/suite_test.go +++ b/test/functional/suite_test.go @@ -34,6 +34,7 @@ import ( . "github.com/onsi/gomega" keystone_test "github.com/openstack-k8s-operators/keystone-operator/api/test/helpers" common_test "github.com/openstack-k8s-operators/lib-common/modules/test/helpers" + mariadb_test "github.com/openstack-k8s-operators/mariadb-operator/api/test/helpers" "k8s.io/client-go/kubernetes" "k8s.io/client-go/kubernetes/scheme" @@ -70,6 +71,7 @@ var ( testEnv *envtest.Environment th *common_test.TestHelper keystone *keystone_test.TestHelper + mariadb *mariadb_test.TestHelper ctx context.Context cancel context.CancelFunc logger logr.Logger @@ -150,7 +152,9 @@ var _ = BeforeSuite(func() { th = common_test.NewTestHelper(ctx, k8sClient, timeout, interval, logger) Expect(th).NotTo(BeNil()) keystone = keystone_test.NewTestHelper(ctx, k8sClient, timeout, interval, logger) - Expect(th).NotTo(BeNil()) + Expect(keystone).NotTo(BeNil()) + mariadb = mariadb_test.NewTestHelper(ctx, k8sClient, timeout, interval, logger) + Expect(mariadb).NotTo(BeNil()) // Start the controller-manager in a goroutine webhookInstallOptions := &testEnv.WebhookInstallOptions