Skip to content

Conversation

@yec-akamai
Copy link
Contributor

@yec-akamai yec-akamai commented Jan 15, 2026

📝 Description

This PR adds support to ACLP alert definition.

Depends on linode/linodego#878. Will need to review and merge the linodego PR first.

✔️ How to Test

Unit test:

make test-unit PKG_NAME=monitoralertdefinition

Integration test:

make test-int PKG_NAME=monitoralertdefinition
make test-int PKG_NAME=monitoralertdefinitions

@yec-akamai yec-akamai added the new-feature for new features in the changelog. label Jan 15, 2026
@yec-akamai yec-akamai requested a review from Copilot January 16, 2026 15:52
Copy link
Contributor

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 support for ACLP (Akamai Cloud Linode Platform) Monitor Alert Definition resources and data sources to the Terraform provider, enabling users to manage and query alert definitions for monitoring services.

Changes:

  • Added resource and data sources for managing monitor alert definitions
  • Implemented template-based test fixtures for alert definitions and channels
  • Updated dependency versions and provider registration

Reviewed changes

Copilot reviewed 33 out of 34 changed files in this pull request and generated 15 comments.

Show a summary per file
File Description
linode/monitoralertdefinition/framework_resource.go Implements the core CRUD operations for alert definition resource
linode/monitoralertdefinition/framework_models.go Defines data models and flattening logic for alert definitions
linode/monitoralertdefinition/framework_resource_schema.go Defines the Terraform schema for alert definition resource
linode/monitoralertdefinitions/framework_datasource.go Implements data source for listing alert definitions
linode/monitoralertdefinitions/framework_models.go Defines models for alert definitions data source
linode/monitoralertdefinition/resource_test.go Integration tests for alert definition resource
linode/monitoralertdefinitions/datasource_test.go Integration tests for alert definitions data source
linode/monitoralertdefinition/datasource_test.go Integration tests for single alert definition data source
linode/helper/framework_data.go Adds helper function for Float64 type handling
linode/framework_provider.go Registers new resources and data sources
docs/resources/monitor_alert_definition.md Documentation for alert definition resource
docs/data-sources/monitor_alert_definition.md Documentation for single alert definition data source
docs/data-sources/monitor_alert_definitions.md Documentation for alert definitions data source
go.mod Updates dependencies and adds linodego replace directive
Comments suppressed due to low confidence (1)

linode/monitoralertdefinition/tmpl/basic.gotf:1

  • Corrected spelling of 'ansible' reference in description to be consistent with actual test purpose.
{{ define "alert_definition_basic" }}

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

@linode linode deleted a comment from Copilot AI Jan 16, 2026
@yec-akamai yec-akamai marked this pull request as ready for review January 16, 2026 16:27
@yec-akamai yec-akamai requested a review from a team as a code owner January 16, 2026 16:27
@yec-akamai yec-akamai requested review from psnoch-akamai and vshanthe and removed request for a team January 16, 2026 16:27
@vshanthe vshanthe requested review from a team, dawiddzhafarov and ezilber-akamai and removed request for a team January 29, 2026 06:09
@zliang-akamai zliang-akamai self-requested a review January 29, 2026 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-feature for new features in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant