This repository has been archived by the owner on Jul 9, 2024. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@shrinandj I realize now that the bug I was experiencing was actually fixed in v0.4 of the code. I applied the mm.yaml as is (still on v0.3) and that is why I experienced the bug. I updated the mm.yaml to the refer to the latest tag (v0.6 not sure if we should do that or latest tag).
I also updated the IAM permissions required in the README since I ran into some issues there as well.
ec2:DescribeInstances is needed for line 81 in aws_minion_manager.py
DescribeLaunchConfiguration needs to be plural.