Added indexing user activity events operation #19
Conversation
bshien
force-pushed
the
index-events-2
branch
2 times, most recently
from
68ffd3f
to
476f61e
Compare
bshien
changed the title
bshien
changed the title
bshien
force-pushed
the
index-events-2
branch
3 times, most recently
from
0231394
to
4fde209
Compare
bshien
requested review from
getsaurabh02,
peterzhuamazon,
gaiksaya,
prudhvigodithi and
nhtruong
as code owners
There was a problem hiding this comment.
Hi @bshien
Please rebase on main and resolve conflicts as a few files already updated by @prudhvigodithi in #21.
Thanks.
Signed-off-by: Brandon Shien <[email protected]>
bshien
force-pushed
the
index-events-2
branch
2 times, most recently
from
f50741f
to
974fa81
Compare
| - issues.opened | ||
| - issues.closed | ||
| - issues.labeled | ||
| - issues.unlabeled | ||
| - issues.transferred | ||
| - issues.assigned | ||
| - issue_comment.created | ||
| - push | ||
| - pull_request.closed | ||
| - pull_request.opened | ||
| - pull_request.labeled | ||
| - pull_request.unlabeled | ||
| - pull_request.assigned | ||
| - pull_request_review.submitted | ||
| - pull_request_review_comment.created | ||
| - gollum | ||
| - release.released | ||
| - project.edited |
There was a problem hiding this comment.
These needs to be enabled in opensearch-infra app before take effect.
There was a problem hiding this comment.
Right, it would be great if you could enable these. If we don't want to enable it's ok because most of the activity is coming from issue and pull_request
There was a problem hiding this comment.
We probably can ignore the project.edited for now and release.released.
I would start with the issues, pull_request and gollum now and bake for a few days.
You can comment out the others.
bshien
force-pushed
the
index-events-2
branch
2 times, most recently
from
8096d24
to
bf878a3
Compare
| # - push | ||
| - pull_request.closed | ||
| - pull_request.opened | ||
| - pull_request.labeled | ||
| - pull_request.unlabeled | ||
| - pull_request.assigned | ||
| - pull_request_review.submitted | ||
| - pull_request_review_comment.created | ||
| - gollum | ||
| # - release.released | ||
| # - project.edited |
There was a problem hiding this comment.
Spacing, or probably move all three commented ones to the bottom of the array.
Signed-off-by: Brandon Shien <[email protected]>
Description
Coming from opensearch-project/opensearch-metrics#76
Added operation to index events pertaining to user activity into an OpenSearch cluster for further processing to monitor Maintainer activity.
Issues Resolved
Part of opensearch-project/opensearch-metrics#57
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.