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

[8.1] [Test] Fix FollowIndexSecurityIT by granting needed previleges (#84467) #84471

Merged
merged 2 commits into from
Mar 1, 2022

Conversation

ywangd
Copy link
Member

@ywangd ywangd commented Mar 1, 2022

Backports the following commits to 8.1:

…ic#84467)

CCR user on the leader cluster needs more privileges than what are
documented (elastic#61308). Specifically it needs to renew the retention lease
at a fixed time interval. This PR fixes it by granting the "manage"
index privilege to the CCR user on the leader cluster.

Note we still want to revisit privileges required CCR or at least fix
our documentation. This will be tracked with elastic#61308.

Resolves: elastic#84156
@ywangd ywangd added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport labels Mar 1, 2022
@ywangd
Copy link
Member Author

ywangd commented Mar 1, 2022

@elasticmachine update branch

@elasticsearchmachine elasticsearchmachine merged commit f37bdc7 into elastic:8.1 Mar 1, 2022
@ywangd ywangd deleted the backport/8.1/pr-84467 branch March 1, 2022 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v8.1.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants