Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Nov 4, 2025

When enrollment fails it retries indefinitely. This is fine when triggered from CLI but when running from Action it can lead to confusing outcomes (e.g Migration never finishes)

This PR addresses this in 2 ways
First it sets 3 retries for action triggered enrollments
Second it handles 401 as something that we should not retry on.

Previously indefinitely in agent activity
image

With fix:
image

Fixes: #10909


This is an automatic backport of pull request #10946 done by Mergify.

@mergify mergify bot requested a review from a team as a code owner November 4, 2025 09:07
@mergify mergify bot added the backport label Nov 4, 2025
@mergify mergify bot requested review from swiatekm and ycombinator and removed request for a team November 4, 2025 09:07
@github-actions github-actions bot added Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team Team:Elastic-Agent Label for the Agent team skip-changelog labels Nov 4, 2025
@elasticmachine
Copy link
Collaborator

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/elastic-agent (Team:Elastic-Agent)

@elasticmachine
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

cc @michalpristas

@michalpristas michalpristas merged commit b7550ce into 9.2 Nov 4, 2025
23 checks passed
@michalpristas michalpristas deleted the mergify/bp/9.2/pr-10946 branch November 4, 2025 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport skip-changelog Team:Elastic-Agent Label for the Agent team Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants