Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

Security Groups as dict #46

Merged
merged 1 commit into from Mar 24, 2015
Merged

Security Groups as dict #46

merged 1 commit into from Mar 24, 2015

Conversation

mattmb
Copy link
Contributor

@mattmb mattmb commented Mar 23, 2015

This changes the security groups section of the yaml file into a dict.
This means you can create multiple SGs with multiple rules and they can reference each other.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.43%) to 44.34% when pulling 8baf209 on feature/sg-refs into 093d2f9 on master.

@stuartornum
Copy link
Contributor

LGTM - although I have not tested it yet... will do now

@coveralls
Copy link

Coverage Status

Coverage increased (+1.49%) to 45.4% when pulling 7084669 on feature/sg-refs into 093d2f9 on master.

Security Groups as dict
d9b459b 
This changes the security groups section of the yaml file into a dict.
This means you can create multiple SGs with multiple rules and they can
reference each other.
stuartornum added a commit that referenced this pull request Mar 24, 2015
@stuartornum
Merge pull request #46 from ministryofjustice/feature/sg-refs
e913f02 
Security Groups as dict
@stuartornum stuartornum merged commit e913f02 into master Mar 24, 2015
@coveralls
Copy link

Coverage Status

Coverage increased (+1.65%) to 48.56% when pulling d9b459b on feature/sg-refs into aecbe0f on master.

@mattmb mattmb mentioned this pull request Mar 25, 2015
Closed
@ashb ashb deleted the feature/sg-refs branch April 8, 2015 10:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@mattmb @coveralls @stuartornum