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 /etc/default/teleport as EnvironmentFile to Teleport AMIs #43626

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

webvictim
Copy link
Contributor

All Teleport unit files in AMIs should load environment variable overrides from /etc/default/teleport in the same way as regular Teleport package installations.

Updates #43616

changelog: Teleport AMIs now optionally source environment variables from /etc/default/teleport as regular Teleport package installations do.

@webvictim webvictim self-assigned this Jun 28, 2024
@webvictim webvictim added terraform-deployment-examples Issues relating to Terraform deployment examples under examples/aws/terraform ami labels Jun 28, 2024
@webvictim webvictim changed the title ami: Add /etc/default/teleport as EnvironmentFile to Teleport AMIs Add /etc/default/teleport as EnvironmentFile to Teleport AMIs Jun 28, 2024
@github-actions github-actions bot requested review from jimbishopp and r0mant June 28, 2024 03:07
@webvictim webvictim enabled auto-merge June 28, 2024 16:50
@webvictim webvictim added this pull request to the merge queue Jul 9, 2024
Merged via the queue into master with commit 0040ee3 Jul 9, 2024
39 of 42 checks passed
@webvictim webvictim deleted the gus/terraform/environmentfile branch July 9, 2024 03:24
@public-teleport-github-review-bot

@webvictim See the table below for backport results.

Branch Result
branch/v14 Create PR
branch/v15 Create PR
branch/v16 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ami backport/branch/v14 backport/branch/v15 backport/branch/v16 size/sm terraform-deployment-examples Issues relating to Terraform deployment examples under examples/aws/terraform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants