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

Added example for configuring a custom region #2345

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

Conversation

dnoland1
Copy link
Contributor

@dnoland1 dnoland1 commented Mar 13, 2020

No description provided.

@dnoland1 dnoland1 changed the title Added example for configuration a custom region Added example for configuring a custom region Mar 13, 2020
@ryanelliottsmith
Copy link
Contributor

I've been struggling to get the awsCloudProvider override working:

cloud_provider:
   name: aws
   awsCloudProvider:
     service_override:
       "ec2_ap-southeast-2":
         url: https://ec2.ap-southeast-2.amazonaws.com/
         signing-region: custom-signing-region
         service: ec2
         region: ap-southeast-2

results in:

rke up                                                                                                                                                 1
INFO[0000] Running RKE version: v1.0.4
INFO[0000] Initiating Kubernetes cluster
FATA[0000] Failed to register cloud provider: Failed to parse cloud config file: Failed to parse AWS cloud config

Whereas this works:

cloud_provider:
  name: aws
  customCloudProvider: |-
    [serviceoverride "ap-southeast-2"]
    URL = https://ec2.ap-southeast-2.amazonaws.com/
    SigningRegion = custom-signing-region
    Service = ec2
    Region = ap-southeast-2

@dkeightley - any ideas?

@dkeightley
Copy link
Contributor

This is what I had experienced earlier as well, let's open an issue to track this. We should hold off merging this until we can verify the example succeeds.

@ryanelliottsmith
Copy link
Contributor

rancher/rke#1969

@catherineluse
Copy link
Contributor

Do we know yet if this example can be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants