instance_id error

[467754239]

get_instance(instance_id)

errormsg:

ValueError Traceback (most recent call last)
in ()
----> 1 conn.get_instance(instance_ids[0])

/data/xxxx/lib/python2.7/site-packages/Aliyun-1.1.0-py2.7.egg/aliyun/ecs/connection.pyc in get_instance(self, instance_id)
184 int(resp['InternetMaxBandwidthOut']),
185 dateutil.parser.parse(resp['CreationTime']),
--> 186 dateutil.parser.parse(resp['ExpiredTime']),
187 resp['InstanceChargeType'],
188 resp['Description'],

/data/xxx/lib/python2.7/site-packages/python_dateutil-2.6.0-py2.7.egg/dateutil/parser.pyc in parse(timestr, parserinfo, **kwargs)
1166 return parser(parserinfo).parse(timestr, **kwargs)
1167 else:
-> 1168 return DEFAULTPARSER.parse(timestr, **kwargs)
1169
1170

/data/xxx/lib/python2.7/site-packages/python_dateutil-2.6.0-py2.7.egg/dateutil/parser.pyc in parse(self, timestr, default, ignoretz, tzinfos, **kwargs)
560
561 if len(res) == 0:
--> 562 raise ValueError("String does not contain a date.")
563
564 repl = {}

ValueError: String does not contain a date.

[bayodesegun]

This error is coming from Line 186 of aliyun.ecs.connection.py, in the get_instance() method:

dateutil.parser.parse(resp['ExpiredTime']),

ExpiredTime will return an empty string if the instance has no ExpiredTime (this is very common).

This line of code can be improved to check this, I'll submit a pull request for that ASAP.