Skip to content

PS-11435: Add auth_openid_connect suite to default MTR runs - #6142

Open
nogueiraanderson wants to merge 1 commit into
percona:trunkfrom
nogueiraanderson:PS-11435-add-auth-openid-connect-default-mtr-trunk
Open

PS-11435: Add auth_openid_connect suite to default MTR runs#6142
nogueiraanderson wants to merge 1 commit into
percona:trunkfrom
nogueiraanderson:PS-11435-add-auth-openid-connect-default-mtr-trunk

Conversation

@nogueiraanderson

@nogueiraanderson nogueiraanderson commented Sep 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Feature

  • The auth_openid_connect MTR suite now runs in default mtr invocations and in the Jenkins parallel MTR pipeline.
  • idp.test and proxy.test now skip on servers built without the plugin, instead of failing on an empty SONAME when the IdP is reachable.

Why

  • The OIDC plugin ships in trunk with an MTR suite that CI never executes, so regressions in OIDC authentication go uncaught.
  • The check_suites CI gate requires DEFAULT_SUITES and the per-worker suite split to change together, so both files move in one PR.

Tickets

@nogueiraanderson nogueiraanderson mentioned this pull request Sep 1, 2026
Merged
@nogueiraanderson
nogueiraanderson force-pushed the PS-11435-add-auth-openid-connect-default-mtr-trunk branch from b87b4df to 123b571 Compare September 1, 2026 08:47
@nogueiraanderson
PS-11435 [trunk] Add auth_openid_connect suite to default MTR runs
2180647 
The suite now runs in default mtr invocations and in the Jenkins
parallel MTR per-worker split.

Also guard the shared set_idp_vars.inc include: on servers built with
-DWITH_AUTH_OPENID_CONNECT=OFF, AUTH_OIDC is empty and idp.test and
proxy.test would run INSTALL PLUGIN with an empty SONAME and fail the
default MTR run instead of skipping.
@nogueiraanderson
nogueiraanderson force-pushed the PS-11435-add-auth-openid-connect-default-mtr-trunk branch from 123b571 to 2180647 Compare September 1, 2026 09:36
inikep
inikep approved these changes Sep 3, 2026
View reviewed changes

@inikep inikep left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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

Reviewers

@inikep inikep inikep approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nogueiraanderson @inikep