Skip to content

the rollup action must adhere to common admin-level rollup action names#65469

Merged
talevy merged 1 commit intoelastic:masterfrom
talevy:fix-rollup-action-role
Nov 25, 2020
Merged

the rollup action must adhere to common admin-level rollup action names#65469
talevy merged 1 commit intoelastic:masterfrom
talevy:fix-rollup-action-role

Conversation

@talevy
Copy link
Copy Markdown
Contributor

@talevy talevy commented Nov 24, 2020

the name of this rollup action is associated to security privileges.

renaming rollupV2 to rollup/action will have this action match
the appropriate rollup privileges.

the name of this rollup action is associated to security privileges.

renaming `rollupV2` to `rollup/action` will have this action match
the appropriate rollup privileges.
@talevy talevy added >non-issue :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data v8.0.0 Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v7.11.0 labels Nov 24, 2020
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

Copy link
Copy Markdown
Contributor

@csoulios csoulios left a comment

Choose a reason for hiding this comment

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

LGTM!

@talevy talevy merged commit cf16c40 into elastic:master Nov 25, 2020
@talevy talevy deleted the fix-rollup-action-role branch November 25, 2020 18:22
talevy added a commit that referenced this pull request Nov 25, 2020
…es (#65469)

the name of this rollup action is associated to security privileges.

renaming `rollupV2` to `rollup/action` will have this action match
the appropriate rollup privileges.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>non-issue :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v7.11.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants