-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add cpu utilization as a parameter with which desires can be obtained #74
Conversation
2c09f54
to
352a776
Compare
7412df2
to
82dd4c5
Compare
i noticed a lot of whitespace change. @jolynch what is there a formatter that you used? |
I don't think the model should be aware what the current instance is. I would like to see the current instance to be used as signal to the planner that one of the least_regret plans must be of that instance. that way we could:
this also introduces more challenges to compositions of models since a kv cluster would likely have a different current instance than the composed cassandra. I'm thinking we should depreciate composition and have the client invoke the planner for each model separately. |
4cbb8a5
to
ffb2e55
Compare
ffb2e55
to
a76552e
Compare
a76552e
to
67413e8
Compare
…pes (+5 squashed commits) Squashed commits: [aa15bd9] joey changes on new types [592def6] consolidate current instance type (str), count (interval), cpu% (interval) into a new model object [82dd4c5] required cluster type parameter is now per zone [0ce2064] Moved instance type definition inside interface (joey comments addressed) [352a776] Add cpu utilization as a parameter in extra model arguments; If you have cpu usage already use it directly
d953a3b
to
fac3418
Compare
fac3418
to
de475e0
Compare
Use CPU utilization directly within extra model arguments. If we have utilization numbers, we can directly leverage them to obtain desired instance types. If we don't, then use the default reads and writes to compute CPU util which in turn can be used to obtain desired instance types.
Also added pytest in the setup so that it can integrate seamlessly with pycharm.