Skip to content

[Connectors] Include manage_connector privilege in generated api keys#187361

Merged
jedrazb merged 1 commit intoelastic:mainfrom
jedrazb:connector-api-include-manage-connector-privilege-for-api-keys
Jul 2, 2024
Merged

[Connectors] Include manage_connector privilege in generated api keys#187361
jedrazb merged 1 commit intoelastic:mainfrom
jedrazb:connector-api-include-manage-connector-privilege-for-api-keys

Conversation

@jedrazb
Copy link
Copy Markdown
Contributor

@jedrazb jedrazb commented Jul 2, 2024

Summary

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

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

Note: this privilege was merged to ES yesterday, so CI might fail if the ES image was not updated yet.

Checklist

Delete any items that are not applicable to this PR.

@jedrazb jedrazb added release_note:skip Skip the PR/issue when compiling release notes v8.15.0 labels Jul 2, 2024
@jedrazb jedrazb marked this pull request as ready for review July 2, 2024 12:54
@jedrazb jedrazb requested a review from a team as a code owner July 2, 2024 12:54
Copy link
Copy Markdown
Member

@sphilipse sphilipse left a comment

Choose a reason for hiding this comment

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

lgtm

@jedrazb
Copy link
Copy Markdown
Contributor Author

jedrazb commented Jul 2, 2024

I will wait with merging until we can verify with up-to-date ES docker image, even if it implies merging tomorrow

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@jedrazb
Copy link
Copy Markdown
Contributor Author

jedrazb commented Jul 2, 2024

Screenshot 2024-07-02 at 17 07 07

All good! Validated with "Generate API key" button on the latest stack snapshot Merging

@jedrazb jedrazb merged commit bd9a2ad into elastic:main Jul 2, 2024
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Search v8.15.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants