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

Migrate pack #2 of datasources and resources to terraform plugin framework #866

Merged
merged 8 commits into from
Jun 13, 2024

Conversation

cjgajard
Copy link
Contributor

@cjgajard cjgajard commented May 7, 2024

Includes migration to terraform plugin framework for:

  • pagerduty_service_dependency
  • data.pagerduty_service_integration
  • data.pagerduty_service
  • pagerduty_addon

Fix variable and property names, and other suggestions in ./pagerdutyplugin from linters golangci-lint and revive.

Acceptance tests execution results after Terraform objects migration...

TF_ACC=1 go test $(go list ./... |grep -v 'vendor') -v -count=1 -run TestAccPagerDutyBusinessServiceDependency -timeout 120m
?       github.com/PagerDuty/terraform-provider-pagerduty       [no test files]
=== RUN   TestAccPagerDutyBusinessServiceDependency_Basic
--- PASS: TestAccPagerDutyBusinessServiceDependency_Basic (28.53s)
=== RUN   TestAccPagerDutyBusinessServiceDependency_Parallel
--- PASS: TestAccPagerDutyBusinessServiceDependency_Parallel (50.23s)
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/pagerduty     80.017s
testing: warning: no tests to run
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/pagerdutyplugin       0.975s [no tests to run]
testing: warning: no tests to run
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/util  0.529s [no tests to run]
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test $(go list ./... |grep -v 'vendor') -v -count=1 -run TestAccDataSourcePagerDutyIntegration -timeout 120m
?       github.com/PagerDuty/terraform-provider-pagerduty       [no test files]
=== RUN   TestAccDataSourcePagerDutyIntegration_Basic
--- PASS: TestAccDataSourcePagerDutyIntegration_Basic (18.58s)
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/pagerduty     19.118s
testing: warning: no tests to run
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/pagerdutyplugin       0.745s [no tests to run]
testing: warning: no tests to run
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/util  0.863s [no tests to run]
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test $(go list ./... |grep -v 'vendor') -v -count=1 -run TestAccDataSourcePagerDutyService -timeout 120m
?       github.com/PagerDuty/terraform-provider-pagerduty       [no test files]
=== RUN   TestAccDataSourcePagerDutyService_Basic
--- PASS: TestAccDataSourcePagerDutyService_Basic (15.15s)
=== RUN   TestAccDataSourcePagerDutyService_HasNoTeam
--- PASS: TestAccDataSourcePagerDutyService_HasNoTeam (14.92s)
=== RUN   TestAccDataSourcePagerDutyService_HasOneTeam
--- PASS: TestAccDataSourcePagerDutyService_HasOneTeam (14.15s)
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/pagerduty     44.886s
testing: warning: no tests to run
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/pagerdutyplugin       0.412s [no tests to run]
testing: warning: no tests to run
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/util  0.746s [no tests to run]
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test $(go list ./... |grep -v 'vendor') -v -count=1 -run TestAccPagerDutyAddon -timeout 120m
?       github.com/PagerDuty/terraform-provider-pagerduty       [no test files]
=== RUN   TestAccPagerDutyAddon_Basic
--- PASS: TestAccPagerDutyAddon_Basic (7.14s)
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/pagerduty     7.664s
testing: warning: no tests to run
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/pagerdutyplugin       0.736s [no tests to run]
testing: warning: no tests to run
PASS
ok      github.com/PagerDuty/terraform-provider-pagerduty/util  0.852s [no tests to run]

@cjgajard cjgajard force-pushed the tpf-addon branch 2 times, most recently from e73b872 to 99eb75a Compare May 22, 2024 16:13
@cjgajard cjgajard requested a review from imjaroiswebdev June 13, 2024 18:11
Copy link
Contributor

@imjaroiswebdev imjaroiswebdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job migrating all these TF objects to Terraform Plugin Framework. Thank you @cjgajard 💪🏽 🎉

@imjaroiswebdev imjaroiswebdev merged commit e396059 into PagerDuty:master Jun 13, 2024
1 check passed
@cjgajard cjgajard deleted the tpf-addon branch June 17, 2024 14:43
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.

2 participants