-
Notifications
You must be signed in to change notification settings - Fork 20
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
ARSN-458: Add action scuba:GetMetricsBatch #2281
ARSN-458: Add action scuba:GetMetricsBatch #2281
Conversation
Hello bourgoismickael,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
/approve |
863cdb2
to
78d630b
Compare
9cd5087
to
858e5a4
Compare
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option
The following options are set: approve |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following options are set: approve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be used to differentiate GetMetricsBatch
from GetMetrics
in monitoring, right ?
Because otherwise I see no need for a different authentication.
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue ARSN-458. Goodbye bourgoismickael. The following options are set: approve |
@dvasilas Yes. I put it there as the type of the action is checked. But if we differentiate in monitoring it is also better to differentiate in the permissions as well, but we can document to rather use an action like |
GetMetricsBatch action is used by identisee
branch 7.70 for vault
and 8.1 for scuba