Skip to content

Conversation

AlexanderWert
Copy link
Member

Closes #211

@AlexanderWert AlexanderWert requested a review from bmorelli25 June 20, 2022 13:56
@github-actions github-actions bot added the aws-λ-extension AWS Lambda Extension label Jun 20, 2022
Copy link
Contributor

@simitt simitt left a comment

Choose a reason for hiding this comment

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

Haven't specifically tested, but the docs look good and straight forward to follow.

@ghost
Copy link

ghost commented Jun 20, 2022

❕ Build Aborted

Either there was a build timeout or someone aborted the build.

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Start Time: 2022-06-21T05:33:49.412+0000

  • Duration: 63 min 56 sec

Test stats 🧪

Test Results
Failed 0
Passed 110
Skipped 2
Total 112

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

Copy link
Member

@bmorelli25 bmorelli25 left a comment

Choose a reason for hiding this comment

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

Nice! Very easy to follow. I have a few recommendations—most are grammar-related. In addition, I have some recommendations to use code callouts instead of comments.

AlexanderWert and others added 13 commits June 21, 2022 07:27
@AlexanderWert AlexanderWert merged commit fbf9f7a into elastic:main Jun 21, 2022
bmorelli25 added a commit to bmorelli25/apm-aws-lambda that referenced this pull request Jun 21, 2022
…tic#222)

* Added docs section for using secrets manager with the extension

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

Co-authored-by: Brandon Morelli <[email protected]>
(cherry picked from commit fbf9f7a)
bmorelli25 pushed a commit to bmorelli25/apm-aws-lambda that referenced this pull request Jun 21, 2022
…tic#222)

* Added docs section for using secrets manager with the extension

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

Co-authored-by: Brandon Morelli <[email protected]>
(cherry picked from commit fbf9f7a)

# Conflicts:
#	docs/monitoring-aws-lambda.asciidoc
@bmorelli25
Copy link
Member

💚 All backports created successfully

Status Branch Result
8.3
8.2

Questions ?

Please refer to the Backport tool documentation

bmorelli25 added a commit that referenced this pull request Jun 21, 2022
#223)

* Added docs section for using secrets manager with the extension

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

Co-authored-by: Brandon Morelli <[email protected]>
(cherry picked from commit fbf9f7a)

Co-authored-by: Alexander Wert <[email protected]>
bmorelli25 added a commit that referenced this pull request Jun 21, 2022
#224)

* Added docs section for using secrets manager with the extension

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/create-secret-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

* Update docs/secrets-manager/add-permissions-content.asciidoc

Co-authored-by: Brandon Morelli <[email protected]>

Co-authored-by: Brandon Morelli <[email protected]>
(cherry picked from commit fbf9f7a)

# Conflicts:
#	docs/monitoring-aws-lambda.asciidoc

Co-authored-by: Alexander Wert <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Describe usage of Secret Manager in the docs

3 participants