d8dfun/virtual-kubelet
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

[AWS Fargate] Execution role required if CloudWatch log group is specified (#194)

Browse Source 
Execution role required if CloudWatch log group is specified.
This commit is contained in:
Kit Ewbank
2018-05-10 13:37:28 -04:00
committed by Onur Filiz
parent f3ebde2533
commit fc22300b71
2 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
providers/aws/config.go
View File

@@ -102,6 +102,9 @@ func (p *FargateProvider) loadConfig(r io.Reader) error {
if config.OperatingSystem != providers.OperatingSystemLinux { if config.OperatingSystem != providers.OperatingSystemLinux {
return fmt.Errorf("Fargate does not support operating system %v", config.OperatingSystem) return fmt.Errorf("Fargate does not support operating system %v", config.OperatingSystem)
} }
if config.CloudWatchLogGroupName != "" && config.ExecutionRoleArn == "" {
return fmt.Errorf("Execution role required if CloudWatch log group is specified")
}
// Validate advertised capacity. // Validate advertised capacity.
if q, err = resource.ParseQuantity(config.CPU); err != nil { if q, err = resource.ParseQuantity(config.CPU); err != nil {

1
providers/aws/fargate.toml
View File

@@ -32,6 +32,7 @@ ExecutionRoleArn = ""
# Amazon CloudWatch log group name used to store container logs. Optional. # Amazon CloudWatch log group name used to store container logs. Optional.
# If omitted, container logs will not be available. # If omitted, container logs will not be available.
# If specified, an execution role with access to CloudWatch logs is required.
CloudWatchLogGroupName = "" CloudWatchLogGroupName = ""
# AWS Fargate platform version. Optional. Defaults to "LATEST". # AWS Fargate platform version. Optional. Defaults to "LATEST".