Skip to content

Commit da73273

Browse files
committed
fixes resource name
1 parent 64bc396 commit da73273

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

ecs-task.cfndsl.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@
186186
task_def.merge!({DependsOn: depends_on }) if depends_on.length > 0
187187
task_def.merge!({ ExtraHosts: task['extra_hosts'] }) if task.has_key?('extra_hosts')
188188
task_def.merge!({VersionConsistency: task['version_consistency']}) if task.has_key?('version_consistency')
189-
task_def.merge!({ResourceRequirement: task['resource_requirement']}) if task.has_key?('resource_requirement')
189+
task_def.merge!({ResourceRequirements: task['resource_requirements']}) if task.has_key?('resource_requirements')
190190

191191
if task.key?('secrets')
192192
secrets = []

spec/resource_requirement_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
context 'cftest' do
66
it 'compiles test' do
7-
expect(system("cfhighlander cftest #{@validate} --no-validate --tests tests/resource_requirement.test.yaml")).to be_truthy
7+
expect(system("cfhighlander cftest #{@validate} --tests tests/resource_requirement.test.yaml")).to be_truthy
88
end
99
end
1010

@@ -76,7 +76,7 @@
7676
end
7777

7878
it "to have property ContainerDefinitions" do
79-
expect(resource["Properties"]["ContainerDefinitions"]).to eq([{"Name"=>"nginx","ResourceRequirement" => [{"Type" => "GPU", "Value" => 1}],"Image"=>{"Fn::Join"=>["", [{"Fn::Sub"=>"nginx/nginx"}, ":", "latest"]]},"LogConfiguration"=>{"LogDriver"=>"awslogs", "Options"=>{"awslogs-group"=>{"Ref"=>"LogGroup"}, "awslogs-region"=>{"Ref"=>"AWS::Region"}, "awslogs-stream-prefix"=>"nginx"}},"VersionConsistency"=>"disabled"}])
79+
expect(resource["Properties"]["ContainerDefinitions"]).to eq([{"Name"=>"nginx","ResourceRequirements" => [{"Type" => "GPU", "Value" => 1}],"Image"=>{"Fn::Join"=>["", [{"Fn::Sub"=>"nginx/nginx"}, ":", "latest"]]},"LogConfiguration"=>{"LogDriver"=>"awslogs", "Options"=>{"awslogs-group"=>{"Ref"=>"LogGroup"}, "awslogs-region"=>{"Ref"=>"AWS::Region"}, "awslogs-stream-prefix"=>"nginx"}},"VersionConsistency"=>"disabled"}])
8080
end
8181

8282
it "to have property RequiresCompatibilities" do

tests/resource_requirement.test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ task_definition:
99
repo: nginx
1010
image: nginx
1111
version_consistency: disabled
12-
resource_requirement:
12+
resource_requirements:
1313
- Type: 'GPU'
1414
Value: 1
1515

0 commit comments

Comments
 (0)