Skip to content

Include manage_connector privilege in generated api keys#2674

Merged
jedrazb merged 1 commit intomainfrom
include-manage-connector-privilege-in-cli
Jul 3, 2024
Merged

Include manage_connector privilege in generated api keys#2674
jedrazb merged 1 commit intomainfrom
include-manage-connector-privilege-in-cli

Conversation

@jedrazb
Copy link
Copy Markdown
Contributor

@jedrazb jedrazb commented Jul 3, 2024

Closes https://github.com/elastic/search-team/issues/7837

We introduced manage_connector privilege in ES: elastic/elasticsearch#110128

Let's use it for new generated API keys for connectors.

See Kibana PR for reference with similar change: elastic/kibana#187361

This is not backward compatible, since the new privilege is introduced in 8.15. Hence CLI won't work with <8.15 ES versions

Checklists

Pre-Review Checklist

  • this PR does NOT contain credentials of any kind, such as API keys or username/passwords (double check config.yml.example)
  • this PR has a meaningful title
  • this PR links to all relevant github issues that it fixes or partially addresses

@jedrazb jedrazb added the v8.15.0 label Jul 3, 2024
@jedrazb jedrazb marked this pull request as ready for review July 3, 2024 08:52
@jedrazb jedrazb requested a review from a team July 3, 2024 08:52
@jedrazb jedrazb merged commit 25cdaab into main Jul 3, 2024
@jedrazb jedrazb deleted the include-manage-connector-privilege-in-cli branch July 3, 2024 09:51
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 3, 2024

💔 Failed to create backport PR(s)

The backport operation could not be completed due to the following error:
There are no branches to backport to. Aborting.

The backport PRs will be merged automatically after passing CI.

To backport manually run:
backport --pr 2674 --autoMerge --autoMergeMethod squash

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants