-
Notifications
You must be signed in to change notification settings - Fork 212
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
Migrate pack #6 of datasources and resources to terraform plugin framework #903
base: master
Are you sure you want to change the base?
Conversation
Migrate data source automation actions action
10554ab
to
edfb8c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cjgajard Great job! This pretty good to me, let's keep it open until PagerDuty/go-pagerduty
gets updated.
@@ -76,3 +76,5 @@ require ( | |||
google.golang.org/protobuf v1.33.0 // indirect | |||
gopkg.in/ini.v1 v1.67.0 // indirect | |||
) | |||
|
|||
replace github.com/PagerDuty/go-pagerduty => github.com/cjgajard/go-pagerduty v0.0.0-20240712203852-d83eacdec105 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reminder: drop this replace and update deps after PagerDuty/go-pagerduty
gets updated.
Includes migration to terraform plugin framework for:
pagerduty_automation_actions_action
data.pagerduty_automation_actions_action
pagerduty_automation_actions_action_service_association
pagerduty_automation_actions_action_team_association
pagerduty_automation_actions_runner
pagerduty_automation_actions_runner_team_association
data.pagerduty_automation_actions_runner
Acceptance tests results after migration of above TF Objects...