idvoretskyi / horizon-cloud

An operations service to deploy, manage, and scale Horizon apps
0 stars 0 forks source link

API should include units in property names #129

Open mglukhovsky opened 8 years ago

mglukhovsky commented 8 years ago

Right now we have something that looks like this:

{"HorizonConfigVersion"=>{"Applied"=>1, "Desired"=>1},
 "KubeConfig"=>{"NumHorizon"=>1, "NumRDB"=>1, "SizeRDB"=>10},
 "KubeConfigVersion"=>{"Applied"=>1, "Desired"=>1},
 "Name"=>"test",
 "id"=>"test"}

Ideally, property names like SizeRDB would include units of measure. I'm not sure if this would look good with camel casing (SizeRDBGB?), but in general it's good practice to make the units self-evident from the API.