Merge pull request #529 from ovh/fix-0.36

fix: multi-provider doesn't allow Required/Optional to have different value
ovh · Jan 9, 2024 · 4d75568 · 4d75568
2 parents 3d80736 + c45118e
commit 4d75568
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/go.mod b/go.mod
@@ -63,3 +63,5 @@ require (
 )
 
 go 1.20
+
+retract v0.36.0 // OVH_ENDPOINT environment variable prevent a correct usage of the terraform provider
diff --git a/ovh/provider_new.go b/ovh/provider_new.go
@@ -37,7 +37,8 @@ func (p *OvhProvider) Schema(_ context.Context, _ provider.SchemaRequest, resp *
 	resp.Schema = schema.Schema{
 		Attributes: map[string]schema.Attribute{
 			"endpoint": schema.StringAttribute{
-				Optional:    true,
+				Required:    os.Getenv("OVH_ENDPOINT") == "",
+				Optional:    os.Getenv("OVH_ENDPOINT") != "",
 				Description: descriptions["endpoint"],
 			},
 			"application_key": schema.StringAttribute{
Original file line number	Diff line number	Diff line change
Expand Up		@@ -63,3 +63,5 @@ require (
		)

		go 1.20

		retract v0.36.0 // OVH_ENDPOINT environment variable prevent a correct usage of the terraform provider