STONEBLD-1735 default to private repositories

redhat-appstudio · Sep 14, 2023 · 618011a · 618011a
1 parent 351c65d
commit 618011a
Show file tree

Hide file tree

Showing 6 changed files with 63 additions and 17 deletions.
diff --git a/deploy/crds/base/jvmbuildservice.io_jbsconfigs.yaml b/deploy/crds/base/jvmbuildservice.io_jbsconfigs.yaml
@@ -115,6 +115,10 @@ spec:
                     type: string
                   prependTag:
                     type: string
+                  public:
+                    description: if this is true and we are automatically creating
+                      registries then we will make it public
+                    type: boolean
                   repository:
                     type: string
                   secretName:

diff --git a/java-components/resource-model/src/main/resources/crds/jvmbuildservice.io_jbsconfigs.yaml b/java-components/resource-model/src/main/resources/crds/jvmbuildservice.io_jbsconfigs.yaml
@@ -115,6 +115,10 @@ spec:
                     type: string
                   prependTag:
                     type: string
+                  public:
+                    description: if this is true and we are automatically creating
+                      registries then we will make it public
+                    type: boolean
                   repository:
                     type: string
                   secretName:

diff --git a/openshift-with-appstudio-test/e2e/util.go b/openshift-with-appstudio-test/e2e/util.go
@@ -295,11 +295,13 @@ func setupConfig(t *testing.T, namespace string, hermetic bool) *testArgs {
 				WorkerThreads: "100",
 				RequestCPU:    "10m",
 			},
-			Registry: v1alpha1.ImageRegistry{
-				Host:       "quay.io",
-				Owner:      owner,
-				Repository: "test-images",
-				PrependTag: strconv.FormatInt(time.Now().UnixMilli(), 10),
+			Registry: v1alpha1.ImageRegistrySpec{
+				ImageRegistry: v1alpha1.ImageRegistry{
+					Host:       "quay.io",
+					Owner:      owner,
+					Repository: "test-images",
+					PrependTag: strconv.FormatInt(time.Now().UnixMilli(), 10),
+				},
 			},
 			RelocationPatterns: []v1alpha1.RelocationPatternElement{
 				{
@@ -878,13 +880,15 @@ func setupMinikube(t *testing.T, namespace string) *testArgs {
 				DisableTLS:    true,
 				Storage:       "756Mi",
 			},
-			Registry: v1alpha1.ImageRegistry{
-				Host:       devIp,
-				Owner:      owner,
-				Repository: "test-images",
-				Port:       "5000",
-				Insecure:   true,
-				PrependTag: strconv.FormatInt(time.Now().UnixMilli(), 10),
+			Registry: v1alpha1.ImageRegistrySpec{
+				ImageRegistry: v1alpha1.ImageRegistry{
+					Host:       devIp,
+					Owner:      owner,
+					Repository: "test-images",
+					Port:       "5000",
+					Insecure:   true,
+					PrependTag: strconv.FormatInt(time.Now().UnixMilli(), 10),
+				},
 			},
 		},
 		Status: v1alpha1.JBSConfigStatus{},

diff --git a/pkg/apis/jvmbuildservice/v1alpha1/jbsconfig_types.go b/pkg/apis/jvmbuildservice/v1alpha1/jbsconfig_types.go
@@ -39,15 +39,23 @@ type JBSConfigSpec struct {
 
 	MavenBaseLocations map[string]string `json:"mavenBaseLocations,omitempty"`
 
-	SharedRegistries []ImageRegistry `json:"sharedRegistries,omitempty"`
-	Registry         ImageRegistry   `json:"registry,omitempty"`
+	SharedRegistries []ImageRegistry   `json:"sharedRegistries,omitempty"`
+	Registry         ImageRegistrySpec `json:"registry,omitempty"`
+
 	// Deprecated: Replaced by explicit declaration of Registry above.
 	ImageRegistry      `json:",inline,omitempty"`
 	CacheSettings      CacheSettings              `json:"cacheSettings,omitempty"`
 	BuildSettings      BuildSettings              `json:"buildSettings,omitempty"`
 	RelocationPatterns []RelocationPatternElement `json:"relocationPatterns,omitempty"`
 }
 
+type ImageRegistrySpec struct {
+	ImageRegistry `json:",inline,omitempty"`
+
+	//if this is true and we are automatically creating registries then we will make it public
+	Public *bool `json:"public,omitempty"`
+}
+
 type JBSConfigStatus struct {
 	Message          string         `json:"message,omitempty"`
 	ImageRegistry    *ImageRegistry `json:"imageRegistry,omitempty"`
@@ -122,7 +130,7 @@ type JBSConfig struct {
 }
 
 func (in *JBSConfig) ImageRegistry() ImageRegistry {
-	ret := in.Spec.Registry
+	ret := in.Spec.Registry.ImageRegistry
 	if ret.Host == "" {
 		ret.Host = "quay.io"
 	}

diff --git a/pkg/apis/jvmbuildservice/v1alpha1/zz_generated.deepcopy.go b/pkg/apis/jvmbuildservice/v1alpha1/zz_generated.deepcopy.go
diff --git a/pkg/reconciler/jbsconfig/jbsconfig.go b/pkg/reconciler/jbsconfig/jbsconfig.go
@@ -595,13 +595,17 @@ func (r *ReconcilerJBSConfig) cacheDeployment(ctx context.Context, log logr.Logg
 
 func (r *ReconcilerJBSConfig) handleNoOwnerSpecified(ctx context.Context, log logr.Logger, config *v1alpha1.JBSConfig) error {
 
+	vis := imagecontroller.ImageVisibilityPrivate
+	if config.Spec.Registry.Public != nil && *config.Spec.Registry.Public {
+		vis = imagecontroller.ImageVisibilityPublic
+	}
 	repo := imagecontroller.ImageRepository{}
 	err := r.client.Get(ctx, types.NamespacedName{Namespace: config.Namespace, Name: v1alpha1.DefaultImageSecretName}, &repo)
 	if err != nil {
 		if errors.IsNotFound(err) {
 			repo.Name = v1alpha1.DefaultImageSecretName
 			repo.Namespace = config.Namespace
-			repo.Spec.Image.Visibility = imagecontroller.ImageVisibilityPublic
+			repo.Spec.Image.Visibility = vis
 			err := controllerutil.SetOwnerReference(config, &repo, r.scheme)
 			if err != nil {
 				return err