MAESTRO-2791 refactor to support start/stop deployment, refresh keys for authentication, various additional helpful params to the tasks
fix typo in my name
fix typo in my name
MAESTRO-2791 fixed empty elsif and tweaked validate function to by more DRY
MAESTRO-2791 fixed missing indent
MAESTRO-2791 default logwrapper def to using info level, instead of debug
MAESTRO-2791 fix logging around wait() to be more descriptive
MAESTRO-2791 check logger type rather than using begin/rescue
MAESTRO-2791 remove rescue, since it's not strictly necessary to log right here
MAESTRO-2791 merge in some of Carlos's previous changes that still gel with what the plugin is doing.  the one big one not added back in was the looping through rightscale ids that may have been placed in the context to shut them all down, which would allow user to start, start, start, then single stop them all.  Will need to add this back in separately if we really feel there is a use necessity for this
MAESTRO-2791 fix to only lookup server when we don't already have server_id
MAESTRO-2791 used short form of rescue for catch-all
MAESTRO-2791 fixed the timeout waiting for deployment (there were bugs) and tweaked wait to handle timeout <= 0 in a better way within that method too
MAESTRO-2791 only ever going to be a single item in the array, so just assign instead of push
MAESTRO-2791 not using index
MAESTRO-2791 also check for empty, in case the API returns an empty list.  keeping nil check for good measure
MAESTRO-2791 changes a couple log messages for clarity
MAESTRO-2791 fix to properly return instance instead of server when server comes online after waiting. tweaked a couple log messages to be more clear
MAESTRO-2791 adjust logging to always display the id, then name for servers & deployments for consistency
MAESTRO-2791 forgot the => :P