Skip to content

Conversation

lanzrein
Copy link

  • Add an optional environment variable inside of coderd deployment called AWS_STS_REGIONAL_ENDPOINTS value is regional
  • Add a value in values.yaml : awsStsRegionalEndpoints.enable (false by default) which activates the environment variable AWS_STS_REGIONAL_ENDPOINTS

I have not yet tested this on a coder deployment, I will remove the WIP once I can confirm it works as intended.

@ericpaulsen ericpaulsen self-requested a review March 17, 2022 19:33
@lanzrein lanzrein changed the title WIP: Add support for AWS_STS_REGIONAL_ENDPOINTS Feat: Add support for AWS_STS_REGIONAL_ENDPOINTS Mar 25, 2022
@lanzrein lanzrein changed the title Feat: Add support for AWS_STS_REGIONAL_ENDPOINTS feat: Add support for AWS_STS_REGIONAL_ENDPOINTS Mar 25, 2022
@lanzrein lanzrein marked this pull request as ready for review March 25, 2022 08:58
@lanzrein
Copy link
Author

I have tested this on my local deployment the following :

  • Set to enable: false -- environment variable is missing.
  • Set to enable: true -- adds the env var
  • Revert to enable: false -- removes the variable as expected.

@lanzrein
Copy link
Author

@ericpaulsen do you want me to squash some commits ?

Copy link
Member

@ericpaulsen ericpaulsen left a comment

Choose a reason for hiding this comment

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

this looks good to me. however, prior to merging, i'd like for our engineers to provide their input, in case i'm missing anything.

we have squash & merge on by default. thank you for contributing!

@deansheather
Copy link
Member

I'm thinking we might get more customer value if we instead let customers just specify arbitrary environment variables for coderd. I don't want to dismiss your work, though. This is a great addition. Thoughts? @coadler @f0ssel

@deansheather
Copy link
Member

Sorry about this taking so long, I'm going try to make a separate PR today that adds support for arbitrary env variables on coderd and I'll ping you for review :)

@lanzrein
Copy link
Author

lanzrein commented Apr 4, 2022

Sorry about this taking so long, I'm going try to make a separate PR today that adds support for arbitrary env variables on coderd and I'll ping you for review :)

Hello, if you are busy at the moment I do not mind implementing this feature. How I see it would require to modify this MR to allow for env objects rather than a single awsStsRegionalEndpoints.enable boolean flag.
Cheers.

@deansheather
Copy link
Member

deansheather commented Apr 5, 2022

Thanks for this PR, I've made the env vars change in #236 which will land in 1.29.1 and 1.30.0.

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.

3 participants