- Notifications
You must be signed in to change notification settings - Fork 519
[rubrik] Add Monitoring Jobs datastream #12178
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
[rubrik] Add Monitoring Jobs datastream #12178
Conversation
🚀 Benchmarks reportTo see the full report comment with |
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.
LGTM left some small questions/suggestions. README is also missing a metrics section for this new data stream.
packages/rubrik/data_stream/monitoring_jobs/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
...ata_stream/monitoring_jobs/_dev/test/pipeline/test-monitoring-jobs-metrics.log-expected.json Outdated Show resolved Hide resolved
packages/rubrik/data_stream/monitoring_jobs/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
packages/rubrik/data_stream/monitoring_jobs/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
| Please remove the |
| @devamanv - The metrics includes the count for the jobs. Is there a way we could associate this metrics with the Rubrik cluster name or id? |
| @muthu-mps it's possible to gather cluster wide metrics to know how many jobs of a certain kind are running on that cluster, something like this: { "name": "cluster1", "id": "b1a9e557-0b47-4bc1-841d-e6379e733016", "archiveJobs": { "count": 24 }, "backupJobs": { "count": 646 }, "conversionJobs": { "count": 0 }, "logBackupJobs": { "count": 0 }, "recoveryJobs": { "count": 5 }, "replicationJobs": { "count": 15 } }
This would require a change in the query, but is possible. Does that align with what you suggested? |
Yes @devamanv. Adding |
💚 Build Succeeded
History
cc @devamanv |
|
muthu-mps left a comment
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.
LGTM!
| Package rubrik - 0.2.0 containing this change is available at https://epr.elastic.co/package/rubrik/0.2.0/ |


Proposed commit message
The PR adds a new Job Monitoring data stream to the new developed Rubrik package. This fetches job summary information for protection and recovery jobs separated by job type that have been scheduled, are currently running, or completed in the past 24 hours. Following are the metrics collected:
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots