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

bazel: mount AWS credentials file into Bazel container #2341

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

msanft
Copy link
Contributor

@msanft msanft commented Sep 18, 2023

Context

Some internal CLI tools, like the versionsapi client, utilize the AWS API. For some operations, it thus is necessary to have AWS credentials available in the Bazel environment. Our Bazel container didn't have the AWS credentials available previously.

Proposed change(s)

  • Mount the AWS credential / config directory into the Bazel container.

Checklist

  • Add labels (e.g., for changelog category)
  • Is PR title adequate for changelog?
  • Link to Milestone

@msanft msanft added the no changelog Change won't be listed in release changelog label Sep 18, 2023
@msanft msanft added this to the v2.12.0 milestone Sep 18, 2023
@msanft msanft requested a review from katexochen as a code owner September 18, 2023 07:22
@netlify
Copy link

netlify bot commented Sep 18, 2023

Deploy Preview for constellation-docs canceled.

Name Link
🔨 Latest commit 7e1f961
🔍 Latest deploy log https://app.netlify.com/sites/constellation-docs/deploys/6507fabc47267500080cc411

@katexochen katexochen removed their request for review September 19, 2023 14:08
@msanft msanft merged commit 49c37b3 into main Sep 19, 2023
@msanft msanft deleted the fix/bazel/mount-aws-creds branch September 19, 2023 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Change won't be listed in release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants