Replace hardcoded NS in KUTTL with variable#453
Replace hardcoded NS in KUTTL with variable#453abays wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abays 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 |
|
/test functional |
KUTTL failure does not seem related to this PR's particular KUTTL changes, but might be: Will try again and see what happens. /test designate-operator-build-deploy-kuttl |
DESIGNATE_KUTTL_NAMESPACE is user-configurable in install_yamls (via ?= assignment), but test scripts and asserts had "designate-kuttl-tests" hardcoded in databaseHostname assertions. Tests would fail when run with a non-default namespace.
Replace all hardcoded namespace references with the $NAMESPACE env var that KUTTL provides to test scripts. Specifically, for databaseHostname validation, switch from static YAML asserts to script-based checks that dynamically construct the expected value.
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com