Skip to content

[test_operator] Read horizontest admin password from osp-secret#4067

Open
stuggi wants to merge 1 commit into
openstack-k8s-operators:mainfrom
stuggi:fix-horizontest-hardcoded-password
Open

[test_operator] Read horizontest admin password from osp-secret#4067
stuggi wants to merge 1 commit into
openstack-k8s-operators:mainfrom
stuggi:fix-horizontest-hardcoded-password

Conversation

@stuggi

@stuggi stuggi commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

install_yamls PR openstack-k8s-operators/install_yamls#1158 replaced hardcoded passwords with dynamically generated per-service secrets. The horizontest runner defaulted adminPassword to "12345678" which no longer matches the deployed admin password.

Fetch AdminPassword from the osp-secret Secret at runtime and override the default before building the HorizonTest CR.

Jira: OSPRH-32754

install_yamls PR openstack-k8s-operators/install_yamls#1158 replaced
hardcoded passwords with dynamically generated per-service secrets.
The horizontest runner defaulted adminPassword to "12345678" which no
longer matches the deployed admin password.

Fetch AdminPassword from the osp-secret Secret at runtime and override
the default before building the HorizonTest CR.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Martin Schuppert <mschuppert@redhat.com>

@fmount fmount left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm thanks @stuggi!

stuggi added a commit to stuggi/test-operator that referenced this pull request Jul 17, 2026
Depends-On: openstack-k8s-operators/ci-framework#4067

Signed-off-by: Martin Schuppert <mschuppert@redhat.com>

@abays abays left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@openshift-ci

openshift-ci Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: abays
Once this PR has been reviewed and has the lgtm label, please assign kstrenkova for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/e178d8770ac14d889f1aac09197b45a5

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 56m 48s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 36m 52s
cifmw-crc-podified-edpm-baremetal FAILURE in 53m 36s
cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 1h 32m 46s
✔️ cifmw-multinode-tempest SUCCESS in 2h 04m 08s
✔️ cifmw-pod-zuul-files SUCCESS in 5m 48s
✔️ openstack-k8s-operators-content-provider-bootc SUCCESS in 2h 23m 59s
cifmw-crc-podified-edpm-baremetal-bootc FAILURE in 1h 24m 33s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 51s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 46s
✔️ cifmw-molecule-test_operator SUCCESS in 4m 42s

@stuggi

stuggi commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

recheck

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/531fd06f6a5746ecb487f5c70ee65b0e

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 37m 39s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 31m 18s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 47m 13s
cifmw-crc-podified-edpm-baremetal-minor-update NODE_FAILURE Node(set) request 099-0000146076 failed in 0s
✔️ cifmw-multinode-tempest SUCCESS in 2h 10m 35s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 33s
openstack-k8s-operators-content-provider-bootc TIMED_OUT in 46m 27s
⚠️ cifmw-crc-podified-edpm-baremetal-bootc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider-bootc
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 04s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 22s
✔️ cifmw-molecule-test_operator SUCCESS in 4m 07s

@fmount

fmount commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

recheck

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/2221787436e04a82ae8c02b55dad62a9

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 32m 05s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 28m 15s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 52m 15s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 2h 18m 07s
✔️ cifmw-multinode-tempest SUCCESS in 1h 56m 08s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 50s
✔️ openstack-k8s-operators-content-provider-bootc SUCCESS in 1h 12m 05s
cifmw-crc-podified-edpm-baremetal-bootc FAILURE in 34m 09s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 15s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 23s
✔️ cifmw-molecule-test_operator SUCCESS in 4m 08s

@stuggi

stuggi commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

recheck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants