Skip to content

Commit

Permalink
Migrate data source licenses
Browse files Browse the repository at this point in the history
  • Loading branch information
cjgajard committed Jul 4, 2024
1 parent 72d57ad commit 9103f30
Show file tree
Hide file tree
Showing 6 changed files with 153 additions and 20 deletions.
1 change: 1 addition & 0 deletions pagerduty/data_source_pagerduty_licenses.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ import (
"github.com/heimweh/go-pagerduty/pagerduty"
)

// Deprecated: Migrated to pagerdutyplugin.dataSourceLicenses. Kept for testing purposes.
func dataSourcePagerDutyLicenses() *schema.Resource {
return &schema.Resource{
Read: dataSourcePagerDutyLicensesRead,
Expand Down
1 change: 1 addition & 0 deletions pagerduty/provider.go
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,7 @@ func Provider(isMux bool) *schema.Provider {

if isMux {
delete(p.DataSourcesMap, "pagerduty_business_service")
delete(p.DataSourcesMap, "pagerduty_licenses")
delete(p.DataSourcesMap, "pagerduty_service")
delete(p.DataSourcesMap, "pagerduty_service_integration")

Expand Down
30 changes: 15 additions & 15 deletions pagerdutyplugin/data_source_pagerduty_license.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,21 +26,21 @@ func (*dataSourceLicense) Metadata(ctx context.Context, req datasource.MetadataR
}

func (*dataSourceLicense) Schema(ctx context.Context, req datasource.SchemaRequest, resp *datasource.SchemaResponse) {
resp.Schema = schema.Schema{
Attributes: map[string]schema.Attribute{
"id": schema.StringAttribute{Optional: true, Computed: true},
"name": schema.StringAttribute{Optional: true, Computed: true},
"description": schema.StringAttribute{Optional: true, Computed: true},
"type": schema.StringAttribute{Computed: true},
"summary": schema.StringAttribute{Computed: true},
"role_group": schema.StringAttribute{Computed: true},
"current_value": schema.Int64Attribute{Computed: true},
"allocations_available": schema.Int64Attribute{Computed: true},
"valid_roles": schema.ListAttribute{Computed: true, ElementType: types.StringType},
"self": schema.StringAttribute{Computed: true},
"html_url": schema.StringAttribute{Computed: true},
},
}
resp.Schema = schema.Schema{Attributes: dataSourceLicenseAttributes}
}

var dataSourceLicenseAttributes = map[string]schema.Attribute{
"id": schema.StringAttribute{Optional: true, Computed: true},
"name": schema.StringAttribute{Optional: true, Computed: true},
"description": schema.StringAttribute{Optional: true, Computed: true},
"type": schema.StringAttribute{Computed: true},
"summary": schema.StringAttribute{Computed: true},
"role_group": schema.StringAttribute{Computed: true},
"current_value": schema.Int64Attribute{Computed: true},
"allocations_available": schema.Int64Attribute{Computed: true},
"valid_roles": schema.ListAttribute{Computed: true, ElementType: types.StringType},
"self": schema.StringAttribute{Computed: true},
"html_url": schema.StringAttribute{Computed: true},
}

func (d *dataSourceLicense) Configure(ctx context.Context, req datasource.ConfigureRequest, resp *datasource.ConfigureResponse) {
Expand Down
130 changes: 130 additions & 0 deletions pagerdutyplugin/data_source_pagerduty_licenses.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,130 @@
package pagerduty

import (
"context"
"log"
"time"

"github.com/PagerDuty/go-pagerduty"
"github.com/PagerDuty/terraform-provider-pagerduty/util"
"github.com/hashicorp/terraform-plugin-framework/attr"
"github.com/hashicorp/terraform-plugin-framework/datasource"
"github.com/hashicorp/terraform-plugin-framework/datasource/schema"
"github.com/hashicorp/terraform-plugin-framework/diag"
"github.com/hashicorp/terraform-plugin-framework/types"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/id"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/retry"
)

type dataSourceLicenses struct{ client *pagerduty.Client }

var _ datasource.DataSourceWithConfigure = (*dataSourceLicenses)(nil)

func (*dataSourceLicenses) Metadata(ctx context.Context, req datasource.MetadataRequest, resp *datasource.MetadataResponse) {
resp.TypeName = "pagerduty_licenses"
}

func (*dataSourceLicenses) Schema(ctx context.Context, req datasource.SchemaRequest, resp *datasource.SchemaResponse) {
resp.Schema = schema.Schema{
Attributes: map[string]schema.Attribute{
"id": schema.StringAttribute{Optional: true},
"licenses": schema.ListAttribute{
Computed: true,
ElementType: licenseObjectType,
},
},
}
}

func (d *dataSourceLicenses) Configure(ctx context.Context, req datasource.ConfigureRequest, resp *datasource.ConfigureResponse) {
resp.Diagnostics.Append(ConfigurePagerdutyClient(&d.client, req.ProviderData)...)
}

func (d *dataSourceLicenses) Read(ctx context.Context, req datasource.ReadRequest, resp *datasource.ReadResponse) {
var model dataSourceLicensesModel
log.Println("[INFO] Reading PagerDuty licenses")

diags := req.Config.Get(ctx, &model)
if resp.Diagnostics.Append(diags...); diags.HasError() {
return
}

uid := ""
if model.ID.IsNull() {
uid = id.UniqueId()
} else {
uid = model.ID.ValueString()
}

err := retry.RetryContext(ctx, 2*time.Minute, func() *retry.RetryError {
list, err := d.client.ListLicensesWithContext(ctx)
if err != nil {
if util.IsBadRequestError(err) {
return retry.NonRetryableError(err)
}
return retry.RetryableError(err)
}

model = flattenLicenses(uid, list.Licenses, &resp.Diagnostics)
return nil
})
if err != nil {
resp.Diagnostics.AddError(
"Error reading PagerDuty licenses",
err.Error(),
)
}

resp.Diagnostics.Append(resp.State.Set(ctx, &model)...)
}

type dataSourceLicensesModel struct {
ID types.String `tfsdk:"id"`
Licenses types.List `tfsdk:"licenses"`
}

func flattenLicenses(uid string, licenses []pagerduty.License, diags *diag.Diagnostics) dataSourceLicensesModel {
elements := make([]attr.Value, 0, len(licenses))
for _, license := range licenses {
model := flattenLicense(&license)
e, d := types.ObjectValue(licenseObjectType.AttrTypes, map[string]attr.Value{
"id": model.ID,
"name": model.Name,
"description": model.Description,
"type": model.Type,
"summary": model.Summary,
"role_group": model.RoleGroup,
"current_value": model.CurrentValue,
"allocations_available": model.AllocationsAvailable,
"valid_roles": model.ValidRoles,
"self": model.Self,
"html_url": model.HTMLURL,
})
diags.Append(d...)
if d.HasError() {
continue
}
elements = append(elements, e)
}

return dataSourceLicensesModel{
Licenses: types.ListValueMust(licenseObjectType, elements),
ID: types.StringValue(uid),
}
}

var licenseObjectType = types.ObjectType{
AttrTypes: map[string]attr.Type{
"id": types.StringType,
"name": types.StringType,
"description": types.StringType,
"type": types.StringType,
"summary": types.StringType,
"role_group": types.StringType,
"current_value": types.Int64Type,
"allocations_available": types.Int64Type,
"valid_roles": types.ListType{ElemType: types.StringType},
"self": types.StringType,
"html_url": types.StringType,
},
}
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ func TestAccDataSourcePagerDutyLicenses_Basic(t *testing.T) {
name := fmt.Sprintf("tf-%s", acctest.RandString(5))

resource.Test(t, resource.TestCase{
PreCheck: func() { testAccPreCheck(t) },
Providers: testAccProviders,
PreCheck: func() { testAccPreCheck(t) },
ProtoV5ProviderFactories: testAccProtoV5ProviderFactories(),
Steps: []resource.TestStep{
{
Config: testAccDataSourcePagerDutyLicensesConfig(name),
Expand All @@ -30,8 +30,8 @@ func TestAccDataSourcePagerDutyLicenses_WithID(t *testing.T) {
name := fmt.Sprintf("tf-%s", acctest.RandString(5))

resource.Test(t, resource.TestCase{
PreCheck: func() { testAccPreCheck(t) },
Providers: testAccProviders,
PreCheck: func() { testAccPreCheck(t) },
ProtoV5ProviderFactories: testAccProtoV5ProviderFactories(),
Steps: []resource.TestStep{
{
Config: testAccDataSourcePagerDutyLicensesConfigWithID(name),
Expand Down Expand Up @@ -80,7 +80,7 @@ func testAccDataSourcePagerDutyLicensesWithID(n string, id string) resource.Test
a := r.Primary.Attributes

if val, ok := a["id"]; !ok || val != id {
return fmt.Errorf("Expected id to match provided value: %s", id)
return fmt.Errorf("Expected id to match provided value: %s\n%#v", id, a)
}

return testLicenses(a)
Expand Down
1 change: 1 addition & 0 deletions pagerdutyplugin/provider.go
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ func (p *Provider) DataSources(_ context.Context) [](func() datasource.DataSourc
func() datasource.DataSource { return &dataSourceExtensionSchema{} },
func() datasource.DataSource { return &dataSourceIntegration{} },
func() datasource.DataSource { return &dataSourceLicense{} },
func() datasource.DataSource { return &dataSourceLicenses{} },
func() datasource.DataSource { return &dataSourceService{} },
func() datasource.DataSource { return &dataSourceStandardsResourceScores{} },
func() datasource.DataSource { return &dataSourceStandardsResourcesScores{} },
Expand Down

0 comments on commit 9103f30

Please sign in to comment.