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

Clarification of Customer Comment # 20230525 #6374

Conversation

leanneeliatra
Copy link
Contributor

@leanneeliatra leanneeliatra commented Feb 8, 2024

Description

A customer submitted a query in relation to this page of the documentation: https://opensearch.org/docs/latest/security/multi-tenancy/multi-tenancy-config/

Customer query: Is this for kibana or opensearch dashboards? n seems to override any sort filter.

Answer to customers query:
This is for OpenSearch Dashboards. The configuration mentioned is for OpenSearch Dashboards, and while it retains references to Kibana for legacy reasons, it pertains to OpenSearch Dashboards specifically. Future updates may remove these references.

Fix: Add clarification to the documentation.

Issues Resolved
This ticket addresses one part of #4314

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.

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>
@leanneeliatra
Copy link
Contributor Author

Can I request review of this PR please?
cc @kolchfa-aws , @Naarcha-AWS, @vagimeli @AMoo-Miki , @natebower , @dlvenable

@Naarcha-AWS Naarcha-AWS added backport 2.11 PR: Backport label for 2.11 4 - Doc review PR: Doc review in progress labels Feb 8, 2024
@Naarcha-AWS Naarcha-AWS self-assigned this Feb 8, 2024
@@ -21,6 +21,7 @@ config:
index: '.kibana'
do_not_fail_on_forbidden: false
```
The configuration mentioned is for OpenSearch Dashboards, which inherits much of Kibana's functionality but is now developed and maintained as part of the OpenSearch project.
Copy link
Collaborator

Choose a reason for hiding this comment

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

We can probably move this sentence to line 11.

Something like, "Multi-tenancy is enabled in OpenSearch Dashboards by default. If you need to disable or change settings related to multi-tenancy, see the kibana settings in config/opensearch-security/config.yml, as shown in the following example:"

Choose a reason for hiding this comment

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

Great thank you for the suggestion @Naarcha-AWS. I have made that edit in line with your suggestion.

Copy link
Contributor

@stephen-crawford stephen-crawford left a comment

Choose a reason for hiding this comment

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

Just signing off from a technical standpoint that this is correct. I agree with @Naarcha-AWS's suggestions though. Thanks for the help @leanneeliatra

…entance clarification

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>
@Leannelb
Copy link

Just signing off from a technical standpoint that this is correct. I agree with @Naarcha-AWS's suggestions though. Thanks for the help @leanneeliatra

Thanks @scrawfor99, that edit is now complete so with @Naarcha-AWS's approval we should be able to merge now.
Thanks both!

@leanneeliatra
Copy link
Contributor Author

I was logged in on my personal account there: Leannelb, that's me: leanneeliatra.
All commits committed and signed off under leanneeliatra and I'm logged back in under leanneeliatra now!

@Naarcha-AWS Naarcha-AWS added extensions and removed 4 - Doc review PR: Doc review in progress labels Feb 13, 2024
@leanneeliatra
Copy link
Contributor Author

Should I do anything now please @Naarcha-AWS, will I click Update Branch or leave you to merge when ready? Thanks.

@Naarcha-AWS Naarcha-AWS merged commit 9799a90 into opensearch-project:main Feb 13, 2024
4 checks passed
@Naarcha-AWS
Copy link
Collaborator

@leanneeliatra: merging now.

opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 13, 2024
* 20230525 clarificaton for customer comment on refernes to kibana

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>

* 20230525 clarificaton for customer comment on refernes to kibana - scentance clarification

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>

---------

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>
(cherry picked from commit 9799a90)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Naarcha-AWS pushed a commit that referenced this pull request Feb 13, 2024
* 20230525 clarificaton for customer comment on refernes to kibana



* 20230525 clarificaton for customer comment on refernes to kibana - scentance clarification



---------


(cherry picked from commit 9799a90)

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
tianjing-li pushed a commit to tianjing-li/documentation-website that referenced this pull request Feb 14, 2024
* 20230525 clarificaton for customer comment on refernes to kibana

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>

* 20230525 clarificaton for customer comment on refernes to kibana - scentance clarification

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>

---------

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>
Signed-off-by: Tianjing Li <tianjinglimail@gmail.com>
tianjing-li added a commit to tianjing-li/documentation-website that referenced this pull request Feb 14, 2024
Signed-off-by: Tianjing Li <tianjinglimail@gmail.com>

Clarification of Customer Comment # 20230525 (opensearch-project#6374)

* 20230525 clarificaton for customer comment on refernes to kibana

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>

* 20230525 clarificaton for customer comment on refernes to kibana - scentance clarification

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>

---------

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>
Signed-off-by: Tianjing Li <tianjinglimail@gmail.com>

Add searchbp metrics to Performance Analyzer (opensearch-project#5390)

* added searchbp metrics

Signed-off-by: Heather Halter <hdhalter@amazon.com>

* Update reference.md

Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

* Update reference.md

Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

---------

Signed-off-by: Heather Halter <hdhalter@amazon.com>
Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Tianjing Li <tianjinglimail@gmail.com>

Add documentation for new reranking feature in 2.12 (opensearch-project#6368)

* Create reranking.md

document new reranking feature in 2.12

Signed-off-by: HenryL27 <hmlindeman@yahoo.com>

* Doc review and address comments

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>

* Update _search-plugins/search-pipelines/rerank-processor.md

Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>

* Update _search-plugins/search-pipelines/rerank-processor.md

Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>

---------

Signed-off-by: HenryL27 <hmlindeman@yahoo.com>
Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>
Signed-off-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
Co-authored-by: Fanit Kolchina <kolchfa@amazon.com>
Co-authored-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Tianjing Li <tianjinglimail@gmail.com>

Add Query Insights documentation (opensearch-project#6261)

* Query Insights initial documentation

Signed-off-by: Chenyang Ji <cyji@amazon.com>

* Doc review

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

* update endpoints and remove exporters

Signed-off-by: Chenyang Ji <cyji@amazon.com>

* Moved query insights to additional plugins section

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

* Added installation instructions

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>

* editorial comments

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

* Resolve merge conflicts

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

* merge conflicts and link fix

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

* Fix link

Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>

---------

Signed-off-by: Chenyang Ji <cyji@amazon.com>
Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>
Signed-off-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
Co-authored-by: Fanit Kolchina <kolchfa@amazon.com>
Co-authored-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Tianjing Li <tianjinglimail@gmail.com>

Add Running a Workload (opensearch-project#6287)

* Add Running a Workload draft

Signed-off-by: Naarcha-AWS <naarcha@amazon.com>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

* Update running-workloads.md

Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

* Update _benchmark/user-guide/running-workloads.md

Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

* Fix link

Signed-off-by: Naarcha-AWS <naarcha@amazon.com>

* Add additional missing link

Signed-off-by: Naarcha-AWS <naarcha@amazon.com>

* Add running workloads

Signed-off-by: Naarcha-AWS <naarcha@amazon.com>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

* Add numbered steps

Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>

---------

Signed-off-by: Naarcha-AWS <naarcha@amazon.com>
Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
Co-authored-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Tianjing Li <tianjinglimail@gmail.com>
kolchfa-aws added a commit that referenced this pull request Feb 14, 2024
Clarification of Customer Comment # 20230525 (#6374)

* 20230525 clarificaton for customer comment on refernes to kibana



* 20230525 clarificaton for customer comment on refernes to kibana - scentance clarification



---------




Add searchbp metrics to Performance Analyzer (#5390)

* added searchbp metrics



* Update reference.md



* Update reference.md



* Apply suggestions from code review




---------







Add documentation for new reranking feature in 2.12 (#6368)

* Create reranking.md

document new reranking feature in 2.12



* Doc review and address comments



* Apply suggestions from code review




* Update _search-plugins/search-pipelines/rerank-processor.md




* Update _search-plugins/search-pipelines/rerank-processor.md




---------









Add Query Insights documentation (#6261)

* Query Insights initial documentation



* Doc review



* update endpoints and remove exporters



* Moved query insights to additional plugins section



* Added installation instructions



* Apply suggestions from code review




* editorial comments



* Resolve merge conflicts



* merge conflicts and link fix



* Fix link



---------









Add Running a Workload (#6287)

* Add Running a Workload draft



* Apply suggestions from code review



* Update running-workloads.md



* Update _benchmark/user-guide/running-workloads.md



* Fix link



* Add additional missing link



* Add running workloads



* Apply suggestions from code review



* Apply suggestions from code review




* Apply suggestions from code review




* Add numbered steps



---------

Signed-off-by: Naarcha-AWS <naarcha@amazon.com>
Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
Signed-off-by: Tianjing Li <tianjinglimail@gmail.com>
Co-authored-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Co-authored-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
oeyh pushed a commit to oeyh/documentation-website that referenced this pull request Mar 14, 2024
* 20230525 clarificaton for customer comment on refernes to kibana

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>

* 20230525 clarificaton for customer comment on refernes to kibana - scentance clarification

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>

---------

Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>
oeyh pushed a commit to oeyh/documentation-website that referenced this pull request Mar 14, 2024
Clarification of Customer Comment # 20230525 (opensearch-project#6374)

* 20230525 clarificaton for customer comment on refernes to kibana



* 20230525 clarificaton for customer comment on refernes to kibana - scentance clarification



---------




Add searchbp metrics to Performance Analyzer (opensearch-project#5390)

* added searchbp metrics



* Update reference.md



* Update reference.md



* Apply suggestions from code review




---------







Add documentation for new reranking feature in 2.12 (opensearch-project#6368)

* Create reranking.md

document new reranking feature in 2.12



* Doc review and address comments



* Apply suggestions from code review




* Update _search-plugins/search-pipelines/rerank-processor.md




* Update _search-plugins/search-pipelines/rerank-processor.md




---------









Add Query Insights documentation (opensearch-project#6261)

* Query Insights initial documentation



* Doc review



* update endpoints and remove exporters



* Moved query insights to additional plugins section



* Added installation instructions



* Apply suggestions from code review




* editorial comments



* Resolve merge conflicts



* merge conflicts and link fix



* Fix link



---------









Add Running a Workload (opensearch-project#6287)

* Add Running a Workload draft



* Apply suggestions from code review



* Update running-workloads.md



* Update _benchmark/user-guide/running-workloads.md



* Fix link



* Add additional missing link



* Add running workloads



* Apply suggestions from code review



* Apply suggestions from code review




* Apply suggestions from code review




* Add numbered steps



---------

Signed-off-by: Naarcha-AWS <naarcha@amazon.com>
Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
Signed-off-by: Tianjing Li <tianjinglimail@gmail.com>
Co-authored-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Co-authored-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.11 PR: Backport label for 2.11 extensions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants