Skip to content

Commit e36a448

Browse files
committed
Add some TODO comments
1 parent 2841336 commit e36a448

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

ci/ansible-tests-fvt.yaml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,8 @@ spec:
4242
#ansibleCollections: 'git+https://github.com/elfiesmelfie/feature-verification-tests.git,test_operator_changes'
4343
ansibleCollections: 'git+https://github.com/elfiesmelfie/feature-verification-tests.git,master'
4444
# allow privilige escalation so ci_setup/tasks/packages can be run, even if we don't install any packages.
45+
# Since the oc install tasks are in the pre-tasks playbook, and don't use the ci_setup role, this can be removed.
46+
# TODO: Consider updating cifmw ci_setup role to have a separate get_oc_client task.
4547
privileged: true
4648
extraMounts:
4749
- extraVol:
@@ -131,7 +133,10 @@ spec:
131133
# openStackConfigSecret <string>
132134
# OpenStackConfigSecret is the name of the Secret containing the secure.yaml
133135
# TODO: add openstack config secrets, then skip running openstack commands in openstackclient. This also means that the version of promclient, observabilityclient and apdh client can be updated without patching the containers.
134-
#
136+
# clouds.yaml is included by default. Need to check the cloudname.
137+
# The OS_CLOUD_NAME can be set in the env to select the cloud.
138+
# clouds.yaml is mounted at /etc/openstack/clouds.yaml
139+
# The cloud is called default.
135140
# privileged <boolean>
136141
# Use with caution! This parameter specifies whether test-operator should
137142
# spawn

0 commit comments

Comments
 (0)