Repository navigation
[snowflake] Initial release with access_history, data_transfer_history, and stages data streams - #21995
Conversation
…a_transfer_history, and stages data streams.
|
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
Elastic Docs Style Checker (Vale)Summary: 1 suggestion found 💡 Suggestions (1): Optional style improvements. Apply when helpful.
The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
🚀 Benchmarks reportTo see the full report comment with |
1. Update cel code and refactor them. 2. Update script in pipeline.
kcreddy
left a comment
There was a problem hiding this comment.
This release includes access_history, data_transfer_history, and stages data streams,
associated dashboard and visualizations.
@mohitjha-elastic, I don't see any dashboards in this PR. Are they included somewhere else?
ECS git@v9.5.0 / ecs.version: 9.5.0 datastore category Dropped async=true; POST is sync. HTTP 202 polling is still there as a safety path Full batch_size pages continue from the watermark without lag until the backlog drains Duplicate-row wording in the docs User-stage objectName → related.user cloud.origin / cloud.target on data transfer Stage COMMENT → message; STAGE_OWNER → related.user; STAGE_REGION → cloud.region; s3/azure/gcs → cloud.provider
Dashboards are not included in this PR, as the recommended dashboards outlined in the main issue cannot be created using these three data streams alone due to the limited number of available fields. I’ll explore creating a combined dashboard with visualizations from multiple data streams in a follow-up PR once we have a useful set of views. Updated the commit message. |
Reframe the SQL statement.
There was a problem hiding this comment.
Please reformat this; there have been nesting changes that change indentation.
There was a problem hiding this comment.
@efd6 Reformatted the cel code and also ran celfmt command. Thanks for pointing out.
|
✅ All changelog entries have the correct PR link. |
|
🟢 No issues across the latest commits 6ff7c89. Review summaryIssues found across earlier commits fc8ce20 — 1 medium
Issues found across earlier commits c4518bc — 2 medium, 1 low
Issues found across earlier commits fbe3577 — 1 high, 1 medium, 2 low
Package-level:
🤖 AI-Generated Review | Vera Review Bot - v0.4.3 | 📚 Knowledge base: integration-skills
|
💚 Build Succeeded
History
|
|
This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again. |
|
Package snowflake - 0.1.0 containing this change is available at https://epr.elastic.co/package/snowflake/0.1.0/ |
Proposed commit message
Checklist
changelog.ymlfile.How to test this PR locally
To test the integration: