search

adrianco / spigo

Simulate Protocol Interactions in Go
Apache License 2.0
1.12k stars 111 forks source link

Allow varible number of Regions and Zones instead of only 3 #71

Closed jwatson0 closed 8 years ago

jwatson0 commented 8 years ago

Allow varible number of Regions and Zones instead of only 3 (using slices instead of arrays)

I'm using Simianviz to model some internal traditional deployments that don't use AWS. I needed the ability to have a different number of Zones other than 3 in a Region. I didn't see a reason why Regions or Zones should be in a static array with assumptions about its size in other parts of the code, so I converted the arrays to slices and got their length dynamically where needed.

My next goal is to read in Regions and Zones dynamically from the Arch file so that a deployment can be modeled in different ways without changing the code.

jwatson0 commented 8 years ago

Ugh: s/varible/variable/g

adrianco commented 8 years ago

Thanks! This is the first substantial contribution to this project, much appreciated.

hubayirp commented 8 years ago

+1 On Sat, May 21, 2016 at 8:32 AM Adrian Cockcroft notifications@github.com wrote:

Thanks! This is the first substantial contribution to this project, much appreciated.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/adrianco/spigo/pull/71#issuecomment-220784058