Skip to content

FIELDENG-722 Comprehensive instructions for Readme, Terraform for Dynatrace #48

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

slorello89
Copy link

@slorello89 slorello89 commented May 9, 2025

I've written some comprehensive instructions for deploying the extension, I've also written some terraform that will automatically build, sign, upload, activate, deploy the extension in activegate.

@slorello89 slorello89 changed the title adding more comprehensive readme instructions to dynatrace_v2, adding… Comprehensive instructions for Readme, Terraform for Dynatrace May 9, 2025
@slorello89 slorello89 requested review from Copilot and a team May 9, 2025 21:00
Copy link

@Copilot 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 introduces comprehensive updates for deploying Dynatrace monitoring on GCP, including new Terraform configurations, shell scripts for starting/stopping monitoring, and updated documentation.

  • Introduces new Terraform variable definitions and configuration files for GCP integration
  • Adds shell scripts (start-monitoring.sh and stop-monitoring.sh) to manage the monitoring lifecycle
  • Updates README.adoc and sample configuration files to improve installation instructions

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
dynatrace_v2/terraform/gcp/variables.tf Adds variable definitions for GCP and Dynatrace configuration.
dynatrace_v2/terraform/gcp/stop-monitoring.sh Implements a script to stop the monitoring configuration via API call.
dynatrace_v2/terraform/gcp/start-monitoring.sh Implements a script to start monitoring with dynamic parameter updates.
dynatrace_v2/terraform/gcp/monitor.json Provides a JSON template for monitoring configuration.
dynatrace_v2/terraform/gcp/main.tf Contains Terraform resources for deploying ActiveGate and configuring extensions.
dynatrace_v2/terraform/gcp/.tfvars.example Offers a sample variables file for configuration.
dynatrace_v2/terraform/gcp/.gitignore Updates ignore rules to exclude sensitive and transient Terraform files.
dynatrace_v2/README.adoc Updates documentation with expanded instructions for installation and setup.
dynatrace_v2/.gitignore Ignores build artifacts such as extension and bundle zip files.
Files not reviewed (1)
  • dynatrace_v2/terraform/gcp/.terraform.lock.hcl: Language not supported

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@slorello89 slorello89 changed the title Comprehensive instructions for Readme, Terraform for Dynatrace FIELDENG-722 Comprehensive instructions for Readme, Terraform for Dynatrace May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant