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.

Sign up for GitHub

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

Jump to bottom

Fix "Missing Region" error for teleport bootstrap commands #47898

Merged
merged 4 commits into from
Oct 27, 2024
Merged

Fix "Missing Region" error for teleport bootstrap commands #47898

merged 4 commits into from
Oct 27, 2024

Conversation

greedy52
Copy link
Contributor

@greedy52 greedy52 commented Oct 24, 2024 •
edited
Loading

changelog: Fix "Missing Region" error for teleport bootstrap commands

related:

sample outputs on my mac without region specified in ~/.aws/config:

$ teleport discovery bootstrap -c teleport_db.yaml 
Reading configuration at "teleport_db.yaml"...

Warning: No region found from the default AWS config or instance metadata. Defaulting to 'us-east-1'.
To avoid seeing this warning, please provide a region in your AWS config or through the AWS_REGION environment variable.

The agent doesn't require any extra configuration.

sample output on ec2:

$ teleport discovery bootstrap -c teleport_db.yaml 
Reading configuration at "teleport_db.yaml"...
Using region "ca-central-1" from instance metadata.

Configure AWS for Discovery Service
1. Create IAM Policy "TeleportEC2Discovery":
...

@greedy52 greedy52 self-assigned this Oct 24, 2024
@aws-amplify-us-west-2 AWS Amplify (us-west-2)
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-47898.d3pp5qlev8mo18.amplifyapp.com

@aws-amplify-us-west-2 AWS Amplify (us-west-2)
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-47898.d212ksyjt6y4yg.amplifyapp.com

@greedy52 greedy52 force-pushed the STeve/fix_missing_region_configurator branch from 891e816 to 4d436e0 Compare October 24, 2024 14:10
@greedy52 greedy52 marked this pull request as ready for review October 24, 2024 14:33
GavinFrazar
GavinFrazar approved these changes Oct 24, 2024
View reviewed changes
@greedy52 greedy52 added this pull request to the merge queue Oct 27, 2024
Merged via the queue into master with commit 09c6561 Oct 27, 2024
41 checks passed
@greedy52 greedy52 deleted the STeve/fix_missing_region_configurator branch October 27, 2024 17:00
@public-teleport-github-review-bot
Copy link

@greedy52 See the table below for backport results.

Branch Result
branch/v16 Create PR

@greedy52 greedy52 mentioned this pull request Oct 28, 2024
Merged
@greedy52 greedy52 mentioned this pull request Nov 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nklaassen nklaassen nklaassen approved these changes

@GavinFrazar GavinFrazar GavinFrazar approved these changes

Assignees

@greedy52 greedy52

Labels
backport/branch/v16 size/md
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@greedy52 @nklaassen @GavinFrazar