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

Enable Override of EBS Region, AZ, & Endpoint Props #518

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

akutz
Copy link
Collaborator

@akutz akutz commented Apr 25, 2017

This patch introduces the ability for the EBS driver to override the client's provided region, availability zone, and endpoint properties by specifying them in the server-side config with a suffix of !important (like CSS).

Many thanks to libStorage & REX-Ray community member @shamimgeek for discovering this! Thanks Shamim!

@akutz akutz added the ebs label Apr 25, 2017
@akutz akutz self-assigned this Apr 25, 2017
@akutz akutz requested a review from codenrhoden April 25, 2017 20:49
@codecov-io
Copy link

codecov-io commented Apr 25, 2017

Codecov Report

Merging #518 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #518   +/-   ##
======================================
  Coverage    28.4%   28.4%           
======================================
  Files          34      34           
  Lines        2003    2003           
======================================
  Hits          569     569           
  Misses       1373    1373           
  Partials       61      61

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 3a8dabc...7f6a15d. Read the comment docs.

Copy link
Contributor

@codenrhoden codenrhoden left a comment

Choose a reason for hiding this comment

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

Looks good at first glance. Definitely want to hear it works from @shamimgeek. This will be a nice way to pick up eucalyptus support!

@codenrhoden codenrhoden added this to the 2017.04 milestone May 1, 2017
@akutz akutz force-pushed the feature/ebs-important-az-region branch 2 times, most recently from f33269e to 46ac91c Compare May 2, 2017 01:51
@codenrhoden codenrhoden self-assigned this May 2, 2017
This patch introduces the ability for the EBS driver to override the
client's provide region, availability zone, and endpoint properties by
specifying them in the server-side config with a suffix of !important
(like CSS).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants