Cherry-pick: Fixing reset activity data on deployment initialization#6227
Open
guimafelipe wants to merge 3 commits intorelease/1.8-stablefrom
Open
Cherry-pick: Fixing reset activity data on deployment initialization#6227guimafelipe wants to merge 3 commits intorelease/1.8-stablefrom
guimafelipe wants to merge 3 commits intorelease/1.8-stablefrom
Conversation
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
guimafelipe
force-pushed
the
user/felipeda/cherry-pick-reset-activity-1.8
branch
from
b802efc to
7190c07
Compare
Wrap the telemetry reset behavior changes with containment gates using WINAPPSDK_CHANGEID_12345 so the fix can be rolled back if needed. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
guimafelipe
force-pushed
the
user/felipeda/cherry-pick-reset-activity-1.8
branch
from
7190c07 to
57501f2
Compare
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
agniuks
approved these changes
Feb 19, 2026
Contributor
|
we should merge and rebuild once #6245 is in |
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cherry-pick of #5865 to release/1.8-stable.
Summary
On the method
Initializefrom the deployment manager, the two inside methodsInstallLicencesandDeployPackageswere resetting the activity data without setting it back, causing loss of data on our telemetry measurements.Install licences
Moved the
Reset()call from inside the license-file loop to the beginning of the method, keeping the install stage info while clearing stale data.Deploy packages
Added a
Reset()call at the beginning of the method and movedSetInstallStage(AddPackage)inside the per-package loop (after the per-packageReset()).Original PR: #5865