[rackspace|lb] Add support for timeout attribute

[decklin]

I needed support for this attribute. Per the API docs it is not included in individual LB responses and the "detail" listing but not the regular listing. Added it to relevant tests.

This is pretty trivial and I can merge if no one has any issues but since I haven't touched Fog in a while I figure it'd be good to let someone look over it.

[geemus]

@decklin - thanks!

@krames - could you review? Thanks!

[krames]

@decklin Thanks for the contribution! (And the tests!!!)

I did notice that you made timeout a required attribute for the update. According to the REST docs, the timeout attribute as well as the other attributes in the update are optional.

http://docs.rackspace.com/loadbalancers/api/v1.0/clb-devguide/content/Update_Load_Balancer_Attributes-d1e1812.html

I wonder what happens if you don't specify a timeout value on creation and then update the load balancer without specifying a timeout value. I suspect it would keep it's default value, but let me double check.

Other than that it looks good to me.

[decklin]

I took a look at this, and the (default) timeout value comes back with the newly created LB whether specified or not. So, modifying another attribute and saving it will work (and explicitly include the timeout in that request).

[krames]

Thanks for looking into that, it was my only concern. I am going to go ahead with the merge.

Thanks again for the contribution!

[geemus]

Thanks!