Skip to content

[DPE-9030] feat: add roles-mapping config option#441

Open
imanenami wants to merge 13 commits intomainfrom
dpe-9030
Open

[DPE-9030] feat: add roles-mapping config option#441
imanenami wants to merge 13 commits intomainfrom
dpe-9030

Conversation

@imanenami
Copy link
Copy Markdown
Contributor

@imanenami imanenami commented Dec 10, 2025

Description:

  • Added roles-mapping config option, compliant with opensearch spec. and implementation.
  • Added E2E OAuth client tests involving both CLI & kafka-python clients.

@imanenami imanenami marked this pull request as ready for review December 10, 2025 13:33
zmraul
zmraul previously approved these changes Dec 12, 2025
Copy link
Copy Markdown
Contributor

@zmraul zmraul left a comment

Choose a reason for hiding this comment

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

Nice! Besides from the parsing validation, looks good

Comment thread config.yaml
Comment thread src/core/structured_config.py
Comment thread src/core/structured_config.py
marcoppenheimer
marcoppenheimer previously approved these changes Feb 3, 2026
@imanenami imanenami force-pushed the dpe-8940 branch 2 times, most recently from ce96320 to 476f83e Compare February 4, 2026 05:43
Base automatically changed from dpe-8940 to main February 4, 2026 14:34
@imanenami imanenami dismissed stale reviews from marcoppenheimer and zmraul February 4, 2026 14:34

The base branch was changed.

@marcoppenheimer
Copy link
Copy Markdown
Contributor

Bump @imanenami

Signed-off-by: Iman Enami <44609233+imanenami@users.noreply.github.com>
Comment thread src/events/provider.py Dismissed
Comment thread tests/integration/helpers/oauth.py Dismissed
@imanenami
Copy link
Copy Markdown
Contributor Author

imanenami commented Feb 17, 2026

Nothing changed here, just rebase and conflict resolution

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants