Configure correctly for old versions

Azure · Nov 21, 2023 · b88e716 · b88e716
1 parent c59e128
commit b88e716
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/internal/controllers/reconciliation/integration_test.go b/internal/controllers/reconciliation/integration_test.go
@@ -80,7 +80,7 @@ func TestControllerPodBasics(t *testing.T) {
 		return []*apiv1.ResourceSlice{slice}
 	})
 
-	require.NoError(t, New(rm, mgr.DownstreamRestConfig, 5, true))
+	require.NoError(t, New(rm, mgr.DownstreamRestConfig, 5, testutil.AtLeastVersion(t, 15)))
 	mgr.Start(t)
 
 	syn := &apiv1.Synthesizer{}
@@ -181,7 +181,7 @@ func TestControllerCRBasics(t *testing.T) {
 		return []*apiv1.ResourceSlice{slice}
 	})
 
-	require.NoError(t, New(rm, mgr.DownstreamRestConfig, 5, true))
+	require.NoError(t, New(rm, mgr.DownstreamRestConfig, 5, testutil.AtLeastVersion(t, 15)))
 	mgr.Start(t)
 
 	syn := &apiv1.Synthesizer{}

diff --git a/internal/testutil/testutil.go b/internal/testutil/testutil.go
@@ -285,3 +285,13 @@ func NewPodController(t testing.TB, mgr ctrl.Manager, fn func(*apiv1.Composition
 		Build(podCtrl)
 	require.NoError(t, err)
 }
+
+func AtLeastVersion(t *testing.T, minor int) bool {
+	versionStr := os.Getenv("DOWNSTREAM_VERSION_MINOR")
+	if versionStr == "" {
+		return true // fail open for local dev
+	}
+
+	version, _ := strconv.Atoi(versionStr)
+	return version >= minor
+}