Thanks @brockn, this is a good start!

This fixes the tests, but I think we could go one step further and deserialize 
the tags to the Image class. It should pretty straightforward if you reuse the 
existing 
[TagSetHandler](https://github.com/jclouds/jclouds/blob/master/apis/ec2/src/main/java/org/jclouds/ec2/xml/TagSetHandler.java)
 to parse the tags. If you take a look at the 
[AWSDescribeInstancesResponseHandler](https://github.com/jclouds/jclouds/blob/master/providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/xml/AWSDescribeInstancesResponseHandler.java)
 you will see how to do that. That class builds a 
[RunningInstance](https://github.com/jclouds/jclouds/blob/master/apis/ec2/src/main/java/org/jclouds/ec2/domain/RunningInstance.java)
 object and adds the tags to it. You could add them to the Image class in a 
very similar way.

Could you give it a try?


---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/55#issuecomment-20493774

Reply via email to