Skip to content

[Fleet] Fix delete by query conflict#241001

Merged
nchaulet merged 5 commits intomainfrom
fix-delete-by-query-conflict
Oct 29, 2025
Merged

[Fleet] Fix delete by query conflict#241001
nchaulet merged 5 commits intomainfrom
fix-delete-by-query-conflict

Conversation

@nchaulet
Copy link
Member

@nchaulet nchaulet commented Oct 28, 2025

Summary

Resolve #236637

Since we introduced the task to persist lask_known_status we got more conflict when deleting agents, those conflicts where not handled properly that PR aims to fix that.

Run the flaky test runner and it seems to fix the issue.

@nchaulet nchaulet self-assigned this Oct 28, 2025
@nchaulet nchaulet added the Team:Fleet Team label for Observability Data Collection Fleet team label Oct 28, 2025
@nchaulet nchaulet requested a review from a team as a code owner October 28, 2025 14:27
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@nchaulet nchaulet added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Oct 28, 2025
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#9652

[✅] x-pack/platform/test/fleet_api_integration/config.space_awareness.ts: 25/25 tests passed.

see run history

@nchaulet nchaulet added backport:version Backport to applied version labels v9.1.7 v9.3.0 v9.2.1 and removed backport:skip This PR does not require backporting labels Oct 28, 2025
q: '*',
refresh: true,
ignore_unavailable: true,
})
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't we use retries here too?

@nchaulet nchaulet requested a review from juliaElastic October 28, 2025 16:03
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @nchaulet

@nchaulet nchaulet enabled auto-merge (squash) October 28, 2025 18:24
@nchaulet nchaulet merged commit 1109b60 into main Oct 29, 2025
14 checks passed
@nchaulet nchaulet deleted the fix-delete-by-query-conflict branch October 29, 2025 07:46
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 9.1, 9.2

https://github.com/elastic/kibana/actions/runs/18900719884

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
9.1 Backport failed because of merge conflicts

You might need to backport the following PRs to 9.1:
- [Fleet] Fix reusable integration multiplespace validation (#236428)
9.2 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 241001

Questions ?

Please refer to the Backport tool documentation

This was referenced Oct 29, 2025
nchaulet added a commit to nchaulet/kibana that referenced this pull request Oct 29, 2025
(cherry picked from commit 1109b60)

# Conflicts:
#	x-pack/platform/test/fleet_api_integration/apis/space_awareness/helpers.ts
nchaulet added a commit to nchaulet/kibana that referenced this pull request Oct 29, 2025
(cherry picked from commit 1109b60)

# Conflicts:
#	x-pack/platform/test/fleet_api_integration/apis/space_awareness/helpers.ts
@nchaulet
Copy link
Member Author

💚 All backports created successfully

Status Branch Result
9.2
9.1

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

nchaulet added a commit that referenced this pull request Oct 29, 2025
# Backport

This will backport the following commits from `main` to `9.1`:
- [[Fleet] Fix delete by query conflict
(#241001)](#241001)

<!--- Backport version: 10.1.0 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Nicolas
Chaulet","email":"nicolas.chaulet@elastic.co"},"sourceCommit":{"committedDate":"2025-10-29T07:46:23Z","message":"[Fleet]
Fix delete by query conflict
(#241001)","sha":"1109b60f11d012f293d2a2b693c19a41c734b237","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","backport:version","v9.3.0","v9.1.7","v9.2.1"],"title":"[Fleet]
Fix delete by query
conflict","number":241001,"url":"https://github.com/elastic/kibana/pull/241001","mergeCommit":{"message":"[Fleet]
Fix delete by query conflict
(#241001)","sha":"1109b60f11d012f293d2a2b693c19a41c734b237"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","9.2"],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/241001","number":241001,"mergeCommit":{"message":"[Fleet]
Fix delete by query conflict
(#241001)","sha":"1109b60f11d012f293d2a2b693c19a41c734b237"}},{"branch":"9.1","label":"v9.1.7","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.2","label":"v9.2.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
nchaulet added a commit that referenced this pull request Oct 29, 2025
# Backport

This will backport the following commits from `main` to `9.2`:
- [[Fleet] Fix delete by query conflict
(#241001)](#241001)

<!--- Backport version: 10.1.0 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Nicolas
Chaulet","email":"nicolas.chaulet@elastic.co"},"sourceCommit":{"committedDate":"2025-10-29T07:46:23Z","message":"[Fleet]
Fix delete by query conflict
(#241001)","sha":"1109b60f11d012f293d2a2b693c19a41c734b237","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","backport:version","v9.3.0","v9.1.7","v9.2.1"],"title":"[Fleet]
Fix delete by query
conflict","number":241001,"url":"https://github.com/elastic/kibana/pull/241001","mergeCommit":{"message":"[Fleet]
Fix delete by query conflict
(#241001)","sha":"1109b60f11d012f293d2a2b693c19a41c734b237"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","9.2"],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/241001","number":241001,"mergeCommit":{"message":"[Fleet]
Fix delete by query conflict
(#241001)","sha":"1109b60f11d012f293d2a2b693c19a41c734b237"}},{"branch":"9.1","label":"v9.1.7","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.2","label":"v9.2.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
qn895 pushed a commit to qn895/kibana that referenced this pull request Oct 30, 2025
ana-davydova pushed a commit to ana-davydova/kibana that referenced this pull request Nov 3, 2025
albertoblaz pushed a commit to albertoblaz/kibana that referenced this pull request Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v9.1.7 v9.2.1 v9.3.0

Projects

None yet

4 participants