Skip to content

Conversation

@JohannesMahne
Copy link

  • Add two ES|QL alerting rules: detect new chargeback groups and detect deployments missing usage data
  • Add comprehensive documentation for alert setup and configuration
  • Update Elasticsearch version requirement to 9.2.0+ for smart lookup join support
  • Add transform startup and monitoring instructions to README

Proposed commit message

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

- Add two ES|QL alerting rules: detect new chargeback groups and detect deployments missing usage data
- Add comprehensive documentation for alert setup and configuration
- Update Elasticsearch version requirement to 9.2.0+ for smart lookup join support
- Add transform startup and monitoring instructions to README
@JohannesMahne JohannesMahne requested a review from a team as a code owner December 2, 2025 15:09
@JohannesMahne JohannesMahne marked this pull request as draft December 2, 2025 15:09
@JohannesMahne JohannesMahne marked this pull request as ready for review December 2, 2025 15:12
Copy link

Copilot AI left a 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 observability alerting capabilities to the chargeback integration and updates documentation to clarify setup requirements and procedures. The changes enhance monitoring of chargeback data quality and provide comprehensive guidance for users setting up the integration.

Key changes:

  • Addition of two ES|QL alert rule templates for detecting new chargeback groups and missing usage data
  • Comprehensive documentation updates including transform startup instructions, deployment group configuration, and alert setup procedures
  • Elasticsearch version requirement update from 8.18.0+ to 9.2.0+ for smart lookup join support

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/chargeback/manifest.yml Version bump to 0.2.7
packages/chargeback/elasticsearch/transform/**/transform.yml Pipeline version updates to match new package version
packages/chargeback/docs/README.md Major documentation expansion with alert templates, transform instructions, and clarified requirements
packages/chargeback/changelog.yml Changelog entry for version 0.2.7 with corrected historical entry types
packages/chargeback/_dev/build/docs/README.md Generated documentation file mirroring README.md changes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Dec 2, 2025
@JohannesMahne JohannesMahne marked this pull request as draft December 3, 2025 06:16
@JohannesMahne JohannesMahne marked this pull request as ready for review December 3, 2025 08:05
Copy link
Contributor

@sholzhauer-es sholzhauer-es left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good improvements to the readme!

All configuration values can be updated via the configuration lookup index:

```
POST chargeback_conf_lookup/_update/config
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This only works if there is a single record. If you have multiple the doc id used needs to change.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. will update it.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated - please check.

Copy link
Contributor

@sholzhauer-es sholzhauer-es left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@JohannesMahne JohannesMahne merged commit 6361fd2 into johannes-chargeback-wip Dec 3, 2025
7 of 8 checks passed
@JohannesMahne JohannesMahne deleted the chargeback-observability-alerts branch December 3, 2025 09:20
@elasticmachine
Copy link

elasticmachine commented Dec 3, 2025

💔 Build Failed

Failed CI Steps

History

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants