Closed janvi-elastic closed 2 months ago
To see the full report comment with /test benchmark fullreport
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)
Hey @piyush-elastic,
I think this could have benefited from being four separate PRs.
Regarding @efd6 request: This is a general ask from our team and helps in improving quality of the reviews. Please consider splitting these large PRs into smaller ones.
I think we can derive useful info by splitting events under sandbox report categories like
SystemSummary
,Networking
,SecurityBypass
,Stealth
, etc. Right now they are all array of objects and we cant derive much info from them. The only thing useful right now is at the top level report information, like# of Reports completed
,Avg. time taken to complete
, etc.If its not part of current ask, maybe we could take up as an enhancement.
@jamiehynds WDYT?
100% - we should be parsing and normalising as much as we can from these reports including category and categorisation fields (as shown in the Zscaler example reports here). those fields are far more valuable than the number of reports completed, etc. @piyush-elastic we can add those before merging the PR - we can't take it as an enhancement for a future release.
Hey @piyush-elastic,
I think this could have benefited from being four separate PRs.
Regarding @efd6 request: This is a general ask from our team and helps in improving quality of the reviews. Please consider splitting these large PRs into smaller ones.
@kcreddy - Do you want us to split PR module wise like Data collection, FM , viz? or data-stream wise?
Do you want us to split PR module wise like Data collection, FM , viz? or data-stream wise?
@piyushw-crest I would like if PRs were separated by datastream (for new datastreams/integrations). I think thats what Dan also meant by 4 PRs. cc: @efd6
I'll add to not split this now. That would just result in review churn, but the request is for future PRs.
Issues
0 New issues
0 Fixed issues
0 Accepted issues
Measures
0 Security Hotspots
97.8% Coverage on New Code
0.0% Duplication on New Code
Package zscaler_zia - 3.0.0 containing this change is available at https://epr.elastic.co/search?package=zscaler_zia
Type of change
Proposed commit message
Checklist
changelog.yml
file.How to test this PR locally
Related issues
Screenshot