这是indexloc提供的服务,不要输入任何密码
Skip to content
This repository was archived by the owner on Mar 4, 2021. It is now read-only.

Conversation

@dipthegeezer
Copy link
Contributor

Hi

When using cloud formation, the ASG created has a random string appended, such as:

foo-my-app-MyAutoScalingGroup-1727U49KMHGUT

We are currently using the opt in feature with Chaosmonkey to select a subset of our ASGs for Chaosmonkey to target.

So we put 'simianarmy.chaos.ASG.foo-my-app-MyAutoScalingGroup-1727U49KMHGUT.enabled =1' into the properties file.

However this can be a little problematic as occasionally we update the stack. This creates a new random string as a suffix to the ASG name and causes the opt-in configuration to fail to match the previously matched ASG.

Is there a way we can use some kind of pattern matching and if not could we submit a patch to solve this problem.

Dip

@coryb
Copy link
Contributor

coryb commented Jun 5, 2013

Hi Dip,

My suggestion would be to create a CloudFormationChaosMonkey that extends BasicChaosMonkey and override isGroupEnabled to ignore the random suffix on the ASG name. We would also need to change BasicMonkeyServer.getChaosMonkeyClass to be configurable (ie use simianarmy.client.chaos.class property to specify new monkey class at startup.

-Cory

@dipthegeezer
Copy link
Contributor Author

Cool, I'll have a go at making the changes and submit a patch back.

dipthegeezer and others added 5 commits June 7, 2013 17:20
…gs in logical name

  - Added new CloudFormationChaosMonkey class to remove random string in ASG name
  - New method copyAs to InstanceGroup Interface
  - Added implemention for CopyAs to BasicInstanceGroup
  - Added basic tests.
- Added unit tests
- Refactored getEffectiveProbability() to allow use of our override function
- Made chaos monkey class configurable via properties file
- Added CloudFormationChaosMonkey integration test
@dipthegeezer
Copy link
Contributor Author

Added a pull request. If there are any issues let me know and I'll fix asap.

@cloudbees-pull-request-builder

SimianArmy-pull-requests #21 SUCCESS
This pull request looks good

@cloudbees-pull-request-builder

SimianArmy-pull-requests #22 SUCCESS
This pull request looks good

@cloudbees-pull-request-builder

SimianArmy-pull-requests #23 SUCCESS
This pull request looks good

@cloudbees-pull-request-builder

SimianArmy-pull-requests #24 SUCCESS
This pull request looks good

@michaelnflx
Copy link
Contributor

The changes look pretty good. Thanks for the contribution!

michaelnflx added a commit that referenced this pull request Jun 13, 2013
Cloud Formation Auto Generated Names and ASG opt in/out
@michaelnflx michaelnflx merged commit aa98384 into Netflix:master Jun 13, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants