Skip to content

Fix for false ERROR status during long vPro activation#491

Draft
ipsita-npg wants to merge 4 commits intomainfrom
vpro-activation-fix-host-error-state
Draft

Fix for false ERROR status during long vPro activation#491
ipsita-npg wants to merge 4 commits intomainfrom
vpro-activation-fix-host-error-state

Conversation

@ipsita-npg
Copy link
Contributor

@ipsita-npg ipsita-npg commented Jan 27, 2026

Description

Problem statement:
During vPro activation, the synchronous/blocking AMT activate command blocks for 30-60+ seconds, preventing lastCheckTimestamp updates during activation process. This causes the health check to report STATUS_NOT_READY, incorrectly transitioning the host to ERROR state in the UI

Fixes # (issue)

  • Implement async activation with statusUpdateCallback mechanism
  • Update lastCheckTimestamp during long activation process
  • Report proper activation status (ACTIVATING/ACTIVATED/FAILED) to dm-manager

Validation :
Thorough validation is required since the existing logic and design have been modified.
This scenario can be properly validated after a failure: Activation status transitions to ACTIVATION_FAILED and the UI shows “Failed” or "Error" instead of progress

Complete failed activation:
UI status transitions from “In Progress” to “Failed”, Agent count remains 10/10 and Host shows an activation error (not an agent error) ----> This scenario needs to be validated

Any Newly Introduced Dependencies

Please describe any newly introduced 3rd party dependencies in this change. List their name, license information and how they are used in the project.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes
  • I have not introduced any 3rd party dependency changes
  • I have performed a self-review of my code

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant