[aws_mq] Add Terraform resource labels to aws_mq package - #20709
Open
mrodm wants to merge 1 commit into
Open
Conversation
Add four standard resource labels to the AWS provider `default_tags` block in all Terraform `main.tf` files for the aws_mq package: - activemq_audit_logs - activemq_general_logs - rabbitmq_general_logs Labels added to each file: division = "engineering" org = "obs" team = "obs-infraobs-integrations" project = "integrations-aws_mq-package" The `team` value is derived from the `owner.github` field in manifest.yml, and `project` follows the pattern `integrations-<package_name>-package` using the `name` field. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
🚀 Benchmarks reportTo see the full report comment with |
💚 Build Succeeded
cc @mrodm |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR standardizes AWS resource tagging for the aws_mq integration’s Terraform dev deploy configs by adding four org/team/project labels to the AWS provider default_tags across all existing data stream main.tf files.
Changes:
- Adds
division,org,team, andprojecttags underprovider "aws" { default_tags { ... } }for theaws_mqpackage Terraform deployments. - Applies the same label set consistently across the
activemq_audit_logs,activemq_general_logs, andrabbitmq_general_logsdata streams.
If you use Claude Code for integration work and don’t already have the repo skills installed, consider running:
/plugin install integration-skills@elastic-integration-skills
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| packages/aws_mq/data_stream/rabbitmq_general_logs/_dev/deploy/tf/main.tf | Adds standard default_tags labels for RabbitMQ general logs Terraform resources. |
| packages/aws_mq/data_stream/activemq_general_logs/_dev/deploy/tf/main.tf | Adds standard default_tags labels for ActiveMQ general logs Terraform resources. |
| packages/aws_mq/data_stream/activemq_audit_logs/_dev/deploy/tf/main.tf | Adds standard default_tags labels for ActiveMQ audit logs Terraform resources. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
|
||
| division = "engineering" | ||
| org = "obs" | ||
| team = "obs-infraobs-integrations" # owner.github in manifest.yml |
|
|
||
| division = "engineering" | ||
| org = "obs" | ||
| team = "obs-infraobs-integrations" # owner.github in manifest.yml |
|
|
||
| division = "engineering" | ||
| org = "obs" | ||
| team = "obs-infraobs-integrations" # owner.github in manifest.yml |
teresaromero
approved these changes
Aug 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed commit message
Author's Checklist
main.tffiles updated with the correctteamandprojectvalues derived frommanifest.yml