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

Handle govcloud-specific edge cases. #251

Merged
merged 3 commits into from
Feb 6, 2017

Conversation

jmcarp
Copy link
Contributor

@jmcarp jmcarp commented Jan 20, 2017

Handle govcloud-specific edge cases.

  • Handle errors on spot instance and fleet requests
  • Handle false zero limit on EIP

Contributor License Agreement

By submitting this work for inclusion in awslimitchecker, I agree to the following terms:

  • The contribution included in this request (and any subsequent revisions or versions of it)
    is being made under the same license as the awslimitchecker project (the Affero GPL v3,
    or any subsequent version of that license if adopted by awslimitchecker).
  • My contribution may perpetually be included in and distributed with awslimitchecker; submitting
    this pull request grants a perpetual, global, unlimited license for it to be used and distributed
    under the terms of awslimitchecker's license.
  • I have the legal power and rights to agree to these terms.

@codecov-io
Copy link

codecov-io commented Jan 20, 2017

Codecov Report

Merging #251 into develop will not impact coverage.

@@          Coverage Diff           @@
##           develop   #251   +/-   ##
======================================
  Coverage      100%   100%           
======================================
  Files           22     22           
  Lines         1648   1661   +13     
  Branches       257    260    +3     
======================================
+ Hits          1648   1661   +13
Impacted Files Coverage Δ
awslimitchecker/services/ec2.py 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e733b3f...2488702. Read the comment docs.

@jmcarp
Copy link
Contributor Author

jmcarp commented Jan 30, 2017

@jantman: if this looks all right to you, I'll backfill tests. Let me know!

@jantman
Copy link
Owner

jantman commented Feb 5, 2017

@jmcarp Aplogies for this sitting for so long, I've had some family stuff going on that limited my time spent on personal projects.

Looks great to me. If you can backfill the tests, I'll get this in the next release.

PS - Hey, you're one of those 18F people! You rock!

@jmcarp jmcarp force-pushed the govcloud-edge-cases branch 3 times, most recently from 0d4aa24 to 37c459a Compare February 6, 2017 00:37
* Handle errors on spot instance and fleet requests
* Handle false zero limit on EIP
@jmcarp jmcarp force-pushed the govcloud-edge-cases branch from 37c459a to 30b98f7 Compare February 6, 2017 00:41
@jmcarp jmcarp force-pushed the govcloud-edge-cases branch from 30b98f7 to 33c994f Compare February 6, 2017 00:55
@jantman jantman merged commit b8aea0f into jantman:develop Feb 6, 2017
@jantman
Copy link
Owner

jantman commented Feb 6, 2017

Thanks so much for this!

@jantman
Copy link
Owner

jantman commented Mar 11, 2017

This was just released in 0.8.0, which is now live on PyPI. Apologies for the long delay before this release.

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.

3 participants