unkin/terraform-provider-artifactapi
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: include all optional fields in schema for all resource types

Browse Source
This commit is contained in:
Ben Vincent
2026-06-07 16:47:03 +10:00
parent 1405675e8f
commit 31057182aa
1 changed files with 15 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/provider/resource_remote.go
+15 -31
View File
@@ -131,25 +131,19 @@ func (r *remoteResource) Schema(_ context.Context, _ resource.SchemaRequest, res
Description: "Serve stale cached content when upstream is unreachable.",
Optional: true, Computed: true, Default: booldefault.StaticBool(true),
},
}
if r.packageType == "docker" {
attrs["ban_tags_enabled"] = schema.BoolAttribute{
Description: "Enable tag banning.",
"ban_tags_enabled": schema.BoolAttribute{
Description: "Enable tag banning (docker only).",
Optional: true, Computed: true, Default: booldefault.StaticBool(false),
}
attrs["ban_tags"] = schema.ListAttribute{
Description: "Tags to ban (e.g. latest, edge).",
},
"ban_tags": schema.ListAttribute{
Description: "Tags to ban (docker only).",
Optional: true,
ElementType: types.StringType,
}
}
if r.packageType == "terraform" {
attrs["releases_remote"] = schema.StringAttribute{
Description: "Name of the CDN remote for download URL rewriting.",
},
"releases_remote": schema.StringAttribute{
Description: "Name of the CDN remote for download URL rewriting (terraform only).",
Optional: true, Computed: true, Default: stringdefault.StaticString(""),
}
},
}
resp.Schema = schema.Schema{
@@ -268,14 +262,9 @@ func (r *remoteResource) modelToAPI(ctx context.Context, m remoteResourceModel)
api.Blocklist = listToStrings(ctx, m.Blocklist)
api.MutablePatterns = listToStrings(ctx, m.MutablePatterns)
api.ImmutablePatterns = listToStrings(ctx, m.ImmutablePatterns)
if r.packageType == "docker" {
api.BanTagsEnabled = m.BanTagsEnabled.ValueBool()
api.BanTags = listToStrings(ctx, m.BanTags)
}
if r.packageType == "terraform" {
api.ReleasesRemote = m.ReleasesRemote.ValueString()
}
api.BanTagsEnabled = m.BanTagsEnabled.ValueBool()
api.BanTags = listToStrings(ctx, m.BanTags)
api.ReleasesRemote = m.ReleasesRemote.ValueString()
return api
}
@@ -296,14 +285,9 @@ func (r *remoteResource) apiToModel(ctx context.Context, api remoteAPI) remoteRe
QuarantineEnabled: types.BoolValue(api.QuarantineEnabled),
QuarantineDays: types.Int64Value(api.QuarantineDays),
StaleOnError: types.BoolValue(api.StaleOnError),
}
if r.packageType == "docker" {
m.BanTagsEnabled = types.BoolValue(api.BanTagsEnabled)
m.BanTags = stringsToList(ctx, api.BanTags)
}
if r.packageType == "terraform" {
m.ReleasesRemote = types.StringValue(api.ReleasesRemote)
BanTagsEnabled: types.BoolValue(api.BanTagsEnabled),
BanTags: stringsToList(ctx, api.BanTags),
ReleasesRemote: types.StringValue(api.ReleasesRemote),
}
return m
}