- Notifications
You must be signed in to change notification settings - Fork 25.5k
Reduce logging levels for meter usage tests #131935
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
Merged
not-napoleon merged 2 commits into elastic:main from not-napoleon:quiet-test-logging-for-meter-usages Jul 25, 2025
Merged
Reduce logging levels for meter usage tests #131935
not-napoleon merged 2 commits into elastic:main from not-napoleon:quiet-test-logging-for-meter-usages Jul 25, 2025
+5 −5
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Pinging @elastic/es-core-infra (Team:Core/Infra) |
rjernst approved these changes Jul 25, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This was referenced Jul 25, 2025
not-napoleon added a commit to not-napoleon/elasticsearch that referenced this pull request Jul 25, 2025
This PR reduces the logging level of the test logging added in elastic#111360 to Trace. The issue that logging was intended to investigate has been closed, and there doesn't appear to be any current need for this logging. If we need it in the future, it will be trivial to re-enable.
not-napoleon added a commit to not-napoleon/elasticsearch that referenced this pull request Jul 25, 2025
This PR reduces the logging level of the test logging added in elastic#111360 to Trace. The issue that logging was intended to investigate has been closed, and there doesn't appear to be any current need for this logging. If we need it in the future, it will be trivial to re-enable.
not-napoleon added a commit to not-napoleon/elasticsearch that referenced this pull request Jul 25, 2025
This PR reduces the logging level of the test logging added in elastic#111360 to Trace. The issue that logging was intended to investigate has been closed, and there doesn't appear to be any current need for this logging. If we need it in the future, it will be trivial to re-enable.
not-napoleon added a commit to not-napoleon/elasticsearch that referenced this pull request Jul 25, 2025
This PR reduces the logging level of the test logging added in elastic#111360 to Trace. The issue that logging was intended to investigate has been closed, and there doesn't appear to be any current need for this logging. If we need it in the future, it will be trivial to re-enable.
elasticsearchmachine pushed a commit that referenced this pull request Jul 25, 2025
This PR reduces the logging level of the test logging added in #111360 to Trace. The issue that logging was intended to investigate has been closed, and there doesn't appear to be any current need for this logging. If we need it in the future, it will be trivial to re-enable.
elasticsearchmachine pushed a commit that referenced this pull request Jul 25, 2025
This PR reduces the logging level of the test logging added in #111360 to Trace. The issue that logging was intended to investigate has been closed, and there doesn't appear to be any current need for this logging. If we need it in the future, it will be trivial to re-enable.
elasticsearchmachine pushed a commit that referenced this pull request Jul 25, 2025
This PR reduces the logging level of the test logging added in #111360 to Trace. The issue that logging was intended to investigate has been closed, and there doesn't appear to be any current need for this logging. If we need it in the future, it will be trivial to re-enable.
szybia added a commit to szybia/elasticsearch that referenced this pull request Jul 28, 2025
…-tracking * upstream/main: Fix MergeWithLowDiskSpaceIT testRelocationWhileForceMerging (elastic#131806) [ML] Prevent the trained model deployment memory estimation from double-counting allocations. (elastic#131990) ES|QL Assert current thread during query planning and execution (elastic#131807) Add ElasticsearchIndexDeletionPolicy and EngineConfig policy wrapper (elastic#130442) [TEST] Adds tests for ESTestCase randomSubset methods (elastic#131745) Simplify esql session (elastic#131925) Simplify EsqlExecution info serialization (elastic#131823) Add utility to check for project global block (elastic#131927) [DOCS] Update ES|QL applies to's (elastic#131805) Handle structured log messages (elastic#131027) Mute org.elasticsearch.test.rest.yaml.RcsCcsCommonYamlTestSuiteIT test {p0=search/600_flattened_ignore_above/flattened ignore_above multi-value field} elastic#131967 Mute org.elasticsearch.xpack.remotecluster.CrossClusterEsqlRCS2EnrichUnavailableRemotesIT testEsqlEnrichWithSkipUnavailable elastic#131965 Mute org.elasticsearch.xpack.restart.FullClusterRestartIT testWatcherWithApiKey {cluster=UPGRADED} elastic#131964 [ES|QL] Fix aggregate_metric_double sorting and mv_expand issues (elastic#131658) Reduce logging levels for meter usage tests (elastic#131935)
elasticsearchmachine pushed a commit that referenced this pull request Aug 14, 2025
This PR reduces the logging level of the test logging added in #111360 to Trace. The issue that logging was intended to investigate has been closed, and there doesn't appear to be any current need for this logging. If we need it in the future, it will be trivial to re-enable.
szybia added a commit to szybia/elasticsearch that referenced this pull request Aug 15, 2025
* upstream/8.19: [8.19] Fix MergeWithLowDiskSpaceIT testRelocationWhileForceMerging (elastic#132913) Allow the test to also accept partial, this seems to be normal (elastic#132614) (elastic#132949) Add `complete` attribute to .fleet-agents docs (elastic#127651) (elastic#132903) [8.19] docs: add 429 semantic_text troubleshooting (elastic#132747) Reduce logging levels for meter usage tests (elastic#131935) (elastic#131942) Add 8.19.2 Connectors release notes (elastic#132714)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged :Core/Infra/Logging Log management and logging utilities Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests v8.18.5 v8.19.1 v9.0.5 v9.1.1 v9.2.0
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
This PR reduces the logging level of the test logging added in #111360 to Trace. The issue that logging was intended to investigate has been closed, and there doesn't appear to be any current need for this logging. If we need it in the future, it will be trivial to re-enable.