search

adrianco / spigo

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

Dynamically create menu of architectures #24

Open adrianco opened 9 years ago

adrianco commented 9 years ago

At present there is a fixed set of fsm, lamp, netflixoss, migration. However there is now an ability to load from a json spec and create a new architecture, so there needs to be a dynamic menu.

I suggest separating the output files intended for the SPA into a separate subdirectory from the architecture and metrics files, then anything in that directory gets loaded by the SPA.

To make the transition easier, graphjson files will stay in /json

I will move arch and metrics files into /json_arch and /json_metrics by changing the go code and moving existing files over

adrianco commented 9 years ago

Arch and metrics files have been moved

kkemple commented 9 years ago

Taking a look tonight

adrianco commented 9 years ago

Both lamp and netflixoss are now generated from json_arch. A new netflix_arch file has a lot more complexity, it's got a separate api and a www endpoint and three different cassandra clusters. I can't currently visualize it because I ran into local SPA install issues.

adrianco commented 8 years ago

The file https://github.com/adrianco/spigo/blob/master/ui/js/toolbar/index.js contains a fixed list of architectures near the end. This should be changed to dynamically generate a list based on what is found in the architectures directory.

hubayirp commented 8 years ago

IOT model based on Riak - nice! On Sun, Apr 3, 2016 at 10:27 AM Adrian Cockcroft notifications@github.com wrote:

The file https://github.com/adrianco/spigo/blob/master/ui/js/toolbar/index.js contains a fixed list of architectures near the end. This should be changed to dynamically generate a list based on what is found in the architectures directory.

— 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/issues/24#issuecomment-205016391