Fix libcloud bug #33367

[tonybaloney]

What does this PR do?

Addresses the issue highlighted in #33367

What issues does this PR fix or reference?

#33367

Previous Behavior

libcloudfuncs do not support strings

New Behavior

libcloudfuncs will support node state as either a string or an integer

Tests written?

Yes

[tonybaloney]

Just committed a test for now to prove that this is currently broken. Then I'll fix the code and (hopefully) the test should pass

[tonybaloney]

CC @supertom

[cachedout]

@techhat Could you take a look please?

[tonybaloney]

build error "Traceback (most recent call last):
File "/usr/lib64/python2.7/site-packages/tornado/testing.py", line 119, in call
result = self.orig_method(_args, *_kwargs)
File "/testing/tests/integration/netapi/rest_tornado/test_app.py", line 371, in test_get_no_mid
self.assertEqual(len(response_obj['return'][0]), 2)
AssertionError: 1 != 2"
pretty sure that wasn't me

[techhat]

@cachedout, LGTM.

[supertom]

Thanks @tonybaloney for the fix!

[meggiebot]

Need in 2016.3

[tonybaloney]

You read my mind. I saw the release notes yesterday and realised this fix wasn't in there. +1.

[rallytime]

Done in #35235