Fix

Author: remyleone

scaleway/resource_object_bucket.go

@@ -9,7 +9,6 @@ import (
 	"github.com/aws/aws-sdk-go/service/s3"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
-	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
 )
 
 func resourceScalewayObjectBucket() *schema.Resource {
@@ -31,18 +30,6 @@ func resourceScalewayObjectBucket() *schema.Resource {
 				ForceNew:    true,
 				Description: "The name of the bucket",
 			},
-			"acl": {
-				Type:        schema.TypeString,
-				Optional:    true,
-				Default:     "private",
-				Description: "ACL of the bucket: either 'public-read' or 'private'.",
-				ValidateFunc: validation.StringInSlice([]string{
-					s3.ObjectCannedACLPrivate,
-					s3.ObjectCannedACLPublicRead,
-					s3.ObjectCannedACLPublicReadWrite,
-					s3.ObjectCannedACLAuthenticatedRead,
-				}, false),
-			},
 			"tags": {
 				Type: schema.TypeMap,
 				Elem: &schema.Schema{
@@ -78,7 +65,6 @@ func resourceScalewayObjectBucket() *schema.Resource {
 
 func resourceScalewayObjectBucketCreate(ctx context.Context, d *schema.ResourceData, meta interface{}) diag.Diagnostics {
 	bucketName := d.Get("name").(string)
-	acl := d.Get("acl").(string)
 
 	s3Client, region, err := s3ClientWithRegion(d, meta)
 	if err != nil {
@@ -87,7 +73,6 @@ func resourceScalewayObjectBucketCreate(ctx context.Context, d *schema.ResourceD
 
 	_, err = s3Client.CreateBucketWithContext(ctx, &s3.CreateBucketInput{
 		Bucket: aws.String(bucketName),
-		ACL:    aws.String(acl),
 	})
 	if err != nil {
 		return diag.FromErr(err)

scaleway/resource_object_bucket_test.go

@@ -261,32 +261,3 @@ func testAccCheckScalewayObjectBucketVersioning(tt *TestTools, versioningStatus
 		return nil
 	}
 }
-
-func TestAccScalewayObjectBucket_ACL(t *testing.T) {
-	testBucketName := fmt.Sprintf("terraform-test-%d", time.Now().Unix())
-	testBucketACL := "private"
-
-	resource.Test(t, resource.TestCase{
-		PreCheck:     func() { testAccPreCheck(t) },
-		Providers:    testAccProviders,
-		CheckDestroy: testAccCheckScalewayObjectBucketDestroy,
-		Steps: []resource.TestStep{
-			{
-				Config: fmt.Sprintf(`
-					resource "scaleway_object_bucket" "base" {
-						name = "%s"
-						acl = "private"
-					}`, testBucketName),
-				Check: resource.ComposeTestCheckFunc(
-					resource.TestCheckResourceAttr("scaleway_object_bucket.base", "name", testBucketName),
-					resource.TestCheckResourceAttr("scaleway_object_bucket.base", "acl", testBucketACL),
-				),
-			},
-			{
-				ResourceName:      "scaleway_object_bucket.base",
-				ImportState:       true,
-				ImportStateVerify: true,
-			},
-		},
-	})
-}