Skip to content

Commit

Permalink
fix: ignore general s3 set acl not implemented error
Browse files Browse the repository at this point in the history
  • Loading branch information
Qiu Jian committed Jul 31, 2020
1 parent 0deb908 commit 7f829f2
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions pkg/multicloud/objectstore/object.go
Expand Up @@ -52,7 +52,7 @@ func (o *SObject) GetAcl() cloudprovider.TBucketACLType {
}
}
log.Errorf("o.bucket.client.GetObjectAcl error %s", err)
return acl
return cloudprovider.ACLPrivate
}
return acl
}
Expand All @@ -61,7 +61,8 @@ func (o *SObject) SetAcl(aclStr cloudprovider.TBucketACLType) error {
err := o.bucket.client.SetObjectAcl(o.bucket.Name, o.Key, aclStr)
if err != nil {
if strings.Contains(err.Error(), "not implemented") {
return cloudprovider.ErrNotImplemented
// ignore not implemented error
return nil // cloudprovider.ErrNotImplemented
} else {
return errors.Wrap(err, "o.bucket.client.SetObjectAcl")
}
Expand Down

0 comments on commit 7f829f2

Please sign in to comment.