Skip to content

Conversation

@shahneerali
Copy link
Contributor

@shahneerali shahneerali commented Jan 7, 2026

When reading the GPA status.json file, it compares the GPA Version, we also need GPA VMExtension to check GPA status reported time. This is to report GPA is running or not and the status file up-to-date. Two scenarios could be detected by this improvement:

  1. GPA service was down, hopefully it is in audit mode for WS. GPA VMExtension could report it (TBD report warning in subStatus)
  2. When the cx re-opt-in MSP feature with GPA VMExtension, the VM Extension should ignore the stale status.josn file.

Copy link
Collaborator

@ZhidongPeng ZhidongPeng left a comment

Choose a reason for hiding this comment

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

Let's use the similar logic as version check, then we could share most of the code.

@shahneerali shahneerali dismissed ZhidongPeng’s stale review January 8, 2026 18:43

implemented suggestion

@shahneerali shahneerali dismissed ZhidongPeng’s stale review January 8, 2026 21:59

implemented suggestions

@shahneerali shahneerali enabled auto-merge (squash) January 8, 2026 22:13
@shahneerali shahneerali merged commit 2b089ca into Azure:dev Jan 8, 2026
12 checks passed
@shahneerali shahneerali deleted the dev branch January 8, 2026 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants