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

Add new FCU API support in Outscale EC2 driver #890

Closed
wants to merge 11 commits into from

Conversation

jmunhoz
Copy link
Contributor

@jmunhoz jmunhoz commented Oct 5, 2016

Add libcloud support for the following new APIs available in the Flexible Computing Unit (FCU)

  • DescribeInstanceTypes
  • DescribeProductTypes
  • GetProductType
  • DescribeQuota
  • ModifyInstanceKeypair

The patchset includes tests and fixtures for the new APIs.

Signed-off-by: Javier M. Mellid jmunhoz@igalia.com

Update version and namespace matching the current TINA OS API (AWS EC2
compatible)

This patch handle two different versions between the API version used to issue
requests and the API version crafted in the namespace. It is a bug in the
server side currently. The bugfix is work in progress. Meanwhile we need those
bits in place to keep the new APIs working.

Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
@jmunhoz jmunhoz force-pushed the fcu-api-16-q4-001 branch 2 times, most recently from 5543b78 to 3af5dca Compare October 5, 2016 13:26
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
Copy link
Contributor

@tonybaloney tonybaloney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great thanks @jmunhoz for the updates.


if filters:
raise NotImplementedError(
'quota filters are not implemented')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when will this be implemented?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tonybaloney thanks for reviewing. we hope a new release of FCU at the end of November or December. Filters in this API are being reworked cause the original implementation is raising some non expected issues in the server side and we had to rollback although the API doc was published and the interface spec is stable. This way users get a clear message and they would not report this well-know bug.

BTW this first patchset would be also a first update to catch up with the new features and improvements in the FCU. Outscale would like to contribute new APIs and available features implemented along the previous months in the server side. As expected, with each new FCU release we would revisit the non implemented bits and bugfixes to support the new features as soon as they hit production in the server side.

@tonybaloney
Copy link
Contributor

Ok. I'm happy 👍

@asfgit asfgit closed this in 4fb3bc3 Oct 7, 2016
asfgit pushed a commit that referenced this pull request Oct 7, 2016
@jmunhoz
Copy link
Contributor Author

jmunhoz commented Oct 7, 2016

@tonybaloney thanks! 👍

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.

2 participants