Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[LIBCLOUD-940] Add 'end' to ec2 Reserved Instance data #1099

Closed
wants to merge 1 commit into from

Conversation

xofer
Copy link
Contributor

@xofer xofer commented Aug 30, 2017

Add 'end' to ec2 Reserved Instance data

Description

The DescribeReservedInstances API returns 3 fields related to the timeframe of the reservation:

  • start
  • end
  • duration

When a reservation is modified, start changes to the effective date of the modification, but duration remains the same (number of seconds in 1 or 3 years). The only way to determine when a modified reservation expires is to use the end date.

Status

done, ready for review

Checklist (tick everything that applies)

@coveralls
Copy link

Coverage Status

Coverage increased (+0.0002%) to 88.975% when pulling 100cd5a on xofer:ec2-ri-end into 84735c8 on apache:trunk.

@pquentin
Copy link
Contributor

👍

@tonybaloney
Copy link
Contributor

thanks @xofer merging

@asfgit asfgit closed this in d47adca Sep 3, 2017
asfgit pushed a commit that referenced this pull request Sep 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants