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

Add Index Template APIs #7635

Merged
merged 27 commits into from
Jul 31, 2024
Merged

Add Index Template APIs #7635

merged 27 commits into from
Jul 31, 2024

Conversation

Naarcha-AWS
Copy link
Collaborator

@Naarcha-AWS Naarcha-AWS commented Jul 5, 2024

Closes #5394
Closes #2113

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@hdhalter
Copy link
Contributor

hdhalter commented Jul 5, 2024

Thanks, @Naarcha-AWS! Does this also close #2115?

@Naarcha-AWS
Copy link
Collaborator Author

#2115 Looks like an additional API, though fairly short. I'll add it to this PR.

@Naarcha-AWS Naarcha-AWS changed the title Add Index Template APIS Add Index Template APIs Jul 5, 2024
@Naarcha-AWS
Copy link
Collaborator Author

@gbbafna: Can you take a look at this one?

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
@Naarcha-AWS
Copy link
Collaborator Author

Few minor comments. Also, have we verified that the mention curl requests / jsons are working?

Yes, all of the examples in the doc I tested.

@Naarcha-AWS Naarcha-AWS added 4 - Doc review PR: Doc review in progress and removed 3 - Tech review PR: Tech review in progress labels Jul 29, 2024
Copy link
Contributor

@hdhalter hdhalter left a comment

Choose a reason for hiding this comment

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

Looks good, just a few suggestions. Thanks!

_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
@Naarcha-AWS Naarcha-AWS added 5 - Editorial review PR: Editorial review in progress and removed 4 - Doc review PR: Doc review in progress labels Jul 30, 2024
Naarcha-AWS and others added 2 commits July 30, 2024 14:37
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

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

@Naarcha-AWS Please see my comments and changes and let me know if you have any questions. Thanks!

_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved
_api-reference/index-apis/component-template.md Outdated Show resolved Hide resolved

## Example requests

The following example request gets information about an index template using a wildcard express:
Copy link
Collaborator

@natebower natebower Jul 31, 2024

Choose a reason for hiding this comment

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

Suggested change
The following example request gets information about an index template using a wildcard express:
The following example request gets information about an index template by using a wildcard expression:


## Example requests

The following example request gets information about an index template using a wildcard express:
Copy link
Collaborator

Choose a reason for hiding this comment

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

"by using"? (if the request is using the wildcard expression)


The following example request gets information about an index template using a wildcard express:

```
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
```
```json

```
{% include copy-curl.html %}

The following example gets information about all index templates:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The following example gets information about all index templates:
The following example request gets information about all index templates:


The following example gets information about all index templates:

```
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
```
```json

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
@Naarcha-AWS Naarcha-AWS merged commit 47d64f8 into main Jul 31, 2024
6 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 31, 2024
* Add Index Template APIs

Signed-off-by: Archer <[email protected]>

* Add Index Template APIs

Signed-off-by: Archer <[email protected]>

* Add Delete template API

Signed-off-by: Archer <[email protected]>

* Add delete API

Signed-off-by: Archer <[email protected]>

* Add component template.

Signed-off-by: Archer <[email protected]>

* Add missing parameter

Signed-off-by: Archer <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Update _api-reference/index-apis/create-index-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update _api-reference/index-apis/create-index-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Update _api-reference/index-apis/component-template.md

Signed-off-by: Heather Halter <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Co-authored-by: Sarthak Aggarwal <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Update component-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update create-index-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update delete-index-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: Archer <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Co-authored-by: Sarthak Aggarwal <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
(cherry picked from commit 47d64f8)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
github-actions bot pushed a commit that referenced this pull request Jul 31, 2024
mingshl pushed a commit to mingshl/documentation-website that referenced this pull request Aug 2, 2024
* Add Index Template APIs

Signed-off-by: Archer <[email protected]>

* Add Index Template APIs

Signed-off-by: Archer <[email protected]>

* Add Delete template API

Signed-off-by: Archer <[email protected]>

* Add delete API

Signed-off-by: Archer <[email protected]>

* Add component template.

Signed-off-by: Archer <[email protected]>

* Add missing parameter

Signed-off-by: Archer <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Update _api-reference/index-apis/create-index-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update _api-reference/index-apis/create-index-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Update _api-reference/index-apis/component-template.md

Signed-off-by: Heather Halter <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Co-authored-by: Sarthak Aggarwal <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Update component-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update create-index-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update delete-index-template.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: Archer <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Co-authored-by: Sarthak Aggarwal <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
@Naarcha-AWS Naarcha-AWS deleted the index-template-api branch January 8, 2025 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Editorial review PR: Editorial review in progress API backport 2.15
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] The `_index_template' API docs are missing [DOC]Create _component_template API
5 participants