-
Notifications
You must be signed in to change notification settings - Fork 474
Add support of Vertex AI Audit Logs #13799
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
Conversation
🚀 Benchmarks reportTo see the full report comment with |
packages/gcp_vertexai/data_stream/logs/_dev/test/pipeline/test-vertexai.log-expected.json
Outdated
Show resolved
Hide resolved
packages/gcp_vertexai/data_stream/logs/_dev/test/pipeline/test-vertexai.log-expected.json
Outdated
Show resolved
Hide resolved
packages/gcp_vertexai/data_stream/logs/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
packages/gcp_vertexai/data_stream/logs/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
packages/gcp_vertexai/data_stream/logs/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com>
@ishleenk17 - Can we add a section in the documentation for the logs data stream? |
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.
Pull Request Overview
This PR adds support for Vertex AI Audit Logs to the GCP Vertex AI integration. Key changes include:
- Updating the integration manifest and version to reflect the logs support.
- Adding a new auditlogs data stream with its own manifest, field definitions, and Elasticsearch ingest pipeline.
- Enhancing documentation and changelog to guide users on setting up and configuring Vertex AI Audit Logs.
Reviewed Changes
Copilot reviewed 10 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
packages/gcp_vertexai/manifest.yml | Updated version and description to include logs |
packages/gcp_vertexai/docs/README.md | Expanded documentation to cover logs collection and setup instructions |
packages/gcp_vertexai/data_stream/auditlogs/manifest.yml | Added new data stream manifest for Vertex AI audit logs |
packages/gcp_vertexai/data_stream/auditlogs/fields/*.yml | Defined fields for the auditlogs data stream |
packages/gcp_vertexai/data_stream/auditlogs/elasticsearch/ingest_pipeline/default.yml | Created ingest pipeline for processing audit logs |
packages/gcp_vertexai/changelog.yml | Updated changelog with details on the audit logs feature |
packages/gcp_vertexai/_dev/build/docs/README.md | Updated build docs to include audit logs integration |
Files not reviewed (3)
- packages/gcp_vertexai/data_stream/auditlogs/_dev/test/pipeline/test-vertexai.log-expected.json: Language not supported
- packages/gcp_vertexai/data_stream/auditlogs/agent/stream/stream.yml.hbs: Language not supported
- packages/gcp_vertexai/data_stream/auditlogs/sample_event.json: Language not supported
packages/gcp_vertexai/data_stream/auditlogs/elasticsearch/ingest_pipeline/default.yml
Show resolved
Hide resolved
Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com>
Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com>
Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com>
Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com>
Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com>
Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
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!
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.
Approved conditionally, please address my comments
💚 Build Succeeded
History
cc @ishleenk17 |
|
Package gcp_vertexai - 1.2.0 containing this change is available at https://epr.elastic.co/package/gcp_vertexai/1.2.0/ |
* Add support of Vertex AI Audit Logs * update changelog * Update packages/gcp_vertexai/data_stream/logs/manifest.yml Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> * Update fields.yml * Address review comments * Update tests and review comments addressed * Update the datastream name * README update * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> * Update README * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update README * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Update packages/gcp_vertexai/_dev/build/docs/README.md Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> * Generated readme * Update README --------- Co-authored-by: muthu-mps <101238137+muthu-mps@users.noreply.github.com> Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
Proposed commit message
See title
Checklist
changelog.yml
file.Related issues
Screenshots