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.

Sign up for GitHub

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

Jump to bottom

[chore]: Document logs_index of OpensearchExporter #32851

Merged
merged 1 commit into from
Jul 1, 2024
Merged

[chore]: Document logs_index of OpensearchExporter #32851

merged 1 commit into from
Jul 1, 2024

Conversation

arpitjindal97
Copy link
Contributor

Documentation: Mention logs_index field in README

@arpitjindal97 arpitjindal97 requested a review from a team May 3, 2024 20:54
@arpitjindal97 arpitjindal97 changed the title [opensearchexporter] Mention logs_index in readme Mention logs_index in readme of OpensearchExporter May 3, 2024
@arpitjindal97 arpitjindal97 changed the title Mention logs_index in readme of OpensearchExporter [chore]: Document logs_index of OpensearchExporter May 3, 2024
@arpitjindal97
Copy link
Contributor Author

@bryan-aguilar @atoulme Please review

@crobert-1 crobert-1 added the documentation Improvements or additions to documentation label May 3, 2024
@andrzej-stencel
Copy link
Member

Can any of the code owners review? @Aneurysm9, @MitchellGale, @MaxKsyunz, @YANG-DB

@arpitjindal97
Copy link
Contributor Author

I don’t think @Aneurysm9 is active on this project

@arpitjindal97
Copy link
Contributor Author

@bryan-aguilar Can we merge this?

@arpitjindal97
[chore]: Document logs_index of OpensearchExporter
2bed5c1 
Signed-off-by: Arpit Agarwal <arpitjindal97@gmail.com>
@arpitjindal97
Copy link
Contributor Author

@andrzej-stencel Can we merge it?

@arpitjindal97
Copy link
Contributor Author

a gentle reminder

@atoulme atoulme added the ready to merge Code review completed; ready to merge by maintainers label Jun 19, 2024
@YANG-DB
Copy link
Contributor

YANG-DB commented Jun 19, 2024

Yes please merge

This was referenced Jun 19, 2024
Closed
Closed
Closed
Closed
Closed
@LucaLanziani LucaLanziani mentioned this pull request Jun 20, 2024
Closed
@TylerHelmuth TylerHelmuth merged commit 1901425 into open-telemetry:main Jul 1, 2024
145 checks passed
@github-actions github-actions bot added this to the next release milestone Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrzej-stencel andrzej-stencel andrzej-stencel left review comments

@atoulme atoulme atoulme approved these changes

@TylerHelmuth TylerHelmuth TylerHelmuth approved these changes

@YANG-DB YANG-DB YANG-DB approved these changes

@MaxKsyunz MaxKsyunz MaxKsyunz approved these changes

@MitchellGale MitchellGale MitchellGale approved these changes

@Aneurysm9 Aneurysm9 Awaiting requested review from Aneurysm9

Assignees

@andrzej-stencel andrzej-stencel

Labels
documentation Improvements or additions to documentation exporter/opensearch ready to merge Code review completed; ready to merge by maintainers
Projects
None yet
Milestone
v0.104.0
Development

Successfully merging this pull request may close these issues.
8 participants
@arpitjindal97 @andrzej-stencel @YANG-DB @atoulme @TylerHelmuth @MaxKsyunz @MitchellGale @crobert-1