Add origin to client in SLM task#135484
Conversation
|
The previous tests did not catch this because |
|
Hi @samxbr, I've created a changelog YAML for you. |
|
Pinging @elastic/es-data-management (Team:Data Management) |
There was a problem hiding this comment.
Should we just wrap the client here, so that it does not need to be wrapped in any other place?
There was a problem hiding this comment.
I was considering that too, and alternatively I was also thinking about adding it here, but didn't do it because maybe we will want to use a different origin on other place in this class?
But on second thought now, we could always wrap again if we need to.
dakrone
left a comment
There was a problem hiding this comment.
LGTM, should this be backported also? (I can't remember if we need it elsewhere)
Double checked, the related SLM change was on 9.2 |
This change grants the
clientprivilege to call the get snapshot API.