[DNM] Move placement kuttl tests under default kuttl namespace#1125
[DNM] Move placement kuttl tests under default kuttl namespace#1125amartyasinha wants to merge 1 commit into
Conversation
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: amartyasinha 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 |
Currently, zuul only runs default kuttl tests, thus placement kuttl tests were not getting validated. This commit ensure they are also tested. Signed-off-by: Amartya Sinha <amsinha@redhat.com>
|
Build failed (check pipeline). Post ✔️ openstack-meta-content-provider SUCCESS in 2h 58m 10s |
| kind: Namespace | ||
| metadata: | ||
| name: nova-kuttl-multi-cell | ||
| name: nova-kuttl-default |
There was a problem hiding this comment.
so each test suite is ment ot have its onw namespace
and nova and palcmenet shoudl not share oen.
| serviceUser: placement | ||
| status: | ||
| databaseHostname: openstack.placement-kuttl-tests.svc | ||
| databaseHostname: openstack.nova-kuttl-default.svc |
There was a problem hiding this comment.
so this is incorrect
the placement test are a diffent test quite and shoudl eb using ther own nameapces.
you shoudl be able to run the palcemetn test and the nova tests at the same time without them interfering with each other
There was a problem hiding this comment.
Thanks @SeanMooney! I can try looking how to enable both tests run in zuul instead of consolidating them in single namespace. The reason I was trying to move placement tests under default namespace was because the cyborg tests are also defined under kuttl/test-suites/default. But yeah, I understand the placement tests are different than nova/cyborg.
|
Closing for now. Will check soon how to run both kuttl tests in zuul. |
2 participants
Currently, zuul only runs default kuttl tests, thus placement kuttl tests were not getting validated. This commit ensure they are also tested.