Skip to content

Commit

Permalink
tier: Add validation to tier configuration (#256)
Browse files Browse the repository at this point in the history
  • Loading branch information
vadmeste authored Dec 21, 2023
1 parent 2169826 commit 743d3ca
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cgroup/nolinux.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,6 @@ package cgroup
import "errors"

// GetMemoryLimit - Not implemented in non-linux platforms
func GetMemoryLimit(pid int) (limit uint64, err error) {
func GetMemoryLimit(_ int) (limit uint64, err error) {
return limit, errors.New("Not implemented for non-linux platforms")
}
8 changes: 7 additions & 1 deletion tier.go
Original file line number Diff line number Diff line change
Expand Up @@ -111,8 +111,14 @@ func (adm *AdminClient) ListTiers(ctx context.Context) ([]*TierConfig, error) {
type TierCreds struct {
AccessKey string `json:"access,omitempty"`
SecretKey string `json:"secret,omitempty"`

AWSRole bool `json:"awsrole"`
AWSRoleWebIdentityTokenFile string `json:"awsroleWebIdentity,omitempty"`
AWSRoleARN string `json:"awsroleARN,omitempty"`

AzSP ServicePrincipalAuth `json:"azSP,omitempty"`

CredsJSON []byte `json:"creds,omitempty"`
AWSRole bool `json:"awsrole"`
}

// EditTier supports updating credentials for the remote tier identified by tierName.
Expand Down

0 comments on commit 743d3ca

Please sign in to comment.