Skip to content

[adoption_osp_deploy] Add enable_keystone_ldap setting in adoption role#3973

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
jagee:add-adoption-ldap
Jun 16, 2026
Merged

[adoption_osp_deploy] Add enable_keystone_ldap setting in adoption role#3973
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
jagee:add-adoption-ldap

Conversation

@jagee

@jagee jagee commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

This will work with the following data plane adoption patch to add ldap users to keystone in uni03gamma deploys.

openstack-k8s-operators/data-plane-adoption#1386

evallesp
evallesp approved these changes Jun 3, 2026
View reviewed changes

@evallesp evallesp left a comment •
edited
Loading

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
Ci should be fixed by starting the commit with: "[adoption_osp_deploy]"

@openshift-ci openshift-ci Bot added the lgtm label Jun 3, 2026
@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Jun 4, 2026

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/99dd499fbed3478a825eb2db00f30217

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 45m 12s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 23m 51s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 32m 29s
❌ cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 32m 00s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 39s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 50s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 15s
✔️ cifmw-molecule-adoption_osp_deploy SUCCESS in 4m 20s

@jagee jagee force-pushed the add-adoption-ldap branch from 27e70b9 to bd8413b Compare June 4, 2026 20:55
@openshift-ci openshift-ci Bot removed the lgtm label Jun 4, 2026
@jagee jagee force-pushed the add-adoption-ldap branch from bd8413b to c97466e Compare June 8, 2026 21:02
@jagee
[adoption_osp_deploy] Add enable_keystone_ldap setting in adoption role
76eb2aa 
This will work with the following data plane adoption patch to add
ldap users to keystone in uni03gamma deploys.

openstack-k8s-operators/data-plane-adoption#1386
Signed-off-by: Jeremy Agee <jagee@redhat.com>
@jagee jagee force-pushed the add-adoption-ldap branch from c97466e to 76eb2aa Compare June 10, 2026 16:51
@jagee jagee changed the title Add enable keystone with ldap setting in adoption role [adoption_osp_deploy] Add enable_keystone_ldap setting in adoption role Jun 10, 2026
michburk
michburk approved these changes Jun 15, 2026
View reviewed changes

@michburk michburk 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.

/approve

@openshift-ci

openshift-ci Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: michburk

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

The pull request process is described 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

evallesp
evallesp approved these changes Jun 16, 2026
View reviewed changes

@evallesp evallesp 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

@openshift-ci openshift-ci Bot added the lgtm label Jun 16, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit 2c18e34 into openstack-k8s-operators:main Jun 16, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@evallesp evallesp evallesp approved these changes

@michburk michburk michburk approved these changes

Assignees

@evallesp evallesp

Labels

approved lgtm Ready For Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jagee @evallesp @michburk