Skip to content
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

feat: Add JMX and Node exporter TCP ports for Kafka #186

Merged
merged 4 commits into from
Aug 13, 2020

Conversation

paul-pop
Copy link
Contributor

@paul-pop paul-pop commented Aug 12, 2020

Description

Adds new rules to support JMX and Node exporter ports for Kafka.

Motivation and Context

Fixes #185 and provides a pre-defined rule for metric ports.

Breaking Changes

N/A

How Has This Been Tested?

👉  pre-commit run -a
Terraform fmt............................................................Passed
Terraform docs...........................................................Passed
Check for merge conflicts................................................Passed

Pretty straightforward change so just imported my fork version into a sample TF repo and added the new rules.

@paul-pop paul-pop changed the title Add JMX and Node exporter TCP ports for Kafka feat: Add JMX and Node exporter TCP ports for Kafka Aug 12, 2020
@antonbabenko antonbabenko merged commit f49a02f into terraform-aws-modules:master Aug 13, 2020
@antonbabenko
Copy link
Member

Thanks, @paul-pop !

v3.15.0 has been just released.

hoylemd added a commit to pixlee/terraform-aws-security-group that referenced this pull request Oct 14, 2020
* upstream/master:
  Updated CHANGELOG
  chore: Fixed formatting
  feat: Allow custom Name tag for security group (terraform-aws-modules#187)
  Updated CHANGELOG
  feat: Add JMX and Node exporter TCP ports for Kafka (terraform-aws-modules#186)
  Updated CHANGELOG
  feat: aws-provider version bump to >= 2.42, < 4.0 (terraform-aws-modules#184)
  Updated CHANGELOG
  feat: Add support for Solr (terraform-aws-modules#181)
  Updated CHANGELOG
  feat: Added optional revoke_rules_on_delete functionality (required for EMR) (terraform-aws-modules#179)
  Updated CHANGELOG
  feat: Updated version requirements to support Terraform 0.13 (terraform-aws-modules#178)
  Updated CHANGELOG
  fix: Fixed MinIO rules formatting
  feat: Adding support for MinIO (terraform-aws-modules#174)
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Kafka MSK metrics ports
2 participants