Skip to content

Commit ffec86c

Browse files
authored
[tempest] Remove neutronExtraImage parameter (#100)
With the merge of this PR in the ci-framework repo [1] we officialy remove the support for the neutornExtraImage. This parameter can be safely removed now. [1] openstack-k8s-operators/ci-framework#1790 Depends-On: openstack-k8s-operators/ci-framework#1790
1 parent 6315ae6 commit ffec86c

File tree

6 files changed

+0
-34
lines changed

6 files changed

+0
-34
lines changed

api/bases/test.openstack.org_tempests.yaml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -216,10 +216,6 @@ spec:
216216
description: A content of include.txt file that is passed to tempest
217217
via --include-list
218218
type: string
219-
neutronExtraImage:
220-
default: ""
221-
description: Add extra image for Neutron Tempest plugin
222-
type: string
223219
parallel:
224220
default: true
225221
description: Indicate whether tempest should be executed with
@@ -602,9 +598,6 @@ spec:
602598
description: A content of include.txt file that is passed
603599
to tempest via --include-list
604600
type: string
605-
neutronExtraImage:
606-
description: Add extra image for Neutron Tempest plugin
607-
type: string
608601
parallel:
609602
description: Indicate whether tempest should be executed
610603
with --parallel

api/v1beta1/tempest_types.go

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -169,11 +169,6 @@ type TempestRunSpec struct {
169169
// that are part of the external plugin can be executed.
170170
ExternalPlugin []ExternalPluginType `json:"externalPlugin,omitempty"`
171171

172-
// +kubebuilder:validation:Optional
173-
// +kubebuilder:default:=""
174-
// Add extra image for Neutron Tempest plugin
175-
NeutronExtraImage string `json:"neutronExtraImage,omitempty"`
176-
177172
// +kubebuilder:validation:Optional
178173
// Extra images that should be downloaded inside the test pod and uploaded to
179174
// openstack.

api/v1beta1/tempest_types_workflow.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -62,10 +62,6 @@ type WorkflowTempestRunSpec struct {
6262
// Extra images that should be downloaded inside the test pod and uploaded to
6363
// openstack.
6464
ExtraImages *[]ExtraImagesType `json:"extraImagesType"`
65-
66-
// +kubebuilder:validation:Optional
67-
// Add extra image for Neutron Tempest plugin
68-
NeutronExtraImage *string `json:"neutronExtraImage,omitempty"`
6965
}
7066

7167
// TempestconfRunSpec - is used to configure execution of discover-tempest-config

api/v1beta1/zz_generated.deepcopy.go

Lines changed: 0 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/bases/test.openstack.org_tempests.yaml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -216,10 +216,6 @@ spec:
216216
description: A content of include.txt file that is passed to tempest
217217
via --include-list
218218
type: string
219-
neutronExtraImage:
220-
default: ""
221-
description: Add extra image for Neutron Tempest plugin
222-
type: string
223219
parallel:
224220
default: true
225221
description: Indicate whether tempest should be executed with
@@ -602,9 +598,6 @@ spec:
602598
description: A content of include.txt file that is passed
603599
to tempest via --include-list
604600
type: string
605-
neutronExtraImage:
606-
description: Add extra image for Neutron Tempest plugin
607-
type: string
608601
parallel:
609602
description: Indicate whether tempest should be executed
610603
with --parallel

controllers/tempest_controller.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -444,12 +444,6 @@ func (r *TempestReconciler) setTempestConfigVars(envVars map[string]string,
444444

445445
envVars["TEMPEST_WORKFLOW_STEP_DIR_NAME"] = r.GetJobName(instance, workflowStepNum)
446446

447-
// String
448-
value = mergeWithWorkflow(tRun.NeutronExtraImage, wtRun.NeutronExtraImage)
449-
if len(value) != 0 {
450-
envVars["TEMPEST_NEUTRON_IMAGE_URL"] = value
451-
}
452-
453447
extraImages := mergeWithWorkflow(tRun.ExtraImages, wtRun.ExtraImages)
454448
for _, extraImageDict := range extraImages {
455449
envVars["TEMPEST_EXTRA_IMAGES_URL"] += extraImageDict.URL + ","

0 commit comments

Comments
 (0)