Skip to content

Commit f698e92

Browse files
Fix configurable InsecureImportPolicy in all ImageStreams on operator side
1 parent eafd237 commit f698e92

File tree

2 files changed

+9
-15
lines changed

2 files changed

+9
-15
lines changed

pkg/3scale/amp/component/ampimages.go

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,7 @@ func (ampImages *AmpImages) buildAmpBackendImageStream() *imagev1.ImageStream {
8181
Name: ampImages.Options.backendImage,
8282
},
8383
ImportPolicy: imagev1.TagImportPolicy{
84-
// TODO this was originally a double brace expansion from a variable, that is not possible
85-
// natively with kubernetes so we replaced it with a const
86-
Insecure: InsecureImportPolicy,
84+
Insecure: ampImages.Options.insecureImportPolicy,
8785
},
8886
},
8987
},
@@ -126,9 +124,7 @@ func (ampImages *AmpImages) buildAmpZyncImageStream() *imagev1.ImageStream {
126124
Name: ampImages.Options.zyncImage,
127125
},
128126
ImportPolicy: imagev1.TagImportPolicy{
129-
// TODO this was originally a double brace expansion from a variable, that is not possible
130-
// natively with kubernetes so we replaced it with a const
131-
Insecure: InsecureImportPolicy,
127+
Insecure: ampImages.Options.insecureImportPolicy,
132128
},
133129
},
134130
},
@@ -171,9 +167,7 @@ func (ampImages *AmpImages) buildApicastImageStream() *imagev1.ImageStream {
171167
Name: ampImages.Options.apicastImage,
172168
},
173169
ImportPolicy: imagev1.TagImportPolicy{
174-
// TODO this was originally a double brace expansion from a variable, that is not possible
175-
// natively with kubernetes so we replaced it with a const
176-
Insecure: InsecureImportPolicy,
170+
Insecure: ampImages.Options.insecureImportPolicy,
177171
},
178172
},
179173
},
@@ -216,7 +210,7 @@ func (ampImages *AmpImages) buildAmpSystemImageStream() *imagev1.ImageStream {
216210
Name: ampImages.Options.systemImage,
217211
},
218212
ImportPolicy: imagev1.TagImportPolicy{
219-
Insecure: InsecureImportPolicy,
213+
Insecure: ampImages.Options.insecureImportPolicy,
220214
},
221215
},
222216
},
@@ -259,7 +253,7 @@ func (ampImages *AmpImages) buildZyncDatabasePostgreSQLImageStream() *imagev1.Im
259253
Name: ampImages.Options.ZyncDatabasePostgreSQLImage,
260254
},
261255
ImportPolicy: imagev1.TagImportPolicy{
262-
Insecure: InsecureImportPolicy,
256+
Insecure: ampImages.Options.insecureImportPolicy,
263257
},
264258
},
265259
},
@@ -302,7 +296,7 @@ func (ampImages *AmpImages) buildBackendRedisImageStream() *imagev1.ImageStream
302296
Name: ampImages.Options.backendRedisImage,
303297
},
304298
ImportPolicy: imagev1.TagImportPolicy{
305-
Insecure: InsecureImportPolicy,
299+
Insecure: ampImages.Options.insecureImportPolicy,
306300
},
307301
},
308302
},
@@ -345,7 +339,7 @@ func (ampImages *AmpImages) buildSystemRedisImageStream() *imagev1.ImageStream {
345339
Name: ampImages.Options.systemRedisImage,
346340
},
347341
ImportPolicy: imagev1.TagImportPolicy{
348-
Insecure: InsecureImportPolicy,
342+
Insecure: ampImages.Options.insecureImportPolicy,
349343
},
350344
},
351345
},
@@ -388,7 +382,7 @@ func (ampImages *AmpImages) buildSystemMemcachedImageStream() *imagev1.ImageStre
388382
Name: ampImages.Options.systemMemcachedImage,
389383
},
390384
ImportPolicy: imagev1.TagImportPolicy{
391-
Insecure: InsecureImportPolicy,
385+
Insecure: ampImages.Options.insecureImportPolicy,
392386
},
393387
},
394388
},

pkg/3scale/amp/component/system_mysql_image.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ func (s *SystemMySQLImage) buildSystemMySQLImageStream() *imagev1.ImageStream {
6161
Name: s.Options.image,
6262
},
6363
ImportPolicy: imagev1.TagImportPolicy{
64-
Insecure: InsecureImportPolicy,
64+
Insecure: s.Options.insecureImportPolicy,
6565
},
6666
},
6767
},

0 commit comments

Comments
 (0)