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

[release/8.0] [Egress providers - S3 store] Add support for service accounts when running dotnet monitor in kubernetes #6660

Closed
wants to merge 1 commit into from

Conversation

github-actions[bot]
Copy link
Contributor

Backport of #6626 to release/8.0

/cc @schmittjoseph @jwreford99

…unning dotnet monitor in kubernetes (#6626)

* Reference AWSSDK.SecurityToken in the S3 Storage project so that authentication via Open ID connect is possible
It is enough for the project to be referenced for this to work, and there is no reference to it in the code base, because it's existence allows the AWS SDK to work

* Documentation: add docs to cover the use of service accounts for S3

* Update documentation in line with spell checker

Spell checker flagged that Kubernetes should have an uppercase K and to use the American spelling of utilize

Co-authored-by: Justin Anderson <jander-msft@users.noreply.github.com>

* PR Feedback: Add AWSSDK.SecurityToken.dll to the Signing.props file

---------

Co-authored-by: Justin Anderson <jander-msft@users.noreply.github.com>
@github-actions github-actions bot requested a review from a team as a code owner May 16, 2024 17:09
@github-actions github-actions bot added the backport Pull requests that are backports of another pull request label May 16, 2024
@jander-msft
Copy link
Member

8.0 feature set is locked and should not receive any new features.

@jander-msft jander-msft deleted the backport/pr-6626-to-release/8.0 branch May 16, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Pull requests that are backports of another pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants