Skip to content

Small improvements to GetSnapshotsIT#testAllFeatures#142825

Merged
DaveCTurner merged 1 commit intoelastic:mainfrom
DaveCTurner:2026/02/23/GetSnapshotsIT-testAllFeatures-strengthen
Feb 24, 2026
Merged

Small improvements to GetSnapshotsIT#testAllFeatures#142825
DaveCTurner merged 1 commit intoelastic:mainfrom
DaveCTurner:2026/02/23/GetSnapshotsIT-testAllFeatures-strengthen

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

  • Prefix random names for more comprehensible logs
  • Sometimes do not call .policies() and .states()
  • Explain why ?from_sort_value is ignored in selectedSnapshots
  • Do not clear out all SnapshotDetails fields

- Prefix random names for more comprehensible logs
- Sometimes do not call `.policies()` and `.states()`
- Explain why `?from_sort_value` is ignored in `selectedSnapshots`
- Do not clear out all `SnapshotDetails` fields
@DaveCTurner DaveCTurner requested a review from ywangd February 23, 2026 07:29
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v9.4.0 labels Feb 23, 2026
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Meta label for distributed team. label Feb 23, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

Copy link
Copy Markdown
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@DaveCTurner DaveCTurner merged commit 1ca1b32 into elastic:main Feb 24, 2026
35 checks passed
@DaveCTurner DaveCTurner deleted the 2026/02/23/GetSnapshotsIT-testAllFeatures-strengthen branch February 24, 2026 08:31
szybia added a commit to szybia/elasticsearch that referenced this pull request Feb 24, 2026
…on-sliced-reindex

* upstream/main:
  Update docs for v9.3.1 release (elastic#142887)
  Update docs for v9.2.6 release (elastic#142888)
  Improves visibility of vector index options and inference configuration (elastic#141653)
  Disable CAE in microsoft-graph-authz plugin (elastic#142848)
  Small improvements to `GetSnapshotsIT#testAllFeatures` (elastic#142825)
  Fix IndexSettingsTests synthetic ID tests (elastic#142654)
  [Test] Unmute tests of SnapshotShutdownIT (elastic#142921)
  Fixing metrics_info.json kibana definition file name (elastic#142813)
  [Packaging] Disable glibc 2.43 malloc huge pages in Wolfi images (elastic#142894)
  Mute org.elasticsearch.xpack.searchablesnapshots.SearchableSnapshotsTSDBSyntheticIdIntegTests testSearchableSnapshot elastic#142918
  Add shard heap usage to ClusterInfo (elastic#139557)
  ESQL: Load script fields row-by-row (elastic#142807)
  ESQL: Consolidate doc values memory tracking (elastic#142816)
  ES-14124  Create Index Count Limit User documentation Page (elastic#142570)
  Add a es819 codec test to verify tryRead returns null if may contain duplicates (elastic#142409)
  Support arithmetic operations for dense_vectors: scalar version (elastic#141060)
  [Transform] Allow project_routing (elastic#142421)
  Refactor query rewrite async actions for knn and sparse_vector queries (elastic#142889)
  Do not mark bulk indexing requests as retried after primary relocations (elastic#142157)
sidosera pushed a commit to sidosera/elasticsearch that referenced this pull request Feb 24, 2026
- Prefix random names for more comprehensible logs
- Sometimes do not call `.policies()` and `.states()`
- Explain why `?from_sort_value` is ignored in `selectedSnapshots`
- Do not clear out all `SnapshotDetails` fields
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants