Skip to content

Store last successful MS host for agent reconnection fallback - #14002

Draft
Pearl1594 wants to merge 5 commits into
4.20from
store-lastsuccessful-MS
Draft

Store last successful MS host for agent reconnection fallback#14002
Pearl1594 wants to merge 5 commits into
4.20from
store-lastsuccessful-MS

Conversation

@Pearl1594

Copy link
Copy Markdown
Contributor

Description

The issue is that once a bad hosts configuration update made, there's no way to get the Agents back in Up state without SSH-ing on each of them and manually updating the agent.properties file (because they're not listening to any new update from a control plane, since they're not connected anymore).

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • Build/CI
  • Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

mprokopchuk and others added 3 commits August 28, 2026 10:39
* Store last successful MS host for agent reconnection fallback

* Add newline at end of CapacityManagerImplTest.java
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 51.85185% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 16.33%. Comparing base (6136ac9) to head (d42e2aa).

Files with missing lines Patch % Lines
...gent/src/main/java/com/cloud/agent/AgentShell.java 65.00% 5 Missing and 2 partials ⚠️
agent/src/main/java/com/cloud/agent/Agent.java 0.00% 6 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               4.20   #14002      +/-   ##
============================================
+ Coverage     16.32%   16.33%   +0.01%     
- Complexity    13556    13564       +8     
============================================
  Files          5669     5669              
  Lines        501399   501425      +26     
  Branches      60902    60904       +2     
============================================
+ Hits          81847    81913      +66     
+ Misses       410390   410341      -49     
- Partials       9162     9171       +9     
Flag Coverage Δ
uitests 4.14% <ø> (ø)
unittests 17.19% <51.85%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@DaanHoogland

Copy link
Copy Markdown
Contributor

it seems we have been working in the same area @Pearl1594 , #14000 is about agent rebalancing for local agents and #14001 about rebalancing remote agents. Can you have a look at those as well, please?

@Pearl1594

Copy link
Copy Markdown
Contributor Author

It seems so - but they don't clash. I can have a look at them. Thanks @DaanHoogland

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants