[test_operator] Remove redundant HorizonTest parameters - #4071
Conversation
915d841 to
66d6bcb
Compare
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 56m 54s |
|
recheck |
|
/lgtm |
openshift-ci
Bot
added
lgtm
approved
do-not-merge/contains-merge-commits
and removed
lgtm
labels
Parameters adminUsername and adminPassword are being deprecated in the test-operator [1]. Their defaults are also being removed, therefore it is needed to remove them in ci-framework as well. Parameters projectName, user, and password were never fully working in the test-operator, as the values were hardcoded regardless of what was set in the CR. The effort to remove them is in test-operator PR [2], which needs their removal in ci-framework as well. [1] openstack-k8s-operators/test-operator#473 [2] openstack-k8s-operators/test-operator#474 Signed-off-by: Katarina Strenkova <kstrenko@redhat.com>
80071fc to
290da3d
Compare
|
/lgtm |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: imatza-rh, michburk The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/retest |
|
/hold |
|
/hold cancel |
bbfe5bb
into
openstack-k8s-operators:main
4 participants
Parameters adminUsername and adminPassword are being deprecated in the test-operator [1]. Their defaults are also being removed, therefore it is needed to remove them in ci-framework as well.
Parameters projectName, user, and password were never fully working in the test-operator, as the values were hardcoded regardless of what was set in the CR. The effort to remove them is in test-operator PR [2], which needs their removal in ci-framework as well.
[1] openstack-k8s-operators/test-operator#473
[2] openstack-k8s-operators/test-operator#474