Skip to content

Commit

Permalink
Remove remaining container images from test roles
Browse files Browse the repository at this point in the history
The two containerImage parameters removed here are the only ones
remaining in the test roles. For normal usage, this is defaulted in
the operators and we shouldn't need to set it.

For periodic CI, these values are overriden with particular non-latest
container URLs via the kustomization mechanism.

So i think these shouldn't be necessary and can be removed too.

Related:

openstack-k8s-operators#110
openstack-k8s-operators#165
  • Loading branch information
jistr committed Oct 4, 2023
1 parent 677d191 commit 3d755be
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion tests/roles/glance_adoption/tasks/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@
enabled: true
template:
databaseInstance: openstack
containerImage: {{ container_registry|default("quay.io") }}/{{ container_namespace|default("podified-antelope-centos9") }}/openstack-glance-api:{{ container_tag | default("current-podified") }}
customServiceConfig: |
[DEFAULT]
enabled_backends=default_backend:rbd
Expand Down
1 change: 0 additions & 1 deletion tests/roles/ovn_adoption/tasks/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,6 @@
enabled: true
template:
ovnNorthd:
containerImage: {{ container_registry|default("quay.io") }}/{{ container_namespace|default("podified-antelope-centos9") }}/openstack-ovn-northd:{{ container_tag | default("current-podified") }}
networkAttachment: internalapi
replicas: 1
'
Expand Down

0 comments on commit 3d755be

Please sign in to comment.