Skip to content

docs: AWS OpenSearch#26008

Merged
smallinsky merged 1 commit intomasterfrom
smallinsky/docs_opensearch
May 11, 2023
Merged

docs: AWS OpenSearch#26008
smallinsky merged 1 commit intomasterfrom
smallinsky/docs_opensearch

Conversation

@smallinsky
Copy link
Copy Markdown
Contributor

@smallinsky smallinsky commented May 10, 2023

What

OpenSearch docs.
ref: #21874

@smallinsky smallinsky force-pushed the smallinsky/docs_opensearch branch 2 times, most recently from 48df8de to c99adba Compare May 10, 2023 16:49
Comment thread docs/cspell.json Outdated
Copy link
Copy Markdown
Contributor Author

@smallinsky smallinsky May 10, 2023

Choose a reason for hiding this comment

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

just ordered by !sort and added opensearch, opensearchcli, "Brosnan" keywords to the dictionary.

@smallinsky smallinsky force-pushed the smallinsky/docs_opensearch branch from c99adba to c3eac20 Compare May 10, 2023 16:53
@smallinsky smallinsky marked this pull request as ready for review May 10, 2023 16:55
@smallinsky smallinsky force-pushed the smallinsky/docs_opensearch branch from 125f71f to 435fde7 Compare May 10, 2023 17:35
@smallinsky smallinsky requested a review from greedy52 May 10, 2023 17:35
Comment thread docs/config.json Outdated
Comment thread docs/pages/database-access/guides/aws-opensearch.mdx Outdated
Comment thread docs/pages/database-access/guides/aws-opensearch.mdx Outdated
Comment thread docs/pages/database-access/guides/aws-opensearch.mdx Outdated
Comment thread docs/pages/database-access/guides/aws-opensearch.mdx Outdated
Comment thread docs/pages/database-access/guides/aws-opensearch.mdx Outdated
Comment thread docs/pages/database-access/guides/aws-opensearch.mdx Outdated
@smallinsky smallinsky force-pushed the smallinsky/docs_opensearch branch from 435fde7 to ee98685 Compare May 10, 2023 19:05
@alexfornuto alexfornuto self-assigned this May 10, 2023
Comment thread docs/img/database-access/guides/aws-opensearch/create-ec2-role.png
Copy link
Copy Markdown
Contributor

@alexfornuto alexfornuto left a comment

Choose a reason for hiding this comment

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

I'm submitting the majority of my edits as #26028. Concerns I could not resolve in that PR follow.

Comment thread docs/pages/database-access/guides/aws-opensearch.mdx Outdated
Comment thread docs/pages/database-access/guides/aws-opensearch.mdx Outdated
@smallinsky smallinsky force-pushed the smallinsky/docs_opensearch branch 2 times, most recently from 51e549a to 9d1d19f Compare May 10, 2023 21:28
@smallinsky smallinsky requested a review from alexfornuto May 10, 2023 21:33
Copy link
Copy Markdown
Contributor

@alexfornuto alexfornuto left a comment

Choose a reason for hiding this comment

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

One outstanding comment, otherwise 👍

Comment thread docs/pages/database-access/guides/aws-opensearch.mdx Outdated
@smallinsky smallinsky force-pushed the smallinsky/docs_opensearch branch from 96f0574 to 32dbbf6 Compare May 11, 2023 07:57
@smallinsky smallinsky added this pull request to the merge queue May 11, 2023
Merged via the queue into master with commit 348d999 May 11, 2023
@smallinsky smallinsky deleted the smallinsky/docs_opensearch branch May 11, 2023 08:22
@public-teleport-github-review-bot
Copy link
Copy Markdown

@smallinsky See the table below for backport results.

Branch Result
branch/v13 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants