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

Add "description" key to opendistro-security tenant create call #2578

Closed
wants to merge 1 commit into from

Conversation

KorvinSzanto
Copy link

Add description json body key to opendistro-security tenant create call used in the sync:opendistro-security command

Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated
  • PR title is ready for changelog and subsystem label(s) applied

Opendistro-security's API endpoint for creating tenants has been updated (I couldn't find the change, maybe someone else can?) to require a description so this PR updates our API call to include a generic description.

Closing issues

closes #2577

@Schnitzel
Copy link
Contributor

superseeded by #2772
@tobybellwood can we make sure we give @KorvinSzanto credits on the release notes as he found it first? Thanks!

@Schnitzel Schnitzel closed this Jul 20, 2021
@tobybellwood
Copy link
Member

Yup! Sorry I forgot about this PR!

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.

API container sync:opendistro-security fails to create tenants
3 participants