Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CELEBORN-1748] Deprecate identity provider configs tied with quota #2952

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

s0nskar
Copy link
Contributor

@s0nskar s0nskar commented Nov 26, 2024

What changes were proposed in this pull request?

Deprecate identity configs related with quota –

        "celeborn.quota.identity.provider"
        "celeborn.quota.identity.user-specific.tenant"
        "celeborn.quota.identity.user-specific.userName"

In favour of identity configs independent of quota

        "celeborn.identity.provider"
        "celeborn.identity.user-specific.tenant"
        "celeborn.identity.user-specific.userName"

Why are the changes needed?

Current identity configs are tied with quota but identity should be free of quota because other pieces like tags are also using it. In future other new components can also make use of identity.

Does this PR introduce any user-facing change?

NA

How was this patch tested?

Existing UTs

@s0nskar s0nskar changed the title Deprecate identity provider configs tied with quota [CELEBORN-1748] Deprecate identity provider configs tied with quota Nov 26, 2024
@s0nskar s0nskar marked this pull request as ready for review November 26, 2024 14:11
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.

1 participant