You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Andrew Bayer <no...@github.com> on 2015/07/27 15:12:08 UTC

[jclouds] JCLOUDS-972. Fix parsing of spot reqs, set sane default for validUntil. (#825)

Get the faultCode and faultMessage to actually be parsed (though I'm
not sure they're ever used), add statusCode, statusMessage and
statusUpdateTime, and have AWSEC2TemplateOptions default to a sane 30
minute lifetime for spot instance requests, so they don't get orphaned
forever if the price is too low etc.
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/825

-- Commit Summary --

  * JCLOUDS-972. Fix parsing of spot reqs, set sane default for validUntil.

-- File Changes --

    M providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/compute/AWSEC2TemplateOptions.java (7)
    M providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/domain/SpotInstanceRequest.java (68)
    M providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/xml/SpotInstanceHandler.java (42)
    M providers/aws-ec2/src/test/java/org/jclouds/aws/ec2/xml/SpotInstanceHandlerTest.java (10)
    M providers/aws-ec2/src/test/resources/describe_spot_instance.xml (7)
    M providers/aws-ec2/src/test/resources/request_spot_instances-ebs.xml (7)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/825.patch
https://github.com/jclouds/jclouds/pull/825.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/825

Re: [jclouds] JCLOUDS-972. Fix parsing of spot reqs, set sane default for validUntil. (#825)

Posted by Ignasi Barrera <no...@github.com>.
+1

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

Re: [jclouds] JCLOUDS-972. Fix parsing of spot reqs, set sane default for validUntil. (#825)

Posted by Andrew Bayer <no...@github.com>.
Rebased, pushed.

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