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

Tweak log level to avoid spamming log #7232

Merged
merged 1 commit into from
Apr 18, 2023
Merged

Tweak log level to avoid spamming log #7232

merged 1 commit into from
Apr 18, 2023

Conversation

andrross
Copy link
Member

This is a common case and info level logging will result in a ton of unhelpful log entries.

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

@andrross
Tweak log level to avoid spamming log
63c8392 
This is a common case and info level logging will result in a ton of
unhelpful log entries.

Signed-off-by: Andrew Ross <andrross@amazon.com>
dreamer-89
dreamer-89 approved these changes Apr 18, 2023
View reviewed changes
Copy link
Member

@dreamer-89 dreamer-89 left a comment

Choose a reason for hiding this comment

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

LGTM!

I have noticed this log entry :)

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.snapshots.DedicatedClusterSnapshotRestoreIT.testIndexDeletionDuringSnapshotCreationInQueue

@andrross andrross merged commit 4088812 into opensearch-project:main Apr 18, 2023
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Apr 18, 2023
Merged
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 18, 2023
@github-actions
Tweak log level to avoid spamming log (#7232)
ca41dbb 
This is a common case and info level logging will result in a ton of
unhelpful log entries.

Signed-off-by: Andrew Ross <andrross@amazon.com>
(cherry picked from commit 4088812)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Apr 18, 2023
Merged
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 18, 2023
@github-actions
Tweak log level to avoid spamming log (#7232)
59e4010 
This is a common case and info level logging will result in a ton of
unhelpful log entries.

Signed-off-by: Andrew Ross <andrross@amazon.com>
(cherry picked from commit 4088812)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@andrross andrross deleted the remote-shards-balancer-logger branch April 18, 2023 17:24
andrross pushed a commit that referenced this pull request Apr 18, 2023
@opensearch-trigger-bot @github-actions
Tweak log level to avoid spamming log (#7232) (#7233)
faf2e4e 
This is a common case and info level logging will result in a ton of
unhelpful log entries.


(cherry picked from commit 4088812)

Signed-off-by: Andrew Ross <andrross@amazon.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>
andrross pushed a commit that referenced this pull request Apr 18, 2023
@opensearch-trigger-bot @github-actions
Tweak log level to avoid spamming log (#7232) (#7234)
df0d8e2 
This is a common case and info level logging will result in a ton of
unhelpful log entries.


(cherry picked from commit 4088812)

Signed-off-by: Andrew Ross <andrross@amazon.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>
shinbay-almaz pushed a commit to shinbay-almaz/OpenSearch that referenced this pull request Apr 19, 2023
@andrross
Tweak log level to avoid spamming log (opensearch-project#7232)
63b3ff1 
This is a common case and info level logging will result in a ton of
unhelpful log entries.

Signed-off-by: Andrew Ross <andrross@amazon.com>
shinbay-almaz pushed a commit to shinbay-almaz/OpenSearch that referenced this pull request Apr 19, 2023
@andrross
Tweak log level to avoid spamming log (opensearch-project#7232)
e94f173 
This is a common case and info level logging will result in a ton of
unhelpful log entries.

Signed-off-by: Andrew Ross <andrross@amazon.com>
Signed-off-by: Almaz Shinbay <shinbay@Almazs-MacBook-Pro-2.local>
austintlee pushed a commit to austintlee/OpenSearch that referenced this pull request Apr 28, 2023
@andrross @austintlee
Tweak log level to avoid spamming log (opensearch-project#7232)
5229130 
This is a common case and info level logging will result in a ton of
unhelpful log entries.

Signed-off-by: Andrew Ross <andrross@amazon.com>
shiv0408 pushed a commit to Gaurav614/OpenSearch that referenced this pull request Apr 25, 2024
@andrross @shiv0408
Tweak log level to avoid spamming log (opensearch-project#7232)
3fe803a 
This is a common case and info level logging will result in a ton of
unhelpful log entries.

Signed-off-by: Andrew Ross <andrross@amazon.com>
Signed-off-by: Shivansh Arora <hishiv@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dreamer-89 dreamer-89 dreamer-89 approved these changes

@kotwanikunal kotwanikunal kotwanikunal approved these changes

@reta reta Awaiting requested review from reta reta is a code owner

@anasalkouz anasalkouz Awaiting requested review from anasalkouz anasalkouz is a code owner

@Bukhtawar Bukhtawar Awaiting requested review from Bukhtawar Bukhtawar is a code owner

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE CEHENKLE is a code owner

@dblock dblock Awaiting requested review from dblock dblock is a code owner

@gbbafna gbbafna Awaiting requested review from gbbafna gbbafna is a code owner

@setiah setiah Awaiting requested review from setiah

@kartg kartg Awaiting requested review from kartg

@mch2 mch2 Awaiting requested review from mch2 mch2 is a code owner

@nknize nknize Awaiting requested review from nknize nknize is a code owner

@owaiskazi19 owaiskazi19 Awaiting requested review from owaiskazi19 owaiskazi19 is a code owner

@Rishikesh1159 Rishikesh1159 Awaiting requested review from Rishikesh1159 Rishikesh1159 is a code owner

@ryanbogan ryanbogan Awaiting requested review from ryanbogan

@saratvemulapalli saratvemulapalli Awaiting requested review from saratvemulapalli saratvemulapalli is a code owner

@shwetathareja shwetathareja Awaiting requested review from shwetathareja shwetathareja is a code owner

@tlfeng tlfeng Awaiting requested review from tlfeng

@VachaShah VachaShah Awaiting requested review from VachaShah VachaShah is a code owner

@xuezhou25 xuezhou25 Awaiting requested review from xuezhou25

Assignees
No one assigned
Labels
backport 2.x Backport to 2.x branch backport 2.7 skip-changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@andrross @kotwanikunal @dreamer-89