Merge pull request #1074 from deepsm007/fix_validation

openshift-merge-bot[bot] · web-flow · commit 59521538ee6d · 2024-07-24T04:08:32.000Z
IR-471: Adding additional validation
diff --git a/pkg/storage/s3/s3.go b/pkg/storage/s3/s3.go
@@ -419,7 +419,7 @@ func (d *driver) ConfigEnv() (envs envvar.List, err error) {
 		return
 	}
 
-	if currentFeatureGates.Enabled(util.ChunkSizeMiBFeatureGateName) && d.Config.ChunkSizeMiB > 0 {
+	if currentFeatureGates.Enabled(util.ChunkSizeMiBFeatureGateName) && d.Config.ChunkSizeMiB > 4 && d.Config.ChunkSizeMiB < 5121 {
 		chunksize := int64(d.Config.ChunkSizeMiB) * 1024 * 1024
 		envs = append(envs, envvar.EnvVar{Name: "REGISTRY_STORAGE_S3_CHUNKSIZE", Value: chunksize})
 	}

Original file line number	Diff line number	Diff line change
`@@ -419,7 +419,7 @@ func (d *driver) ConfigEnv() (envs envvar.List, err error) {`
`419`	`419`	`return`
`420`	`420`	`}`
`421`	`421`
`422`		`- if currentFeatureGates.Enabled(util.ChunkSizeMiBFeatureGateName) && d.Config.ChunkSizeMiB > 0 {`
	`422`	`+ if currentFeatureGates.Enabled(util.ChunkSizeMiBFeatureGateName) && d.Config.ChunkSizeMiB > 4 && d.Config.ChunkSizeMiB < 5121 {`
`423`	`423`	`chunksize := int64(d.Config.ChunkSizeMiB) * 1024 * 1024`
`424`	`424`	`envs = append(envs, envvar.EnvVar{Name: "REGISTRY_STORAGE_S3_CHUNKSIZE", Value: chunksize})`
`425`	`425`	`}`