Skip to content

Commit

Permalink
Merge pull request #1456 from ezio-auditore/minor-fixes
Browse files Browse the repository at this point in the history
Minor lint fixes
  • Loading branch information
guymguym authored Oct 7, 2024
2 parents d7dae26 + 11392ab commit 6d82940
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions pkg/obc/provisioner.go
Original file line number Diff line number Diff line change
Expand Up @@ -577,8 +577,8 @@ func (r *BucketRequest) CreateAccount() error {
return fmt.Errorf("failed to parse NSFS config %q: %w", r.OBC.Spec.AdditionalConfig["nsfsAccountConfig"], err)
}
// We prefer to make sure this account is only used for its appropriate NSFS operations
nsfsAccountConfig.NewBucketsPath = "";
nsfsAccountConfig.NsfsOnly = true;
nsfsAccountConfig.NewBucketsPath = ""
nsfsAccountConfig.NsfsOnly = true
// -1 is the default CLI value which we use to indicate that the UID/GID should not be set
// 0 cannot be used since it is a valid GID/UID value
var IDNullifier = -1
Expand All @@ -587,14 +587,14 @@ func (r *BucketRequest) CreateAccount() error {
nsfsAccountConfig.GID = nil
}
}

accountInfo, err := r.SysClient.NBClient.CreateAccountAPI(nb.CreateAccountParams{
Name: r.AccountName,
Email: r.AccountName,
// defaultResource is left as-is only because AllowBucketCreate is false
DefaultResource: defaultResource,
HasLogin: false,
S3Access: true,
Name: r.AccountName,
Email: r.AccountName,
// defaultResource is left as-is only because AllowBucketCreate is false
DefaultResource: defaultResource,
HasLogin: false,
S3Access: true,
// If this field is to be changed, DefaultResource above will need to be modified as well
AllowBucketCreate: false,
BucketClaimOwner: r.BucketName,
Expand Down

0 comments on commit 6d82940

Please sign in to comment.